hi. On Tue, Oct 16, 2012 at 7:35 PM, Loic GUIBERT <[email protected]> wrote: > Hello, > because of a mail delivery issue notice, I don't know if it was received so I > send it again and complete it... > > > > i try a thing on zonesystem module : add a "show zones numbers" checkbox > option that is preset independent and saved into dt_conf > Like this, users wanting numbers and those not, can be satisfied. > What do you think about it ?
i don't like the numbers because they make the gui look cluttered and i don't see any particular need for them. same goes for the checkbox. also we usually don't have checkboxes in darkroom mode, but use a two-entry combo box with more descriptive entries. i don't think this patch goes the right direction (which would be away from crazy button creep to make it look less like an airplane cockpit). j. > Is the principle good ? If the checkbox option is not in dt gui guide line, i > can implement it differently. > > (pushed on git://github.com/rikles/darktable.git (branch: zonesystem)) > > > Another thing : > I don't know if preset independent options can be useful for some other > modules, > but if it does, it would be necessary to uniform that principle and make a > guide line for this. > Like, for exemple, adding a generic option panel and a button on module > header bar to show it. > This would be usable by whatever module and only shown on modules using it. > This is a proposition only... > > Regards, > > -- > Loic > > > > > De : Henrik Andersson > Envoyé le : Lundi 8 octobre 2012 20h50 > Objet : Re: [darktable-devel] [darktable] Zonesystem (#55) > > > Ansel's zone system is devided into 11 zones > > Our zone system module will only apply to ansels system if you have the > zonbar divided into 11 zone, > all other configurations is not applyable and there for i don't think this is > a good idea, were the number > only is matched Ansel system in 1 configuration of 32 or whatever the roof is > for division. > > Also i agree with Jo about cluttering the ui. > > /H > > 2012/10/8 Loic GUIBERT > Hi, >>Ok for the cherry pick. nice to help you. >> >> >> >>About zone number drawing : for me, this is usefull. I use ZoneSystem a lot , >>capturing picture and developping it, in Ansel Adams "philosophy". >>But I understand that it can be not so usefull for every one. >>So, i've made some correction on drawing code and it doesn't have artifacts >>anymore (as far as i tested it ;-)). And, i've added the current zone output >>box number drawing. >> >>Like there are not, (or i didn't found them) module settings, independant of >>presets, i've done this. >> >>What do you thing about it ? >>git://github.com/rikles/darktable.git (branch: zonesystem) >>I didn't send a pull request because I want to speak about it before. >> >> >> >>PS : I've also fix a little problem on zone highlighting and zone correction >>markers drawing on mouse leave. >> >> >>PS2 : Question, can it be usefull to have some presets independant settings >>on modules ? Saved quietly on the fly, this can permits to customize modules >>work/presentation. >> >> >> >> >> >>Regards >> >>-- >> >>Loic >> >> >> >> De : hanatos >>Envoyé le : Dimanche 23 septembre 2012 12h27 >>Objet : Re: [darktable] Zonesystem (#55) >> >> >> >>hey, i like the +=1 part, but the number drawing introduces a lot of >>graphical noise that i'm not sure is of any practical use for the image? also >>it leads to some drawing artifacts when moving the zones (numbers overlap >>input/output boxes and sometimes disappear in the grey of the output box..). >>so i cherry picked only 4508a51. >> ------------------------------------------------------------------------------ Don't let slow site performance ruin your business. Deploy New Relic APM Deploy New Relic app performance management and know exactly what is happening inside your Ruby, Python, PHP, Java, and .NET app Try New Relic at no cost today and get our sweet Data Nerd shirt too! http://p.sf.net/sfu/newrelic-dev2dev _______________________________________________ darktable-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/darktable-devel
