> On June 8, 2012, 5:23 a.m., Thorsten Zachmann wrote: > > Please commit after fixing the details I noticied. If I miss someting on > > the inline stuff lets discuss it before committing. > > > > One thing I noticed which is not due to your code but only visible when > > your code is there is that if you enter the first/last page in the widget > > the actions do not get enabled/disabled and therefore it shows the old > > state of the actions, e.g. when entering the last page the next and last > > action should be disabled and the first/previous should be enabled.
No, the inlining was only there for historic reasons, and me too lazy to move over into the cpp file, after everything worked :)) Well, not everything works, as you pointed out, did not catch that before. Looking into the kopa code handling setting the active page I have not yet seen why the action updating is not reached in that case :/ - Friedrich W. H. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105180/#review14507 ----------------------------------------------------------- On June 7, 2012, 6:14 p.m., Friedrich W. H. Kossebau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105180/ > ----------------------------------------------------------- > > (Updated June 7, 2012, 6:14 p.m.) > > > Review request for Calligra. > > > Description > ------- > > Currently in Stage/Flow the status bar just has a read-only display for > current page number and pages count. > Which also does not tell if the current pages are Master pages or not. > > Attached patch fixes that, by turning the label into a display that turns > into an editor, like the zoom controller. > > Commit log: > It has 2 states: > in first state it displays > * type of slide/page (incl. if Master one) > * the number of the current page/slide > * the number of the page/slide count > in second state it offers control to > * go to first/previous/next/last slide/page > * go by entered number to slide/page > > Does not use the icons of the original go* actions, as they look to > complex > > > Diffs > ----- > > libs/kopageapp/CMakeLists.txt a353892 > libs/kopageapp/KoPAView.h 01fbdf1 > libs/kopageapp/KoPAView.cpp 6fccdb0 > libs/kopageapp/widgets/KoPageNavigator.h PRE-CREATION > libs/kopageapp/widgets/KoPageNavigator.cpp PRE-CREATION > libs/kopageapp/widgets/KoPageNavigatorButton.cpp PRE-CREATION > libs/kopageapp/widgets/KoPageNavigatorButton_p.h PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/105180/diff/ > > > Testing > ------- > > Used in Flow and Stage. > > > Screenshots > ----------- > > Display mode > http://git.reviewboard.kde.org/r/105180/s/600/ > Edit mode > http://git.reviewboard.kde.org/r/105180/s/601/ > > > Thanks, > > Friedrich W. H. Kossebau > >
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
