Copy location as <map-feature>

[prushforth]

• Change context menu to copy location as a map-feature.
• Change Util._pasteLayer to sniff for <map-feature, paste as a visible layer.

Closes #779

[prushforth]

I guess I should DRY the <map-feature> boilerplate, to avoid .js bloat. Also need to localize strings via mapml-extension PR.

[AliyanH]

How should we deal with the different "Coordinate system for location" options, for ex. I am not sure how helpful it would be to copy "tilematrix", "map" and "tile" as a <map-feature>. As pasting "map", and "tile" features causes an error, while pasting "tilematrix", displays the feature on the top left edge of the copied tile.

[prushforth]

How should we deal with the different "Coordinate system for location" options, for ex. I am not sure how helpful it would be to copy "tilematrix", "map" and "tile" as a <map-feature>. As pasting "map", and "tile" features causes an error, while pasting "tilematrix", displays the feature on the top left edge of the copied tile.

I think for now I will log a message to the console, open an issue to fix this when the <map-feature> PR gets merged.

[prushforth]

@AliyanH do you think the behavior is ok as far as pasting a new layer each time goes, or should it paste into an existing layer? Of course that will only work once the <map-feature> PR is merged...

[AliyanH]

@AliyanH do you think the behavior is ok as far as pasting a new layer each time goes, or should it paste into an existing layer? Of course that will only work once the <map-feature> PR is merged...

I think it makes sense to paste as a new layer with the localized label when pasting on to the viewer. Maybe in the future we can paste into layers through the layer control contextmenu?

Though the label you put for pasting features made me wonder, if we should change the default label for pasting layers/geojson with no label (to something similar: "Pasted layer"), currently it defaults to a localized string of: "Layer"
https://github.com/Maps4HTML/Web-Map-Custom-Element/blob/main/src/mapml/utils/Util.js#L447
https://github.com/Maps4HTML/Web-Map-Custom-Element/blob/main/src/mapml/utils/Util.js#L569

[prushforth]

Depends on Maps4HTML/mapml-extension#54 for localization.

[prushforth]

@AliyanH :

Though the label you put for pasting features made me wonder, if we should change the default label for pasting layers/geojson with no label (to something similar: "Pasted layer"), currently it defaults to a localized string of: "Layer"
https://github.com/Maps4HTML/Web-Map-Custom-Element/blob/main/src/mapml/utils/Util.js#L447
https://github.com/Maps4HTML/Web-Map-Custom-Element/blob/main/src/mapml/utils/Util.js#L569

Makes sense. I may update this and related PR to do that, tbd.