Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User-selectable options, e.g. number of previews on map #411

Closed
Tracked by #535
drnextgis opened this issue Mar 12, 2024 · 5 comments · May be fixed by #535
Closed
Tracked by #535

User-selectable options, e.g. number of previews on map #411

drnextgis opened this issue Mar 12, 2024 · 5 comments · May be fixed by #535
Milestone

Comments

@drnextgis
Copy link
Contributor

drnextgis commented Mar 12, 2024

image

I'm using STAC Browser version 3.1.0. After adjusting the "Items per page" default setting to 100 (If I decrease it to 49, it starts functioning, although it doesn't work with 50 and higher), I can observe an increased number of footprints on the map. However, inexplicably, the STAC browser fails to display previews for all items.

@drnextgis
Copy link
Contributor Author

I was able to reproduce it with Microsoft Planetary Computer STAC API:
image

image

@drnextgis
Copy link
Contributor Author

drnextgis commented Mar 13, 2024

Oh, I've discovered it! The reason is that maxPreviewsOnMap is set to a default value of 50. It would be convenient to have an option to adjust this value from the UI.Typically, STAC Browser users aren't acquainted with its internals, and the current behavior can be somewhat confusing.

@m-mohr
Copy link
Collaborator

m-mohr commented Mar 14, 2024

Customizing some of the options per user is a good idea. We already store the selected locale locally and could expand this to e.g. cardViewMode, maxPreviewsOnMap, geoTiffResolution, and itemsPerPage.

@m-mohr m-mohr changed the title Incomplete previews on the webmap User-selectable options, e.g. number of previews on map Mar 14, 2024
@m-mohr m-mohr added the UX label Mar 14, 2024
@m-mohr m-mohr added the mapping label Jan 7, 2025
@m-mohr
Copy link
Collaborator

m-mohr commented Jan 25, 2025

I guess maxPreviewOnMap should only be used when no limit is specified. If a specific pagination limit is specified, we should probably use that to keep things a bit more intuitive.

@m-mohr m-mohr added this to the 3.4.0 milestone Jan 25, 2025
@m-mohr m-mohr mentioned this issue Jan 25, 2025
25 tasks
@m-mohr
Copy link
Collaborator

m-mohr commented Jan 25, 2025

I decided to remove maxPreviewOnMap completely, it will now depend on what the server/users decide to deliver/retrieve. Part of PR #535

@m-mohr m-mohr closed this as completed Jan 25, 2025
@m-mohr m-mohr linked a pull request Jan 25, 2025 that will close this issue
25 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants