Re: Testers wanted for Sidebar Editors Patch

[Ruben Vermeersch <ruben savanne be>]

Hi Karl, thanks for testing!

On zo, 2008-08-03 at 22:07 +0100, Karl Relton wrote:
> Hi had a quick bash. Here are some initial observations:
> 
> 1) When the user hits the 'Edit' button (or double-clicks a photo), I
> think the Sidebar is going to have to auto-switch to show the edit
> buttons. Still showing tags won't cut it for novice users.

That's fixed now.

> 2) Desaturate, Sepia, Soft Focus and Autocolour all need to bring up
> 'OK/Cancel' buttons in the side-bar, to give the user the option to
> abort if they don't like the look of the preview.

It all depends on whether you want these to be 'one-click actions' or
not. I intentionally made them instant (like they were), to get a quick
workflow. Offcourse, you can always undo your edit by switching back to
the original version, but still, this isn't ideal (we lack undo and it's
quite hard to do currently). Am open to what people think on this issue.

> 3) It wasn't that obvious to my wife that she could click on 'no
> constraint' to set a constraint for crop.

The whole UI is a bit wonky (for some reason GTK makes the dropdown
enormous). I need to figure out what's going on there.

> 4) The constraint for crop is fancy in that a 4x6 can give you either a
> 4x6 or a 6x4 depending on how you draw your selection box. In one sense
> thats nice, but it confused my wife. She would prefer seperate choices
> when setting the constraint. Perhaps have separate radio buttons to
> choose portrait or landscape, or just double up inthe choice list.

Interesting, I should consult the usability guys on this.

> I think this is sidebar style is a good way to go - my wife coped with
> it pretty well (thats a good test!), except for the points noted above.

Awesome! And thanks again!

> Cheers
> Karl
> 
> > I am looking for some testers will to try out the new sidebar
> > editors.
> > This patch contains most of my summer of code work.
> > 
> > The patch can be found here:
> > http://files.savanne.be/linux/patches/f-spot/f-spot-sidebar-editors.patch
> > 
> 
> 


--
Ruben Vermeersch (rubenv)
http://www.savanne.be