Modify

Opened 11 months ago

Closed 10 months ago

Last modified 10 months ago

#23105 closed enhancement (fixed)

[Patch] Add action to select shared/common child objects

Reported by: Woazboat Owned by: team
Priority: normal Milestone: 23.08
Component: Core Version:
Keywords: Cc:

Description (last modified by Woazboat)

It would be quite useful to have an action that allows you to select all common child objects that are shared between all currently selected objects.

E.g.:

  • select all nodes that are shared between ways
  • select all common members of relations

https://github.com/JOSM/josm/pull/125

Attachments (0)

Change History (9)

comment:1 by Woazboat, 10 months ago

Description: modified (diff)
Summary: Add action to select shared nodes of ways[Patch] Add action to select shared/common child objects

comment:2 by taylor.smock, 10 months ago

Milestone: 23.08

comment:3 by taylor.smock, 10 months ago

I was going to apply this today.

I didn't, since when a single way is selected, the behavior is very similar to SelectWayNodesAction from utilsplugin2.

I'm kind of inclined to (a) set the default shortcut to ctrl+shift+n (same as utilsplugin2) and modify utilsplugin2 to not add that action when running a JOSM version with this patch.

Question:

  • Did you intend to replace or partly replace SelectWayNodesAction? Or should we modify updateEnabledState to check to see if there are at least two selected objects?

comment:4 by taylor.smock, 10 months ago

NVM. A use case I didn't think about for SelectWayNodesAction was to select all way nodes in the current selection.

comment:5 by taylor.smock, 10 months ago

Resolution: fixed
Status: newclosed

In 18814/josm:

Fix #23105: Add action to select shared/common child objects (patch by Woazboat, modified)

Modifications are as follows:

  • Basic test added
  • Icon added (copied and modified from selectall.svg)
  • default methods for IPrimitive.getChildren were moved to the appropriate locations
  • Lint cleanups in modified files

Additional notes:

  • The behavior when only one way is selected is very similar to SelectWayNodesAction from utilsplugin2

comment:6 by Woazboat, 10 months ago

Sorry, I didn't have much time in the last weeks to follow up on this or the comments on the github pull request. Thanks for modifying and applying the patch!

comment:7 by gaben, 10 months ago

Does it have documentation yet?

comment:9 by gaben, 10 months ago

Thank you!

@Woazboat, could you please detail the function in a few sentences on the wiki?

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.