Skanlite (was Glimpse) and libksane

[Aaron J. Seigo]

On Friday 04 April 2008, Kåre Särs wrote:
> > scanner to use? ;) attached patch fixes this
>
> I have actually thought about that too, but I just haven't done it before.
> Thanks :)

np

> > * there is no standard quit action. another pet peeve of mine. and
> > another patch for you to consider is attached.
>
> What is this one for? (no visible change that I noticed)

so i can hit Ctrl-Q and it exists just like every other well behaved "regular" 
KDE app ... my fingers are well trained, and there's a reason it's called 
K*Standard*Action::quit() ;)

> > * what is the ">>>> 20 Shut of lamp <<<<" thing on the Other Options tab?
>
> You have found a sane option that libksane can't handle :)

ah, ok .. curious: how does one go about making libksane handle it? =) e.g. 
which class(es) would i look into first?

> Maybe I should add debug printout to the terminal and just don't show the
> option name.

yes, probably a good idea. it doesn't look very polished in the UI.

> > * the layout of the widgets in the tab pages could be nicer, though
> > that's a bit of a detail item. there isn't a Panes and Frames section in
> > the hig, but there is this:
> > http://wiki.openusability.org/guidelines/index.php/Design_and_Layout:Layo
> >ut
> >
> >:Dialogs
>
> One thing that makes layout hard here is that I don't know in advance what
> options are available and their types.

that's fine: use a QGridLayout insert all the labels in column 0 with 
alignment Qt::AlignRight; for all the widgets to the right of the labels, 
make sure each row of them is in their own hboxlayout (with 0 margins) and 
insert that layout into column 1. voila.

> > * any though to moving to toolboxes eventually instead?
>
> Why not.

=)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 194 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080404/486bcdc5/attachment.sig>