Skip to content

TagsWidget: Disable editor if multiple objects are selected - #2353

Merged
dl3sdo merged 2 commits into
OpenOrienteering:masterfrom
dl3sdo:tag-editor-disabling
Apr 1, 2025
Merged

TagsWidget: Disable editor if multiple objects are selected#2353
dl3sdo merged 2 commits into
OpenOrienteering:masterfrom
dl3sdo:tag-editor-disabling

Conversation

@dl3sdo

Copy link
Copy Markdown
Member

If multiple objects were selected the tag editor would just operate on the so-called 'first selected object'.
If a group of objects is selected the user does not know which of them is the first selected object.

If multiple objects were selected the tag editor would just operate on
the so-called 'first selected object'.
If a group of objects is selected the user does not know which of them
is the first selected object.
@dl3sdo

Copy link
Copy Markdown
MemberAuthor

Allowing tag editing when multiple objects are selected also could give the user the false impression that bulk editing of tags is supported by Mapper.

@dl3sdo
dl3sdo requested a review from dg0ytApril 1, 2025 07:34
Comment threadsrc/gui/widgets/tags_widget.cpp Outdated
Co-authored-by: Kai Pastor <dg0yt@darc.de>
@dl3sdo

Copy link
Copy Markdown
MemberAuthor

@dg0yt: thank you for the review and proposed changes. My thought of keeping the && object was driven by some paranoia whether map->getNumSelectedObjects() == 1 will always assure that map->getFirstSelectedObject() returns a valid object.

@dg0ytdg0yt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When we have selected objects, there must exist a first selected object.

@dl3sdo
dl3sdo merged commit 803f195 into OpenOrienteering:masterApr 1, 2025
@dl3sdodl3sdo changed the title TagsWidget: Disable editing if multiple objects are selectedTagsWidget: Disable editor if multiple objects are selectedApr 1, 2025
@dl3sdo
dl3sdo deleted the tag-editor-disabling branch April 2, 2025 06:42
@dl3sdodl3sdo added this to the v0.9.6 milestone Jul 1, 2025
lpechacek pushed a commit to lpechacek/mapper that referenced this pull request Oct 15, 2025
…nteeringGH-2353)
If multiple objects were selected the tag editor would operate on
the so-called 'first selected object' which the user does not know.
---------
Co-authored-by: Kai Pastor <dg0yt@darc.de>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@dl3sdo@dg0yt