Review Request: Add support for wheel events in the page navigator in the status bar (Stage/Flow)

[Commit Hook]

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


This review has been submitted with commit 7d5c6428d73bcd46d0e54b0128dd5e3d3231ab64 by Friedrich W. H. Kossebau to branch master.

- Commit Hook


On June 14, 2012, 4:12 p.m., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105229/
> -----------------------------------------------------------
> 
> (Updated June 14, 2012, 4:12 p.m.)
> 
> 
> Review request for Calligra, Yue Liu and Thorsten Zachmann.
> 
> 
> Description
> -------
> 
> I found that I sometimes would like to scroll the pages with the status bar page navigator.
> So this patch adds support for switching the pages with wheel events.
> It makes the page navigator behave similar to a vertical scroll bar (so not like a normal integer editor),
> as it seems similar in the UI concept to me:
> * control buttons on the left and right (not up/down)
> * page number like a representative/symbol of the current page, and pages are ordered with lower page numbers first
> This way it feels more natural to me (had it first differently).
> 
> But there are some things I am not decided about:
> * should the wheel events also be dealt with if the mouse hovers the buttons?
> * proper behavior in Right-to-left environment is?
> * what to do if the page editor has focus?
>   a) ignore wheel event
>   b) exit edit mode (dropping any changes), then scroll as in display mode
>   c) take current edited number, exit edit mode, select page, then scroll as in display mode
>   d) change the edited number and stay in edit mode, but do not activate a page
>   e) change the edited number and stay in edit mode, activate the page with the number
> 
> Implemented is currently b), because it felt to me better and more consistent.
> Currently only on pressing Enter an edited number is picked up, set respective page selected and the editor left.
> Why would one start enter a new number and then switch to use the mouse wheel? Switching to the wheel is more like
> cancelling the editing mode, not continuing.
> 
> What is your opinion?
> 
> 
> Diffs
> -----
> 
>   libs/kopageapp/widgets/KoPageNavigator.cpp 2ecdc9a 
>   libs/kopageapp/widgets/KoPageNavigatorButton_p.h f7fdc26 
> 
> Diff: http://git.reviewboard.kde.org/r/105229/diff/
> 
> 
> Testing
> -------
> 
> Scrolled with a few slides, with and without page number editing mode.
> 
> 
> Thanks,
> 
> Friedrich W. H. Kossebau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120615/6aebc38c/attachment.htm>