[Kde-accessibility] Review Request 125042: Fix top buttons (close button, hide numpad button)

[Jeremy Whiting]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125042/#review84836
-----------------------------------------------------------

Ship it!


I think since the author hasn't made a commit in over a year and it's in playground anyway, this should be good to go in my opinion.

- Jeremy Whiting


On Sept. 4, 2015, 5:48 a.m., Fredrick Brennan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125042/
> -----------------------------------------------------------
> 
> (Updated Sept. 4, 2015, 5:48 a.m.)
> 
> 
> Review request for KDE Accessibility.
> 
> 
> Repository: kvkbd
> 
> 
> Description
> -------
> 
> Fix top buttons (close button, hide numpad button)
> 
> 
> Diffs
> -----
> 
>   src/main.cpp 49ba346 
>   src/kvkbdapp.cpp 825979a 
>   src/kvkbdapp.h 498f842 
>   src/kbdtray.cpp a9cf135 
>   src/kbddock.cpp 4800022 
>   src/mainwidget.cpp 5db552e 
>   src/standart.xml 7b66c89 
>   src/themeloader.cpp f3d8e60 
>   src/vbutton.cpp 7f7cead 
> 
> Diff: https://git.reviewboard.kde.org/r/125042/diff/
> 
> 
> Testing
> -------
> 
> Hello,
> 
> The top buttons of kvkbd weren't working properly on 
> 
> * Qt: 4.8.7
> * KDE Development Platform: 4.14.11
> 
> No tooltips appeared, they had no labels and they moved if the numpad was enabled/disabled.
> 
> This patch adds labels, fixes tooltips (so people can actually figure out what they do, gray/red isn't very helpful) and makes it so they no longer move when numpad enabled/disabled.
> 
> 
> Thanks,
> 
> Fredrick Brennan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-accessibility/attachments/20150904/2fbbaab1/attachment.html>