> On May 13, 2012, 7:14 p.m., Inge Wallin wrote: > > libs/widgets/KoZoomController.h, line 124 > > <http://git.reviewboard.kde.org/r/104928/diff/1/?file=64127#file64127line124> > > > > While 'document' may be what it's called in the zoom controller, I > > think it is confusing. Granted, the size of the document is not clearly > > defined in a page app, but I still think it makes more sense to call it > > workareaSize() or something. At least that's not possible to confuse with > > any size from KoDocument (if there is anything like that). In general, > > reusing names for different concapts is a reciepe for confusion.
...reusing names for different _concepts_ is a _recipe_ for confusion. - Inge ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104928/#review13786 ----------------------------------------------------------- On May 13, 2012, 1:31 a.m., C. Boemann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104928/ > ----------------------------------------------------------- > > (Updated May 13, 2012, 1:31 a.m.) > > > Review request for Calligra. > > > Description > ------- > > KoPageApp applications Stage and Flow used to have extra space reserved > around the page/slide so objects could be put there. However that made it > really hard for the user to work with. > > This patch changes that so there si no space around the page/slide. But.. > instead you can drag objects outside the page/slide without any limit at all > > > Diffs > ----- > > libs/kopageapp/KoPACanvas.cpp 4a4748c > libs/kopageapp/KoPAView.h 01fbdf1 > libs/kopageapp/KoPAView.cpp 4d13dfa > libs/kopageapp/KoPAViewMode.cpp 6981382 > libs/widgets/KoZoomController.h 4f4c6a2 > libs/widgets/KoZoomController.cpp e10c5b4 > stage/part/KPrViewModeNotes.h b707770 > stage/part/KPrViewModeNotes.cpp c951ade > > Diff: http://git.reviewboard.kde.org/r/104928/diff/ > > > Testing > ------- > > I've tested zooming in and out in Stage normal, notes, and sorter viewmodes, > and in Flow. > > I've tested that running presentations still work, however I've not tested > presenterview. > > I've tested moving shapes outside the page/slide in both Stage and Flow. > > > Thanks, > > C. Boemann > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
