No, I agree with you in that case, that sounds like a bug in my patch and I'll check it out this weekend when I can get back to coding.
Thanks, Jon On Fri, Mar 2, 2018 at 12:28 PM, Andrzej Wolski <[email protected]> wrote: > Jon, > > I probably didn't express myself clearly. What I mean is a situation when > *only* enabled layer if F.Paste and then you disable "Pads Front". Now > nothing is visible on the board, but footprints are still selectable. > > In other words, when no single item belonging to footprint is visible, > footprints should not be selectable. > > Do you still disagree with me? > > Andrzej > > W dniu 2018-03-02 o 15:42, Jon Evans pisze: > > Hi Andrzej, > > This was my intention, which is why I said I was prepared for other people > to have other opinions :-) > > I think that you should still be able to select footprints even if the > "front pads" is hidden from layers like the paste layer, *unless* you are > in high contrast mode. > > -Jon > > On Fri, Mar 2, 2018 at 3:53 AM, Andrzej Wolski <[email protected]> > wrote: > >> I've tried this patch, and there is a small issue: if you have only eg >> front paste layer enabled and front pads are hidden, footprint is still >> selectable. >> >> Andrzej >> >> >> W dniu 2018-02-27 o 04:11, Jon Evans pisze: >> >> This patch changes the selection logic for footprints to fix a reported >> issue[1] and to make the behavior more logical to me. >> >> I know that correct selection behavior is something of a personal >> preference, so I'm ready to be flamed :-) >> >> The new behavior: >> >> A footprint may be selected if: >> 1) The corresponding "Footprints" switch is on in the Items tab, AND >> 2) One or more of the layers that the footprint draws on is visible >> >> This means that if all of the layers are turned off, footprints are not >> selectable (fixes the bug), but it also means that now footprints can be >> selected if any draw layers are visible (for example, if you have only >> F.Mask enabled, you can select a footprint that has solder mask and is on >> the front layer). >> >> Before anyone complains, this is only if high-contrast mode is turned >> OFF. When it is on, you can still only select items that *only* exist on >> that layer (to make moving silkscreen around easier, for example) >> >> Even though this adds some more for-loops to selection filtering, I have >> not noticed any performance hits on some selection of large boards that I >> tested. More testing is welcome. >> >> [1] https://bugs.launchpad.net/kicad/+bug/1751960 >> >> -Jon >> >> >> _______________________________________________ >> Mailing list: https://launchpad.net/~kicad-developers >> Post to : [email protected] >> Unsubscribe : https://launchpad.net/~kicad-developers >> More help : https://help.launchpad.net/ListHelp >> >> >> >> _______________________________________________ >> Mailing list: https://launchpad.net/~kicad-developers >> Post to : [email protected] >> Unsubscribe : https://launchpad.net/~kicad-developers >> More help : https://help.launchpad.net/ListHelp >> >> > >
_______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
