Skip to content

feat: shareable browse URLs and plant detail dialog#202

Open
native-plants-zach wants to merge 2 commits into
CodeForPhilly:mainfrom
native-plants-zach:feat/plant-detail-dialog
Open

feat: shareable browse URLs and plant detail dialog#202
native-plants-zach wants to merge 2 commits into
CodeForPhilly:mainfrom
native-plants-zach:feat/plant-detail-dialog

Conversation

@native-plants-zach

Copy link
Copy Markdown
Collaborator

Summary

  • Add shareable browse URLs that serialize search, sort, filters, zip, and location into query params so filtered views can be bookmarked and shared
  • Add an accessible plant detail dialog with loading/error states, prev/next navigation within browse results, keyboard and touch support, and preserved browse URL state when opening plants

Test plan

  • Open / with filters applied and confirm the URL updates as you change search, sort, filters, and location
  • Copy the URL, open in a new tab, and confirm browse state restores correctly
  • Click a plant tile and confirm detail opens in a modal with position label
  • Use prev/next buttons, arrow keys, and swipe gestures to move between plants in the current result set
  • Close detail and confirm browse filters remain in the URL
  • Open a direct /plants/:name link with query params and confirm detail and browse state load correctly
  • Verify Escape closes the dialog and focus returns to the triggering element

native-plants-zach and others added 2 commits June 24, 2026 16:57
Serialize search, sort, filters, zip, and location into query params so browse views and plant detail pages can be bookmarked and shared.

Co-authored-by: Cursor <cursoragent@cursor.com>
Extract plant detail into a modal with loading and error states, prev/next browsing within results, keyboard and touch navigation, and shareable URLs that keep browse filters when opening plants.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant