Hi Dennis, your pull request has just been applied to git master.
One additional request: could you please have a look at our documentation of module levels and update it to reflect your changes? You will find it in doc/usermanual/darkroom/modules/tone/levels.xml Like with your code changes just make a pull request. Thanks a lot Ulrich Am 06.01.2013 20:15, schrieb Dennis Gnad: > Hi Jo, > > I implemented autoadjust and 3 colorpickers for the levels module, the > colorpickers do immediately alter the module->params. > > Currently I set them in an expose event of the gui, but need to check if > the data from the picker changed to prevent an endless loop when calling > dt_dev_add_history_item inside the expose function, which might not be > the right way to do it? For me everything works ok though... > > If this isn't the right way, where would I catch the data from the > pickers to alter parameters? > > The current code can be seen there (sorry, it is not rebased to master..): > https://github.com/bluesceada/darktable/commit/a6a685cbe2138f585ca8691943e42e9ff8684e2d > > Thanks for the info in advance! > -Dennis > (bluesceada in IRC) > ------------------------------------------------------------------------------ Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft MVPs and experts. ON SALE this month only -- learn more at: http://p.sf.net/sfu/learnnow-d2d _______________________________________________ darktable-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/darktable-devel
