Review Request: Cleaned up "simple mode", grouping buttons

Raphael Kubo da Costa rakuco at freebsd.org
Wed Nov 23 22:37:01 UTC 2011


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


You may want to explicitly add Evan Teran (the KCalc maintainer) to the list of people in this review request.

It'd be good if you could add a screenshot of the proposed changes to the request too.


kcalc.ui
<http://git.reviewboard.kde.org/r/103197/#comment7155>

    Hmm, hardcoding fonts is not good.


- Raphael Kubo da Costa


On Nov. 21, 2011, 4:02 p.m., Eike Krumbacher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103197/
> -----------------------------------------------------------
> 
> (Updated Nov. 21, 2011, 4:02 p.m.)
> 
> 
> Review request for KDE Utils.
> 
> 
> Description
> -------
> 
> Hi!
> 
> This patch cleans up "simple mode". I rearranged some buttons and hide the leftmost column in this mode. X³ as a function is now in the column, where x², x^y and so on can be found. I think, it is now easier to find the functions. The rightmost colum with memory functions is hidden in this mode too.
> Furthermore, I rearranged the order of highlighted buttons by pressing the tab-button. This patch is highly inspired by the mockup of the reporter of bug  https://bugs.kde.org/show_bug.cgi?id=254369 .
> 
> I hope you enjoy
> 
> Eike
> 
> 
> This addresses bug 254369.
>     http://bugs.kde.org/show_bug.cgi?id=254369
> 
> 
> Diffs
> -----
> 
>   kcalc.h 38b3036 
>   kcalc.cpp d7a943b 
>   kcalc.ui 31778b9 
> 
> Diff: http://git.reviewboard.kde.org/r/103197/diff/diff
> 
> 
> Testing
> -------
> 
> * Tested the tab order
> * tested hide/show of modes
> 
> 
> Thanks,
> 
> Eike Krumbacher
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20111123/5e5676c8/attachment.html>


More information about the Kde-utils-devel mailing list