[Okular-devel] Review Request 118236: Save the okular printer settings in a config file, to make them persistent across sessions

[Oliver Sander]

> On May 24, 2014, 2:39 p.m., Albert Astals Cid wrote:
> > I'm undecided about this, i know it is something people want, but it looks extremely fragile the way stuff is saved/restored.
> > 
> > Do you think you could have a look and see if Qt5 offers this feature? And maybe if not implement it there? Qt5 is muuuuuuuuuch more open thatn Qt4 so implementing small features like this should not be that hard. I'll also add John Layt to the review, he's been working on Qt5 Printing stuff and maybe has time to comment a bit.

I support doing it in Qt5 if you think that is better in the long run (and in that case I withdraw this review request).

Unfortunately it cannot be me who implements that in Qt5.  My Qt skills are non-existent, and my time constraints don't allow me to learn it.

However, I'd be happy to send some sort of donation or a box of the famous Aachen chocolate to whoever implements it properly.


- Oliver


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118236/#review58398
-----------------------------------------------------------


On May 24, 2014, 2:39 p.m., Oliver Sander wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118236/
> -----------------------------------------------------------
> 
> (Updated May 24, 2014, 2:39 p.m.)
> 
> 
> Review request for Okular and John Layt.
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> Save the okular printer settings in a config file, to make them persistent across sessions.
> 
> This is the patch submitted by user 'Hulahup' in Comment 4 of https://bugs.kde.org/show_bug.cgi?id=308367
> 
> I extended it to also store the duplex mode of the printer.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt b869aec 
>   conf/printer.h PRE-CREATION 
>   conf/printer.cpp PRE-CREATION 
>   conf/printer.kcfg PRE-CREATION 
>   conf/settings_printer.kcfgc PRE-CREATION 
>   part.cpp d9f1682 
> 
> Diff: https://git.reviewboard.kde.org/r/118236/diff/
> 
> 
> Testing
> -------
> 
> Duplex mode appears to be correctly stored.  Yay!
> 
> 
> Thanks,
> 
> Oliver Sander
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20140526/291f68ea/attachment.html>