Review Request 108367: Complete keyboard preview rework for KControl
Xuetian Weng
wengxt at gmail.com
Sun Jan 13 17:22:09 GMT 2013
> On Jan. 13, 2013, 12:27 p.m., Xuetian Weng wrote:
> > You may want to look this:
> > https://github.com/fcitx/kcm-fcitx/blob/master/layout/keyboardlayoutwidget.cpp
> >
> > It's a more featured widget in fcitx (but the code I mentioned is only depends on qt/xkb).
> >
> > Your implementation is still, hardcode the key position, while this implementation is parsing the xkb model for drawing the keyboard image, one screenshot would explain it:
> > http://susepaste.org/3952607
> >
> > I'd admit your key string is much better :) , but feel free to take my code to improve it
>
> Manuel Tortosa wrote:
> Actually for my needs, this huge generated layout is not suitable at all but will be interesting to extract some parts to, at least, make 101/102/104/105 keyboards looks slighty diifferent. Will take a look. Also our implementation is a direct modification of KDE's one, so the patch will affects only 2 files.
BTW another interesting part of my code is it also handles keyboard event, make you can "see" what you're pressing on the screen. I guess that part maybe more useful/practical to you.
- Xuetian
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108367/#review25363
-----------------------------------------------------------
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/20130113/25790a61/attachment.htm>
More information about the kde-core-devel
mailing list