kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #27099
Re: [PATCH] Pcbnew, plot dialog: grey out advanced net attributes if Gerber X2 is unchecked.
I didn't know about wxUpdateUIEvent at all. I'll have a look and try to
rewrite it using it when I've got time.
Thanks!
On Thu, Jan 5, 2017 at 2:10 PM, Wayne Stambaugh <stambaughw@xxxxxxxxx>
wrote:
> On 1/5/2017 9:03 AM, Chris Pavlina wrote:
> > Okay, thanks for letting me know. If this was policy, I forgot about it.
>
> It's not in our UI policy but it probably should be. I know I've
> mentioned it before. I'll add that to my todo list.
>
> >
> > Do you want me to rework the patch to use wxUpdateUIEvent?
>
> Don't bother unless you have the time. It might be a good exercise for
> Diego to learn how to use wxUpdateUIEvent.
>
> >
> > On Thu, Jan 05, 2017 at 08:57:14AM -0500, Wayne Stambaugh wrote:
> >> Thanks Chris. In the future, please make sure that contributions use
> >> wxUpdateUIEvent for updating UI states. A single code path for updating
> >> UI element states prevents potential state conflicts when setting the
> >> state in multiple wxCommandEvents. We have had issues with this in the
> >> past where a UI element state would get toggled in one command event and
> >> toggled again in a different command event causing the UI element state
> >> to be incorrect.
> >>
> >> Cheers,
> >>
> >> Wayne
> >>
> >> On 1/4/2017 5:18 PM, Chris Pavlina wrote:
> >>> Looks good to me, I pushed it. Thank you for your contribution to
> KiCad.
> >>>
> >>> It's easy for patches to slip through the cracks - in the future, don't
> >>> be afraid to bump a patch a bit sooner, say within a week or two.
> >>> Apologies for losing it.
> >>>
> >>> On Wed, Jan 04, 2017 at 07:45:40PM +0000, Diego Herranz wrote:
> >>>> Hi,
> >>>>
> >>>> Did anyone have a chance to look at this?
> >>>>
> >>>> Many thanks.
> >>>>
> >>>> Regards,
> >>>> Diego
> >>>>
> >>>> On Tue, Nov 22, 2016 at 8:10 AM, Diego Herranz <
> >>>> diegoherranz@xxxxxxxxxxxxxxxx> wrote:
> >>>>
> >>>>>
> >>>>> m_useGerberNetAttributes is useless if m_useGerberX2Attributes is not
> >>>>> checked.
> >>>>> So disabled (greyed out) when Gerber X2 gets unchecked to make it
> clear to
> >>>>> the user.
> >>>>> ---
> >>>>> pcbnew/dialogs/dialog_plot.cpp | 20 ++++++++++++++++++++
> >>>>> pcbnew/dialogs/dialog_plot.h | 3 ++-
> >>>>> pcbnew/dialogs/dialog_plot_base.cpp | 4 +++-
> >>>>> pcbnew/dialogs/dialog_plot_base.fbp | 2 +-
> >>>>> pcbnew/dialogs/dialog_plot_base.h | 3 ++-
> >>>>> 5 files changed, 28 insertions(+), 4 deletions(-)
> >>>>>
> >>>>>
> >>>
> >>>
> >>> _______________________________________________
> >>> 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
>
References
-
[PATCH] Pcbnew, plot dialog: grey out advanced net attributes if Gerber X2 is unchecked.
From: Diego Herranz, 2016-11-22
-
Re: [PATCH] Pcbnew, plot dialog: grey out advanced net attributes if Gerber X2 is unchecked.
From: Diego Herranz, 2017-01-04
-
Re: [PATCH] Pcbnew, plot dialog: grey out advanced net attributes if Gerber X2 is unchecked.
From: Chris Pavlina, 2017-01-04
-
Re: [PATCH] Pcbnew, plot dialog: grey out advanced net attributes if Gerber X2 is unchecked.
From: Wayne Stambaugh, 2017-01-05
-
Re: [PATCH] Pcbnew, plot dialog: grey out advanced net attributes if Gerber X2 is unchecked.
From: Chris Pavlina, 2017-01-05
-
Re: [PATCH] Pcbnew, plot dialog: grey out advanced net attributes if Gerber X2 is unchecked.
From: Wayne Stambaugh, 2017-01-05