← Back to team overview

kicad-developers team mailing list archive

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

 

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
> > > > > >
> > > > > >
> > > > > >
> > > > > > _______________________________________________
> > > > > > 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
> > >
> >
> >
> >
> >
> > _______________________________________________
> > 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