← Back to team overview

kicad-developers team mailing list archive

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

 

Seppe,

Unfortunately, we do not backport new features to the stable branch, as
we do not have enough manpower to maintain it, especially now when the
code is significantly diverged. The stable branch only receives bug
fixes, whereas the product branch is reserved for development.

Regards,
Orson

On 09/25/2017 10:28 AM, Seppe Stas wrote:
> Cool, thanks!
> 
> Any chance this (and/or the double click feature) could be released in
> KiCad 4? I would love to start using it in the stable release...
> 
> Seppe
> 
> On Mon, Sep 25, 2017 at 10:01 AM, Maciej Sumiński <maciej.suminski@xxxxxxx>
> wrote:
> 
>> Seppe,
>>
>> Thank you for your contribution. As the context menu problem is not
>> related to your changes, I decided to commit the patches.
>>
>> I observe the issue with right-click context menu on Arch Linux x86_64
>> with both Xfce and awm. I cannot tell whether the problem occurs in the
>> stable version, as I do not have a ready binary to test. If time
>> permits, I will test it a bit later.
>>
>> Regards,
>> Orson
>>
>> On 09/23/2017 03:58 PM, Seppe Stas wrote:
>>> Hey Maciej, what OS are you using? Also, do you have the same issue in
>> KiCad 4.0.7?
>>>
>>> On 22 Sep 2017, 17:41 +0200, Maciej Sumiński <maciej.suminski@xxxxxxx>,
>> wrote:
>>>> I retract it, apparently the same problem occurs without your patch
>> applied.
>>>>
>>>> On 09/22/2017 05:39 PM, Maciej Sumiński wrote:
>>>>> 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
>>
>>
> 


Attachment: signature.asc
Description: OpenPGP digital signature


References