← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Right click option to change layer and render color

 

Hi Seppe,

There is still one issue: the context menu works correctly if you right
click on a layer label, but if you do the same on a swatch box, then
menu immediately disappears. The only way to keep it open is to hold
right mouse button.

Regards,
Orson

On 09/22/2017 03:12 PM, Seppe Stas wrote:
> Hey Wayne
> 
> I fixed the coding policy violations. I blame my text-editor's trailing
> whitespace detector being broken all of a sudden
> <https://github.com/SublimeText/TrailingSpaces/issues/121> and the spaces
> thing being weird XD.
> 
> The updated patches are attached to this mail. Let me know if you find any
> other issues.
> 
> Greetings
> Seppe
> 
> 2017-09-22 13:35 GMT+02:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
> 
>> Seppe,
>>
>> Please fix your coding policy[1] violations.  I saw some missing spaces
>> between brackets and function call arguments along with some trailing
>> white space.
>>
>> Thanks,
>>
>> Wayne
>>
>> [1]:
>> http://docs.kicad-pcb.org/doxygen/md_Documentation_
>> development_coding-style-policy.html
>>
>> On 9/22/2017 5:21 AM, Maciej Sumiński wrote:
>>> Hi Seppe,
>>>
>>> This is a very good idea. I would have merged the patch, but you need to
>>> implement GERBER_LAYER_WIDGET::OnLayerRightClick(), as in the current
>>> state the code does not compile.
>>>
>>> Regards,
>>> Orson
>>>
>>> On 09/21/2017 06:25 PM, Seppe Stas wrote:
>>>> This patch makes it possible to change the color of layers and render
>>>> options using the right click menu. See attached picture:
>>>> [image: Inline afbeelding 1]
>>>>
>>>> The motivation for this is the current "double left click / middle
>> click"
>>>> being somewhat obscure. I've been using KiCad for over 2 years and
>> recently
>>>> found out about the double click method by looking at KiCad's source
>> code
>>>> (I stumbled on the middle mouse button method earlier). At one point I
>> even
>>>> borrowed a mouse from my colleague to change layer colors when I needed
>> to
>>>> open a 6 layer board since I typically use a trackpad without a middle
>>>> mouse button. My colleagues, most of whom have been using KiCad for
>> longer
>>>> than me, didn't even know changing the layer/render color is possible.
>>>>
>>>> Being able to change properties of a selected item using the right mouse
>>>> button is a lot more common in modern GUI applications and is more
>>>> consistent with how the rest of pcbnew works.
>>>>
>>>> Greetings
>>>> Seppe Stas
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> 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
>>
> 
> 
> 
> _______________________________________________
> 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