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