kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #28012
Re: [PATCH] Fix layer color swatches in Linux
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.
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
>
Follow ups
References