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].
