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

Thorsten Zachmann t.zachmann at zagge.de
Mon May 14 06:11:09 BST 2012



> 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.
> 
> Inge Wallin wrote:
>     ...reusing names for different _concepts_ is a _recipe_ for confusion.

I think this has nothing to do with this review request. Boemann just uses the terms that are used in the used classes. As this is totally unrelated to the review request this should if handled done separately.


- Thorsten


-----------------------------------------------------------
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/20120514/5cd4abfb/attachment.htm>


More information about the calligra-devel mailing list