In message <[email protected]> on Wed, 16 Jan 2013 15:14:52 +0100, 
Pascal Obry <[email protected]> said:

pascal> 
pascal> Richard,
pascal> 
pascal> > pascal> I think we should initialize the aspect ratio (in the GUI 
combo) from
pascal> > pascal> the actual crop of the image, this should be quite easy.
pascal> > 
pascal> > +1, but not enough in my opinion.  The actual crop will only tell you
pascal> > what the actual crop is...  your aspect ratio preset might have been
pascal> > "free" just as well as "5:4", which one should the clipping module
pascal> > choose for you when you get back in develop mode?
pascal> 
pascal> 5:4, if by change your free cropping end-up being 5:4 why not display
pascal> that to the user when he come back to the iop module. At least that's
pascal> far less annoying than today behavior and it does not break the crop.

...
Yeah ok, using the crop size to get an actual aspect ratio makes
sense.

What I wonder, though, is what you're thinking of giving as feedback
to the user?  5:4 becomes a float of 1.25, the quick and dirty way
would be to display 1.25:1 (or 0.8:1 if it was flipped).

Cheers,
Richard
(yeah, I care about user feedback...  or rather, I care about avoiding
having users who will complain about getting weird aspect ratios, in
this particular case ;-))

-- 
Richard Levitte                         [email protected]
                                        http://richard.levitte.org/

"Life is a tremendous celebration - and I'm invited!"
-- from a friend's blog, translated from Swedish

------------------------------------------------------------------------------
Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
and much more. Keep your Java skills current with LearnJavaNow -
200+ hours of step-by-step video tutorials by Java experts.
SALE $49.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122612 
_______________________________________________
darktable-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/darktable-devel

Reply via email to