Skip to content

Conversation

@clarasb
Copy link
Collaborator

@clarasb clarasb commented Aug 12, 2025

This PR:

  • adds a box to the map, that displays the current zoomLevel of the map and the current datasetLevel. The visibilty of the box can be changed in the settings. The initial visibility can be set in config.json (default: false).
  • is dependent on: WebAPI: Add resolutions and spatialUnits to mldataset  xcube#1177.
  • adds information on the number of levels in the selected dataset to the Info-Panel
  • add feature to feature references list in the documentation
grafik

Closes #287.

Copy link
Member

@forman forman left a comment

Choose a reason for hiding this comment

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

Cool feature, great job!

Please apply renamings:

  • Settings state: zoomEnabled --> showZoomBox
  • Branding: allowZoomBox --> showZoomBox
  • In settings panel UI: "Levels" --> "Number of resolution levels"

In a new directory module ZoomBox

  • Move ZoomBox.tsx
  • Move ZoomBox.stories.tsx
  • Add index.tsx to export ZoomBox component

@clarasb clarasb requested a review from forman September 23, 2025 15:31
@clarasb clarasb requested a review from forman October 14, 2025 09:47
@clarasb clarasb requested a review from AliceBalfanz October 27, 2025 09:54
Comment on lines +514 to +517
Displays an information box that showing the current
zoom level of the map and the dataset resolution level (particularly
useful for multi-resolution dataset) used for displaying the dataset
in the map.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Displays an information box that showing the current
zoom level of the map and the dataset resolution level (particularly
useful for multi-resolution dataset) used for displaying the dataset
in the map.
Displays an information box showing the current
map zoom level and the dataset resolution level used for displaying the
dataset on the map. The the dataset resolution level is particularly
useful for multi-resolution datasets.

<tr>
<td><b>Functionality</b></td>
<td>
The visibility of this feature can be controlled in the
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The visibility of this feature can be controlled in the
The visibility of this feature can be controlled through the

Copy link
Contributor

@AliceBalfanz AliceBalfanz left a comment

Choose a reason for hiding this comment

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

Works nicely! Minor comments on wording for the documentation.

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.

Add an indicator of the current zoom level

4 participants