Review Request: Move creation of print dialog to KoView

[Dag Andersen]

> On June 12, 2012, 10:14 a.m., C. Boemann wrote:
> > Uhm how does this play in to the KoPrintDialog.
> > 
> > I'm not trying to prevent your usecase, but on the other hand I don't think we should make our API even more complicated, than it unfortunately already is. so I just want to be sure we have though this through.

The exact problem I have now (sheets too) is that number of pages can change dependent on settings in the print dialog like orientation, paper size etc which triggers the need to update this in the dialog.
The basic problem here is that the print system is designed for wysiwyg apps and does not cater well for the rest of us.
I could not find an less intrusive solution but I am open to suggestions.


- Dag


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


On June 12, 2012, 8:35 a.m., Dag Andersen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105225/
> -----------------------------------------------------------
> 
> (Updated June 12, 2012, 8:35 a.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> View centric apps (plan, sheets) need better control of the print dialog than what is possible today.
> This patch moves the creation of QPrintDialog into a virtual method in KoView to enable apps to access it or create their own dialog.
> 
> 
> Diffs
> -----
> 
>   libs/main/KoMainWindow.cpp 6720a95 
>   libs/main/KoView.h ba46c02 
>   libs/main/KoView.cpp af3aba8 
> 
> Diff: http://git.reviewboard.kde.org/r/105225/diff/
> 
> 
> Testing
> -------
> 
> Printing still works with words
> 
> 
> Thanks,
> 
> Dag Andersen
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120612/1dd6b31e/attachment.htm>