The problem with selecting these integers based on commonly-used aspect ratios, is that the only real use for this feature would be in "freehand" mode when cropping to an *unusual* aspect ratio; the common ones like 16:9 are represented in the "aspect" combo box and it is much easier to select the "16:9, HDTV" option to constrain the ratio.
-- August Schwerdfeger aug...@schwerdfeger.name On Fri, Nov 16, 2018 at 2:51 PM Jan Ingwer Baer <jib...@web.de> wrote: > On 16-Nov-18 18:26, Heiko Bauke wrote: > > Hi, > > > > Am Freitag, 16. November 2018 schrieb Jan Ingwer Baer: > >> If DT shows the aspect ratio it should try to find the nearest integral > >> fraction (of course with values below 30) like 3:2, 16:9, ... > >> > >> > >> For your exampel it should show 8.2:5, or it should use one of the > >> common denominators x:2, x:3, x:4, x:9 ... > > > > I don't think that this is a good idea, as the user can not know how > close the actual aspect ratio is to the approximate value that is indicated > in the UI. > > > > Heiko > > > Of course DT should not round, but it should show the Ratio with an > integer for the small-side. For me showing 8.2/5 or 3.28/2 (with the > pixel-ratio of the OP) is much more meaningful then 1.62/1. I think it > should be possible to select a usable integer numerator/denominator > based on the fractional ratio. > > If the user wants to create a crop with one of the common aspect ratios > showing it as simple float makes it very challenging to find the right > values. For example : The user wants to create a crop with 16/9. > Displaying the aspect ratio as 1.77/1 is not a big help. > > The challenge is to find the right integer. My idea is to define ratio > ranges for selection : > > 1.2 - 1.4 : set denominator to 3 gives 3.6/3 - 4.2/3 ( ~4/3) > 1.4 - 1.6 : set denominator to 2 gives 2.8/2 - 3.2/2 ( ~3/2) > 1.6 - 1.9 : set denominator to 9 gives 14.4/9 - 17.1/9 (~16/9) > > > ___________________________________________________________________________ > darktable developer mailing list > to unsubscribe send a mail to > darktable-dev+unsubscr...@lists.darktable.org > > ___________________________________________________________________________ darktable developer mailing list to unsubscribe send a mail to darktable-dev+unsubscr...@lists.darktable.org
