yep, but i agree the original description was not very good ;) On Sun, Sep 7, 2014 at 5:34 PM, Carlos R. Mafra <[email protected]> wrote: > On Sun, 7 Sep 2014 at 17:16:25 +0800, David Maciejak wrote: >> On Sun, Sep 7, 2014 at 5:12 PM, Carlos R. Mafra <[email protected]> wrote: >> > On Sun, 7 Sep 2014 at 10:56:15 +0800, David Maciejak wrote: >> >> This patch is updating the gradient texture user experience >> >> by auto selecting the first color in the list when possible >> >> and fixing the autoselection after a delete action. >> > >> > Could it happen that a user inadvertently changes the color >> > by simply visiting that panel if the first auto-selected >> > color is different from the current one he's using? >> > >> >> In fact what that is called gradient colors is just a list of colors >> with hue/saturation/brightness value. >> That patch is loading the first one from the list (color, hue, >> saturation, brightness) and setting the 3 sliders according to that >> value instead of using some default hardcoded values. >> The default color value on the top-right is never modified. > > Ok, thanks for the clarification. I always need to be extra sure > before applying patches and your original description let me > wondering. > > Thanks, I'll apply it now (and add the extra comment in the commit log) > > > -- > To unsubscribe, send mail to [email protected].
-- To unsubscribe, send mail to [email protected].
