stamen / maperture

https://stamen.github.io/maperture/
MIT License
39 stars 12 forks source link

Add layer name to feature popup #157

Closed RossThorn closed 1 year ago

RossThorn commented 1 year ago

User story

Maperture is a commonly used tool for our cartography workflows; it is primarily used for creating stylesheets. Right now, the popup shows the source and the attribute for the specific feature of interest.

When working with layers in a stylesheet (especially in a build system that is segmented out) it would be helpful to know what layer a particular feature is in when clicking and seeing the popup. This is particularly useful when QAing a style, so if a bug or unexpected render is spotted, the user can easily find the corresponding layer (file) where the style rules are defined.

This improvement to the tool would speed up stylesheet development work and debugging.

Acceptance criteria

The popup when clicking on a feature in the basemap lists the source for the feature. It would be nice if the popup also showed the layer that feature is being styled in.

For example, the label for Maryland has the composite source of place_label but is in the state-label layer of the stylesheet. it would be helpful to see that

image

RossThorn commented 1 year ago

My proposed US and AC!

User story

The feature popup functionality in Maperture is great for getting on-the-fly information about the data in the tiles. Right now, the popup shows the source and the attribute for the specific feature of interest.

When working with layers in a stylesheet (especially in a build system that is segmented out) it would be helpful to know what layer a particular feature is in when clicking and seeing the popup. This is particularly useful when QAing a style, so if a bug or unexpected render is spotted, the user can easily find the corresponding layer (file) where the style rules are defined.

Acceptance criteria

EDIT: Copied this and @mulloverit's addendum to the original description for better visibility.

mulloverit commented 1 year ago

From grooming meeting, adding the business case: