Review Request: Comic applet using Plasma::Widgets

[Aaron J. Seigo]

On Tuesday 10 March 2009, Matthias Fuchs wrote:
> On Dienstag 10 März 2009 16:47:08 Aaron J. Seigo wrote:
> > On Friday 06 March 2009, you wrote:
> > > > On 2009-03-02 11:46:24, Aaron Seigo wrote:
> > > > > looks good...
> > > > >
> > > > > on a semi-related note, i really think offering a configuration
> > > > > option for the style of arrows is a mistake. the arrows on hover is
> > > > > much more in the spirit of Plasma and the graphics for them look so
> > > > > much better. it offer the maximal space for content as well. i
> > > > > really see no measurable benefit to offering the configuration
> > > > > option other than "because we can". meanwhile, there's more code to
> > > > > maintain and more buttons and levers for users to manage. just my
> > > > > 0.02
> > >
> > > Sorry, I missed your comment.
> > >
> > > Initially I also thought about removing that configuration option, but
> > > then I thought about touch screens. In touch screens the hovering icons
> > > would most likely (have no way to test that) be a pain to use.
> >
> > you can just put the cursor on the item of course, but what i think
> > *probably* should happen in such cases is that we have a standard button
> > widget for these use cases which switches internally what it does based
> > on the type of device. in both cases (desktop, touchscreen) a
> > configuration option doesn't make sense, only changing the presentation
> > does. so a configuration option is a bandage over the challenge rather
> > than an actual solution.
>
> Having the most useful or the least useless "option" (not necessarily
> configure-able)  on by default is for sure the best thing.

having it not in the config dialog is at least a step in the right direction 
imho, but leaving it in the code will likely lead to bitrot.

> The button you are talking about is Plasma::PushButton?

in this case it would be a navigation button that would probably use 
Plasma::PushButtons internally, but handle what a nav interface should do.

-- 
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 Qt Software

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20090310/5cde31e5/attachment.sig