Skip to content

Conversation

hrodmn
Copy link
Contributor

@hrodmn hrodmn commented Mar 7, 2025

The new browser-like capabilities in the STAC endpoints are capable of using more than just the limit query parameter! This just adds a free-text search to the /collections HTML page. We could add bbox and datetime but the layout could get pretty crowded so I just added text search for now.

image

@hrodmn hrodmn requested a review from oliverroick March 7, 2025 12:02
Copy link
Contributor

@oliverroick oliverroick left a comment

Choose a reason for hiding this comment

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

This is great, I was thinking of doing something along the same lines.

We could add bbox and datetime but the layout could get pretty crowded so I just added text search for now.

We could add an “advance search” for more detailed search options, ie. placing the form inputs for bbox and datetime in a collapsible that is hidden by default.


I’ve added two small changes:

  • The label placement in the flex row looked a bit off, so I re-arranged the elements a little to address that.
  • Being a little pedantic here: In the spirit of progressive enhancement, I registered submitSearch as a handler for the submit event, rather than using the onsubmit attribute.

@hrodmn
Copy link
Contributor Author

hrodmn commented Mar 11, 2025

@oliverroick thanks for your help clarifying the html for this feature - html is not my native language.

We could add bbox and datetime but the layout could get pretty crowded so I just added text search for now.

We could add an “advance search” for more detailed search options, ie. placing the form inputs for bbox and datetime in a collapsible that is hidden by default.

That would be cool! Should we add that in this PR or in a separate one?

@hrodmn hrodmn merged commit acdd687 into main Mar 11, 2025
2 checks passed
@hrodmn hrodmn deleted the feat/collection-text-search branch March 11, 2025 10:28
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.

2 participants