Review Request 108367: Complete keyboard preview rework for KControl

[Manuel Tortosa]

> 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. 
>     
>

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


- Manuel


-----------------------------------------------------------
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/20130126/dd086b0f/attachment.htm>