← Back to team overview

kicad-developers team mailing list archive

Re: BUG FIX: module text move


Le 05/08/2013 23:19, Dick Hollenbeck a écrit :
On 08/05/2013 04:01 PM, Dick Hollenbeck wrote:

Rev 4271 probably reverses out a change you made to my original selection clarification work.

I found I could not move a module text reference with M hotkey if it was tucked under a
neighboring module.

My change reverts to how it was originally written, however, it causes more popup menus
especially if you click on an ambiguous position which as both module text and same
module.   I don't think the answer to this second problem is a patch reversal, because
we'd be going in circles then.

What we probably need is a text positioning tool.  So that text is only in play when when
you have that tool in play.

No, I revoke that suggestion.   The answer is simply to turn off front or back text
visibility, then the module text will not be selected and the selection clarification menu
frequency goes down drastically.

So I now stand bolder in defense of this commit, which again, fixes the ability to move
module text positioned under a different module.


No problem.
I recently have some trouble with the text move function.
The selection is better with your fix.

Turn off front or back text has a drawback: you cannot identify the component. This is mainly the case when you have a lot of SMT components in a small area, all having the same shape. In this case, activate the mode module is better (the module texts are not considered in this mode). If you want only to move texts (and not modules) just make a silkscreen layer the active layer. Currently, if there is more than one text and its corresponding footprint (for instance a track), the clarification menu is shown. However, a more restrictive filtering could be done, and something like a "text positioning tool" could be just make by using a restrictive filtering ( for instance do not include tracks in search) when a silkscreen layer is the active layer.

Jean-Pierre CHARRAS

Follow ups