Skip to content

Add a copy tags to clipboard button to item popup in the map view.#2798

Merged
CollinBeczak merged 2 commits intomaproulette:mainfrom
watmildon:main
Mar 18, 2026
Merged

Add a copy tags to clipboard button to item popup in the map view.#2798
CollinBeczak merged 2 commits intomaproulette:mainfrom
watmildon:main

Conversation

@watmildon
Copy link
Contributor

It's often helpful to be able to take the tags from an item and add them to a new item in another editor. This seems like the most sensible place to put it but let me know.

It will copy tags that are not OSM id tags and don't have values over 255 characters (those are often left over from custom challenge creation).

I got this half running on Windows so I could verify my change but you absolutely do not want the hacks that are those changes.

Let me know what you think!

20260311-0218-31.6997651.mp4

Copy link
Collaborator

@CollinBeczak CollinBeczak left a comment

Choose a reason for hiding this comment

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

Looks good to me, thank you for this feature!

@CollinBeczak CollinBeczak merged commit 36e4853 into maproulette:main Mar 18, 2026
1 check passed
@CollinBeczak
Copy link
Collaborator

Let me know if you want any changes to the copy clipboard output

@watmildon
Copy link
Contributor Author

I'll certainly keep an eye on it. I think having a pattern folks can use for which keys are info (leading underscore or at sign?) and which are copyable tags would be nice but I didn't want to add too much policy to start :-D

@govvin
Copy link

govvin commented Mar 19, 2026

Cool feature!

I remember how we "abused" the mustache tags in an old challenge, and eventually the copyable shortcode came up.

Maybe it would be nice to also have an icon for it in the toolbar, so users don't always have to tap the feature for the properties window to pop up?

@CollinBeczak CollinBeczak mentioned this pull request Mar 19, 2026
Sign up for free to 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.

3 participants