← Back to team overview

kicad-developers team mailing list archive

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

 

One thing I would like to see (unless I missed it) is a reset to default
color feature.  This would be useful for legacy canvas users where
getting the colors correct is critical for decent board/footprint canvas
rendering.

On 2/22/2017 3:36 AM, Maciej Sumiński 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
> 


Follow ups

References