← Back to team overview

kicad-developers team mailing list archive

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

 

I got a response from Wayne saying he is ok with the changes (I suppose
the mail was meant to be sent here). Do you think the patches ready to
be merged? They seem complete to me, but I just want to confirm.

Regards,
Orson

On 02/20/2017 06:55 PM, Jon Evans wrote:
> Thanks Orson, no I don't mind changing to static consts!
> 
> Best,
> Jon
> 
> On Mon, Feb 20, 2017 at 12:50 PM, Maciej Sumiński <maciej.suminski@xxxxxxx>
> 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
>>
>>
> 


Attachment: signature.asc
Description: OpenPGP digital signature


Follow ups

References