← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Change from EDA_COLOR_T to COLOR4D and arbitrary color support

 

I think you should just be able to add the branch Maciej linked
earlier, and possibly do a pull -r on that.

2017-02-20 19:06 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
> Please post the rebased patch.  I like to take a look at it and test it.
>
> Thanks,
>
> Wayne
>
> On 2/20/2017 12:50 PM, Maciej Sumiński wrote:
>> Hi Jon,
>>
>> I have just tested the patch, and I really like it. I just need to test
>> it a bit longer, as there are numerous changes, but I am in favor of
>> merging the patch.
>>
>> I also applied one more patch changing a few defines (UNSPECIFIED,
>> BLACK, WHITE) to static consts, I hope you do not mind.
>>
>> If there are other people interested in testing, I have rebased the
>> changes [1] on the current master. There are also a few minor code
>> formatting fixes.
>>
>> Regards,
>> Orson
>>
>> 1. https://code.launchpad.net/~orsonmmz/kicad/+git/kicad/+ref/colors
>>
>> On 02/18/2017 09:24 PM, Jon Evans wrote:
>>> Hi all,
>>>
>>> Attached is a follow-up patch to the COLOR4D change above -- I reverted
>>> from using wxColourPickerCtrl back to wxBitmapButton in the eeschema color
>>> config, because I finally got my Windows 10 testing environment set up, and
>>> found out that for some reason wxColourPickerCtrl looks really ugly in
>>> Windows 10.
>>>
>>> Best,
>>> Jon
>>>
>>> On Fri, Feb 10, 2017 at 8:43 PM, Jon Evans <jon@xxxxxxxxxxxxx> wrote:
>>>
>>>> Hi all,
>>>>
>>>> Attached is a (rather large!) patch that changes the internal color
>>>> representation to COLOR4D across the codebase (except for places that
>>>> directly deal with wx, where wxColour is used).
>>>>
>>>> This patch also enables arbitrary color selection for schematic and symbol
>>>> library editor, as well as pcbnew in the GAL canvas.  GerbView and pcbnew
>>>> legacy canvas still use the old color picker.  Colors will be coerced into
>>>> the legacy palette when switching from GAL to legacy, in a way that tries
>>>> to preserve the hue and value.
>>>>
>>>> Colors are serialized to the settings in CSS format, because it supports
>>>> alpha and represents color components the same way COLOR4D does (as
>>>> floating-point values from 0 to 1)
>>>>
>>>> Tested on Linux.  Can't test on Windows or Mac yet, sorry.
>>>> I realize this is a large changeset and might take a while to review, so
>>>> just ping me if it stops applying on master and I'll update it.
>>>>
>>>> Best,
>>>> Jon
>>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> 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
>>
>
> _______________________________________________
> 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


References