Fix: issue 10611 add copy id feature to Panoramax

[avijitdas126]

Changes

• Added "Copy ID" button/icon to relevant UI components.
• Implemented clipboard functionality to copy the ID with a single click.

Benefits

• Improves user experience by simplifying ID retrieval.
• Aligns Panoramax features with those of Mapillary for consistency.

Reference

Closes #10611

[gedankenstuecke]

While not being a maintainer, I don't think this actually closes what the maintainers suggested to be the solution in #10611 as the PR as-is leads to a degradation of the workflow for mapillary.

The suggestion was to copy the feature that was introduced for mapillary (c.f. #10611 (comment)), which allows adding the image ID as a new key/value pair to the features, instead of just copying the image ID to the clipboard. Could you try to expand on that mapillary feature to instead make it work for the other image providers too?

[avijitdas126]

Thanks for the feedback! I’ve updated the PR to ensure that image IDs are added as key/value pairs for all relevant image providers, not just Mapillary. Let me know if there are any further improvements needed.

…

On Thu, 6 Feb 2025, 7:14 pm Bastian Greshake Tzovaras, < ***@***.***> wrote: While not being a maintainer, I don't think this actually closes what the maintainers suggested to be the solution in #10611 <#10611> as the PR as-is leads to a degradation of the workflow for mapillary. The suggestion was to copy the feature that was introduced for mapillary (c.f. #10611 (comment) <#10611 (comment)>), which allows adding the image ID as a new key/value pair to the features, instead of just copying the image ID to the clipboard. Could you try to expand on that mapillary feature to instead make it work for the other image providers too? — Reply to this email directly, view it on GitHub <#10697 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BBMNGLHFAPUXL4T46RNIIHT2ONRLPAVCNFSM6AAAAABVSRP64OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMZZHA3TAMZTGE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[gedankenstuecke]

I don't think you've pushed any changes to this PR yet?

Also: please check that the button for adding images doesn't overlap with existing UI, c.f. the high-res button for panoramax

[avijitdas126]

I'll enhance the UI implementation and introduce the image ID as a new key/value pair to streamline the process.

[tordans]

Pleas double check the hints that I posted in #10611 (comment) and #10611 (comment)