← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Rearrange Render panel of PcbNew layer widget; add spacers

 

Before we do any more tweaking, we should definitely agree on the
visibility behavior and stick too it.   Otherwise this will be a never
ending cycle of changes.  I think what we have in place is a big
improvement any further tweaking be put off to v6.  This way we have the
time to perhaps do something more granular and let the user define their
own visibility settings.

Wayne

On 02/23/2018 03:46 PM, Jon Evans wrote:
> I'm fine with renaming to SMD pads.
> Right now the logic (implemented by Andrzej, not me by the way) assumes
> that all pads are associated with footprints, and if footprint display
> is turned off, hides pads (SMD and through hole)
> Seems like lots of people have different opinions on what those switches
> should do :-)
> 
> In my humble opinion, it would make more sense to hide everything *but*
> the pads when you uncheck the footprint boxes, but some other people
> really want the pads to be hidden so they can check for "stray traces"
> under the pads.
> 
> So, we can keep tweaking this, but maybe not until after RC1 so we stop
> delaying it any further :-)
> 
> -Jon
> 
> On Fri, Feb 23, 2018 at 3:38 PM, Jeff Young <jeff@xxxxxxxxx
> <mailto:jeff@xxxxxxxxx>> wrote:
> 
>     Hi Jon,
> 
>     Just playing with the new render palette.  It’s a big improvement.
> 
>     However, I noted that turning off Footprints turns off through-hole
>     pads.  Since we have a separate option for that, it’s
>     counter-intuitive to have Footprints also control it.  (And, when
>     one wants to see the copper and nothing else, you want pads without
>     footprints.)
> 
>     Also, it’s easy to think Front Pads or Back Pads should show them. 
>     I understand the distinction, but shouldn’t we name them Front SMD
>     Pads and Back SMD Pads to make it clearer?
> 
>     Cheers,
>     Jeff.
> 
>     > On 23 Feb 2018, at 15:35, jp charras <jp.charras@xxxxxxxxxx
>     <mailto:jp.charras@xxxxxxxxxx>> wrote:
>     >
>     > Le 23/02/2018 à 16:12, Jon Evans a écrit :
>     >> You can commit that, but I can't quite tell why there are two
>     different functions ReFillRender() and
>     >> SyncRenderStates(); they do mostly the same thing so it seems
>     best to just collapse them into one to
>     >> avoid bugs in the future.
>     >> The attached patch removes SyncRenderStates -- please let me know
>     if this makes sense to you also.
>     >>
>     >> Thanks,
>     >> -Jon
>     >>
>     >
>     > I committed you patch.
>     >
>     > It makes sense to me, and this is the right way to fix this issue.
>     > Sometimes code cleanup is needed.
>     >
>     > Thanks.
>     >
>     >>
>     >> On Fri, Feb 23, 2018 at 10:05 AM, jp charras
>     <jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>
>     <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>>>
>     >> wrote:
>     >>
>     >>    Le 23/02/2018 à 15:57, Jon Evans a écrit :
>     >>> If you add a call to ReFillRender() at the top of
>     PCB_LAYER_WIDGET::SyncRenderStates() instead of my
>     >>> change, does it fix it?
>     >>>
>     >>
>     >>    Yes, just adding the call fixes this issue.
>     >>    Can I commit this fix?
>     >>
>     >>
>     >>> On Fri, Feb 23, 2018 at 9:54 AM, jp charras
>     <jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>
>     >>    <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>>
>     <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>
>     <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>>>>
>     >>> wrote:
>     >>>
>     >>>      Le 23/02/2018 à 14:24, Jon Evans a écrit :
>     >>>      > Hi JP, good catch, the attached fixes it for me
>     >>>      >
>     >>>      > -Jon
>     >>>      >
>     >>>      > On Fri, Feb 23, 2018 at 7:18 AM, jp charras
>     <jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>
>     <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>>
>     >>>      <mailto:jp.charras@xxxxxxxxxx
>     <mailto:jp.charras@xxxxxxxxxx> <mailto:jp.charras@xxxxxxxxxx
>     <mailto:jp.charras@xxxxxxxxxx>>>
>     >>    <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>
>     <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>>
>     <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>
>     >>    <mailto:jp.charras@xxxxxxxxxx <mailto:jp.charras@xxxxxxxxxx>>>>>
>     >>>      > wrote:
>     >>>      >
>     >>>      >     Le 23/02/2018 à 01:04, Jon Evans a écrit :
>     >>>      >     > Here are two more quick patches addressing some of
>     the easier changes.
>     >>>      >     > I have some other bugs to look at that seem more
>     urgent, so I'm not going to work
>     >>    on the right-click
>     >>>      >     > menu or settings persistence.
>     >>>      >     >
>     >>>      >     > Thanks,
>     >>>      >     > Jon
>     >>>      >     >
>     >>>      >
>     >>>      >     Hi Jon,
>     >>>      >
>     >>>      >     There is a (minor but annoying) issue about patch:
>     >>>      >
>     >>>      >     "Only show microvia and blind/buried via settings if
>     they are enabled"
>     >>>      >
>     >>>      >     When I load a board that have microvia and
>     blind/buried vias allowed, the layer
>     >>    manager does not
>     >>>      >     show the microvia and blind/buried vias menuitems.
>     >>>      >
>     >>>      >     They are displayed only after opening and closing by
>     OK button the Design Rules menu.
>     >>>      >
>     >>>      >     (Tested on W7/32bits)
>     >>>      >
>     >>>      >     --
>     >>>      >     Jean-Pierre CHARRAS
>     >>>
>     >>>      With this patch, Pcbnew crashes when starting, but not always.
>     >>>      I am thinking m_Layers->ReFillRender(); is called too
>     early, when initializations are not yet
>     >>>      finished
>     >>>
>     >>>      In fact it is called before m_Layers is initialized.
>     >>>
>     >>>      The fix is easy, but when loading a file,
>     syncRenderStates(); is called
>     >>>      (files_io.cpp line 592)
>     >>>
>     >>>      Perhaps the root issue is SyncRenderStates() is missing
>     some initializations.
>     >>>
>     >>>
>     >>>
>     >>>      --
>     >>>      Jean-Pierre CHARRAS
>     >>>
>     >>>
>     >>
>     >>
>     >>    --
>     >>    Jean-Pierre CHARRAS
>     >>
>     >>
>     >
>     >
>     > --
>     > Jean-Pierre CHARRAS
>     >
>     > _______________________________________________
>     > Mailing list: https://launchpad.net/~kicad-developers
>     <https://launchpad.net/~kicad-developers>
>     > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>     > Unsubscribe : https://launchpad.net/~kicad-developers
>     <https://launchpad.net/~kicad-developers>
>     > More help   : https://help.launchpad.net/ListHelp
>     <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