← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Fix layer color swatches in Linux

 

On Tue, Feb 21, 2017 at 09:35:34AM -0500, Wayne Stambaugh wrote:
> My only issue with this change is that the tooltip letting the user know
> that a left button double click or a middle button click would allow
> them to change the color is gone.  Other than that, it looks great.  I
> like the use of Bind instead of Connect.  Bind has a much cleaner
> interface than Connect and according to the wx folks is the proper way
> to handle events.

The tooltip is useless, nobody is going to hover over them for long
enough to see it. I was just talking to someone this morning who never
found the tooltip because he was too busy clicking on the buttons to
make them work.

Every other color well in every other application opens its selection
dialog with a single left click. Why can't ours?

> 
> On 2/21/2017 8:04 AM, John Beard wrote:
> > Hi,
> > 
> > These patches fix the layer/render widget swatches in Linux under new
> > GTK+ toolkits. Fix for: https://bugs.launchpad.net/kicad/+bug/1605411
> > 
> > The patches remove the "button" nature of the swatches, since they
> > weren't actually actuated by a single click, so the button affordance
> > was misleading anyway. Also on OSX, the button was invisible, so it
> > just looked like a flat swatch anyway.
> > 
> > * Linux as it was:
> > https://drive.google.com/file/d/0BxVhl5qZbpYoZlZPeXV1Q0ttT2s/view
> > * OSX as it was:
> > https://launchpadlibrarian.net/274428737/Screen%20Shot%202016-07-22%20at%2019.40.14.png
> > * Linux after this patch: see attachment
> > 
> > This is followed by a refactor to pull the swatch logic out of the
> > layer widget into common, where it can be used by other clients, for
> > example the eeschema display color dialog, if wanted.
> > 
> > Patch #3 is a simple replacement of old WX Connect with Bind for
> > consistency in that file.
> > 
> > Cheers,
> > 
> > John
> > 
> > 
> > 
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help   : https://help.launchpad.net/ListHelp
> > 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp


Follow ups

References