Review Request: Comic applet using Plasma::Widgets

[Matthias Fuchs]

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.
Still I prefer having the option to configure it as at least I prefer a 
specific choice in some cases and I'm not sure what others think about it.

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

I took a note and will look into it when I'm motivated in that direction.