[haiku-commits] Re: r42231 - haiku/trunk/src/apps/showimage

  • From: "Humdinger" <humdingerb@xxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 19 Jun 2011 09:59:49 +0200

Hi Ryan!

Ryan Leavengood, Sat, 18 Jun 2011 18:28:30 -0400:
> On 6/18/11, Humdinger <humdingerb@xxxxxxxxxxxxxx> wrote:
> >
> > I also think changing shortcuts depending on the current mode is a 
> > bad
> > thing. Even more so in this example: consider that you can create a
> > selection without entering the selection mode. DEL then suddenly 
> > doesn't
> > clear the selection, but is trashing the image... Uncool. :)
> 
> Actually it doesn't depend on the mode, just whether there is a
> selection or not. It is still state dependent, just not mode
> dependent.

Yeah... I apparently tested with an older version of ShowImage. A 
ShowImage compiled from trunk won't start under alpha3rc. I'm looking 
into it when the real a3 is out. RSN(tm).
In any case, changing keyboard shortcuts is always bad, I think we all 
agree.

> Hmmm, with the current code the ShowImage window where an image was
> just trashed moves on to the next image.

Maybe the actual trashing could only be done after the user moves 
on/quits the app.
 
> But I think that would be OK. You trash an image,
> you get a chance to undo, then once you move on you have to take the
> extra step of opening the Trash manually to get it back. But that
> should be rare.

Exactly.

Regards,
Humdinger

--=-=--=-=--=-=--=-=--=-=--=-=--=-=--=-=--=-=--=-
      Deutsche Haiku News  -  Haiku Gazette
        http://haiku-gazette.blogspot.com

Other related posts: