Re: print dialog issue

Angelo Naselli <[email protected]>
Newsgroups gmane.comp.kde.gwenview
Message-ID <[email protected]>
Alle 01:37, giovedì 4 gennaio 2007, Aurelien Gateau ha scritto:
> Le Mardi 26 Décembre 2006 21:58, Angelo Naselli a écrit :
> > Hi,
> > I was reviewing our code and testing it I saw there are
> > some problems:
> >
> > 1) if keep ratio is enabled and it is the first time,
> >    wxh is 0.0x0.0, I know the ratio is kept, but i don't like it
> >    so the attached patch should solve the problem.
> 
> Sounds good, please commit.
> 
> > 2) if you want to scale the image the setting is saved but not restored,
> >    and since I wrote that code to use it i have to fix it ;)
> >    Anyway that's weird, I cannot understand why, the code seems to be ok,
> > the only way to fix i found is to change the mScale buttonGroupId, for
> > instance with the mFitToPage one in ui file (1<->2) or setting mScale as
> > default. Before doing this change I'd like to hear something more, can
> > anyone test and explain why?
> 
> I do not really understand what you suggest (can you provide a patch?).
> I had a look at the bug with gdb and it seems that setButton() does not select 
> the button (selectedId() returns -1 after it's called).
ehm, already committed a solution  :-$ , that sounds more as a workaround 
but it works, you can check the changes i commited concernig bug #111641.
I added notes on NEWS, don't know if that is the right way to add them, but
it will be easy to fix before releasing ;)
The PrintDialogPage::setOptions seemd to fail on 
opts["app-gwenview-scale"].toInt( &ok ) ok was false but the value into
gwenviewrc was ok (2 - now 3).
I suspect some other problem like that, I saw some new bug reports on
setting not kept...

Welcome back ;)
Angelo

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

_______________________________________________
Gwenview-general mailing list
Gwenview-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
https://lists.sourceforge.net/lists/listinfo/gwenview-general
signature.asc (application/pgp-signature, 189 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD8DBQBFnNylqEs9DA4DquARAuP2AJ9jroMlIrVC6IczYN4j289gSJEADQCgljmg
I1vpQ6Pvm5KhbW5fL09AUew=
=lqvZ
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.