Review Request 114006: Update min and max value at the same time in order to be more friendly with typing in the min/max box

Yichao Yu yyc1992 at gmail.com
Sat Nov 30 20:17:05 UTC 2013


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


Ping? Anyone have time to check this patch?

- Yichao Yu


On Nov. 21, 2013, 2:20 p.m., Yichao Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114006/
> -----------------------------------------------------------
> 
> (Updated Nov. 21, 2013, 2:20 p.m.)
> 
> 
> Review request for KDE Edu, Laurent Montel and Jeremy Whiting.
> 
> 
> Repository: kalzium
> 
> 
> Description
> -------
> 
> This patch makes the behavior while typing in the min/max boxes more friendly.
> 
> Currently if one types in the maximum value box, the minimum value will stuck at a small value due to the small value in the maximum value box before one finish typing. (I don't really call this a bug, but I think it is not friendly.)
> 
> This patch tries to update both the maximum and minimum values whenever one of them changes (in the right order to always respect the one that changes) so that the actual range displayed is always the closed possible to the input values. It also covers the cases when min/max values change from illegal values to legal values (max > min) in some other ways.
> 
> 
> Diffs
> -----
> 
>   src/spectrumviewimpl.h c596154 
>   src/spectrumviewimpl.cpp 0327202 
>   src/spectrumwidget.h 5117d26 
> 
> Diff: http://git.reviewboard.kde.org/r/114006/diff/
> 
> 
> Testing
> -------
> 
> Compiles and fixes the problem.
> 
> 
> Thanks,
> 
> Yichao Yu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20131130/cbad12d9/attachment.html>


More information about the kde-edu mailing list