Review Request 113413: Improved Keyboard Layout Preview

Andriy Rysin arysin at gmail.com
Fri Nov 15 01:13:33 GMT 2013


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


Hi Shivam, there's a bug report about the reversed positions of small/capital letters (https://bugs.kde.org/show_bug.cgi?id=327393), it looks like the small/capital letter positions on the 1st/2nd levels are correct in the screenshots you attached, but the 3rd/4th levels are reversed (at least in Ukrainian layout), not sure about 5th/6th levels in Neo 2. Could you please take a look? Thanks.

- Andriy Rysin


On Nov. 5, 2013, 3:22 p.m., shivam makkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113413/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2013, 3:22 p.m.)
> 
> 
> Review request for KDE Runtime, kde-workspace and Andriy Rysin.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> Improved keyboard layout preview 
> added geometry feature, multi-level keys (>4) and parsed geometry file using Boost c++ libraries and tool tip showing symbol names.
> my GSoC-13 project !
> 
> 
> Diffs
> -----
> 
>   kcontrol/keyboard/preview/kbpreviewframe.cpp 40359f9 
>   kcontrol/keyboard/preview/keyboardpainter.cpp 0427102 
> 
> Diff: http://git.reviewboard.kde.org/r/113413/diff/
> 
> 
> Testing
> -------
> 
> 
> File Attachments
> ----------------
> 
> English layout default geometry
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/11/05/57ce3e15-26a9-46e2-ad5d-6073e04a5489__snapshot16.jpg
> german neo 2 layout default geometry
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/11/05/231ded09-9f22-44f5-aafe-6759c683b552__snapshot17.jpg
> ukranian layout microsoft natural geometry
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/11/05/8dca14f2-2d00-46d1-95a1-f7eca0ff5831__snapshot18.jpg
> 
> 
> Thanks,
> 
> shivam makkar
> 
>

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


More information about the kde-core-devel mailing list