Review Request 125042: Fix top buttons (close button, hide numpad button)

[Albert Astals Cid]

> On Sept. 4, 2015, 4:01 p.m., Jeremy Whiting wrote:
> > 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.
> 
> Fredrick Brennan wrote:
>     Cool :D What do you think of comment to Montel post above?

I am very sorry this was dropped.

If you need this application you should really try to port it to Qt5.

Do you need help with something?


- Albert


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


On Sept. 4, 2015, 11: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, 11:48 a.m.)
> 
> 
> Review request for KDE Accessibility.
> 
> 
> Repository: kvkbd
> 
> 
> Description
> -------
> 
> Fix top buttons (close button, hide numpad button)
> 
> 
> Diffs
> -----
> 
>   src/kbddock.cpp 4800022 
>   src/kbdtray.cpp a9cf135 
>   src/kvkbdapp.h 498f842 
>   src/kvkbdapp.cpp 825979a 
>   src/main.cpp 49ba346 
>   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/20170121/c02d2ca7/attachment.html>