Review Request 108367: Complete keyboard preview rework for KControl

Andriy Rysin arysin at gmail.com
Fri Feb 1 04:35:28 GMT 2013



> On Jan. 26, 2013, 3:13 a.m., Andriy Rysin wrote:
> > Manuel, I like how it looks in general with rounded corners and space between keys, but I must say that the font size is quite small. E.g. small letters are not easy to read even with good eyes, I wander if there's a way to reach the size of the preview you need without sacrificing readability. I also think that without 3rd/4th level it would be more readable but with 4 symbols in a square of this size it gets too crowded.
> > Also I am not sure if white vs lightblue provides enough differentiation between the levels. I wander if the key size is larger that would be a less of a problem.
> > 
> > Another thing is I am not sure we should just hardcode the font and size (although the size is hardcoded in original code). I wonder if we should calculate the key size and font size based on the preview window size. Start with something small to be able to fit smaller screens but if user wants to zoom in, he can just resize/maximize preview window and see larger picture.
> > 
> > On for the Weng's comment: I agree, hardcoded geometry is not good, we plan to parse geometry from xkb for next version, but I am not sure if Weng's code would be used. For keyboard layout preview we need to have a geometry from kcm UI which may not be applied yet, so we can't use active geometry. Although that might not be that big of a problem if we actually do use active one (if user switches keyboard model he would just have to apply before opening layout preview).
> 
> Manuel Tortosa wrote:
>     The preview you are seeing is exactly this: http://wstaw.org/m/2013/01/13/tribe55.jpg   cannot be bigger as it is part of an installer and thus, should fit on 1024x768 ( also the blue color is just a visual change to fit more into my installer), the size is really easy to change.
>     
>     About hardcodding the font and size, the font and size it is already hardcodded in the current code, in fact quite more hardcodded in the current one. I agree about parsing the model and geometry etc but maybe you can extract some interesant parts. 
>     
>     If you like i can apply some little parts like paintKey and paintLabeledKey which are already abstracted. 
>     
>
> 
> Manuel Tortosa wrote:
>     Ah, also you should consider to re-thnk the number of symbols used on each key: http://wstaw.org/m/2013/01/26/keyboard.jpg  that's German Keyboard - model Neo2
> 
> Xuetian Weng wrote:
>     You don't need to worry about that, it can parse any geometry even it's not active. My code can already do this.
> 
> Thomas L├╝bking wrote:
>     Reg. too small fonts: scale the key when hovering with the mouse? (or select / press it?)
>     What about altering the text position/sizes when a modifier is pressed?

Manuel, I think we use only 4 symbols per key, not sure how that one got generated, I'll take a look
Weng, that's cool, I'll ask Shivam who's working on layout preview to take a look and possibly borrow your code
Thomas, I think for preview asking a user to hover to be able to see is not perfect, if I open preview window I'd like to see the value of this preview (symbols on all keys) without additional hovering, especially if my screen allows it


- Andriy


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108367/#review26188
-----------------------------------------------------------


On Jan. 12, 2013, 5:13 p.m., Manuel Tortosa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108367/
> -----------------------------------------------------------
> 
> (Updated Jan. 12, 2013, 5:13 p.m.)
> 
> 
> Review request for kde-workspace.
> 
> 
> Description
> -------
> 
> I am working in a keyboard preview widget, based on kcontrol's one, for the Chakra Project's installer. If you are interested i can create a patch so it may replace the KDE one.
> 
> This one is 768x292 (the one in kcontrol is 1300+ wide, not suitable for lower resolutions), and it is almost done (i need to center the Ctrl, Alt and ALtGr texts using QTextMetric or whatever).
> 
> If you are interested i can create the patch, the relevant files are kbpreviewframe.cpp+h, the current code (need clean, etc) it is located here:
> 
> http://gitorious.org/chakra/tribe/blobs/info/src/widgets/kbpreview/kbpreviewframe.cpp
> http://gitorious.org/chakra/tribe/blobs/info/src/widgets/kbpreview/kbpreviewframe.h
> 
> The change is kinda big and it is my first attempt to create something with Qt so i prefer to wait some ideas before actually create the relevant patch.
> 
> 
> Diffs
> -----
> 
> 
> Diff: http://git.reviewboard.kde.org/r/108367/diff/
> 
> 
> Testing
> -------
> 
> Basically i have tested it in our installer and works.
> 
> 
> File Attachments
> ----------------
> 
> before
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/12/before.png
> after
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/12/after.jpg
> 
> 
> Thanks,
> 
> Manuel Tortosa
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130201/f3e9ba7e/attachment.htm>


More information about the kde-core-devel mailing list