Review Request: Improve Calculate/Stop buttons when folder Size is being calculated

[David Faure]

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


What I don't like about a single button, is that it creates a race.

The calculation takes too long, you want to abort it. You click on the button, and half a second before, the calculation actually ended. So your click triggers a "Calculate" again, and the annoying calculation that you wanted to stop, is starting again. It can be stopped a second time of course, but this still makes for an annoying user experience.

- David


On July 30, 2011, 2:51 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102149/
> -----------------------------------------------------------
> 
> (Updated July 30, 2011, 2:51 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> This patch improves the Calculate and Stop buttons in folder properties.
> Instead of having two buttons that are enabled/disabled accordingly, we only have one that toggles (technically there are still two buttons but it looks as if there was one)
> 
> I removed that "Calculating..." label when there is already a size shown, instead the stop button says "Stop Calculating" and somehow serves as "indicator".
> Also, I added a line-break after the "Calculating..." so the label doesn't change its size making the other elements moving around.
> 
> I don't know if the additional icons (view-refresh and process-cancel) add too much clutter to the visual interface since this features is not sooo important/frequently used(?) that the buttons need an icon.
> 
> 
> Diffs
> -----
> 
>   kio/kfile/kpropertiesdialog.cpp ba56f18 
> 
> Diff: http://git.reviewboard.kde.org/r/102149/diff
> 
> 
> Testing
> -------
> 
> Compiles and the buttons toggle (and react) fine.
> 
> 
> Screenshots
> -----------
> 
> Screenshot of the dialog while it is calculating
>   http://git.reviewboard.kde.org/r/102149/s/213/
> 
> 
> Thanks,
> 
> Kai Uwe
> 
>

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