kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #28072
Re: [PATCH] Change from EDA_COLOR_T to COLOR4D and arbitrary color support
-
To:
KiCad Developers <kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
From:
Maciej Sumiński <maciej.suminski@xxxxxxx>
-
Date:
Wed, 22 Feb 2017 17:56:12 +0100
-
Authentication-results:
spf=pass (sender IP is 188.184.36.46) smtp.mailfrom=cern.ch; lists.launchpad.net; dkim=none (message not signed) header.d=none;lists.launchpad.net; dmarc=bestguesspass action=none header.from=cern.ch;
-
In-reply-to:
<CA+qGbCA+=8Ftx=0HWT+gO3Uux5e_VCvPhQUmKuASde9uUUy46A@mail.gmail.com>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0
Great, so now they are in the master repository. Thank you for the patches.
Regards,
Orson
On 02/22/2017 02:10 PM, Jon Evans wrote:
> Yes, they are ready to merge.
>
> Best,
> Jon
>
> On Feb 22, 2017 03:37, "Maciej Sumiński" <maciej.suminski@xxxxxxx> wrote:
>
>> 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
>>>>
>>>>
>>>
>>
>>
>>
>> _______________________________________________
>> 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