Review Request: Change Stage and Flow to have infinite type canvas

[Inge Wallin]

> 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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120513/4cf60fdc/attachment.htm>