Skip to content

Conversation

@stdavis
Copy link
Member

@stdavis stdavis commented Dec 5, 2025

  • make graphic easier to see by placing it on top of labels
  • fix bug causing graphic from being hidden when double-click zooming
  • disable clear button when not applicable
  • clear file input state on clear

Copilot AI review requested due to automatic review settings December 5, 2025 23:59
@stdavis stdavis enabled auto-merge (rebase) December 6, 2025 00:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request implements several UI/UX improvements for the advanced search functionality, focusing on map graphics visibility, button state management, and file input handling.

Key Changes:

  • Modified basemap layer configuration to move overlay layers from referenceLayers to baseLayers array, improving graphic visibility
  • Added disabled state management for the clear button to improve UX by disabling it when no area of interest exists
  • Updated file input integration to use the new FileInput API from @ugrc/utah-design-system v1.34.0 with controlled state

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

File Description
src/hooks/useShapefileUpload.ts Updated hook to expose file state and setter, changed signature from FileList to File[] array
src/AppSearch.tsx Added clear button disabled state logic, updated basemap layer structure, integrated new FileInput API with controlled state, enhanced sketch tool event handling
package.json Updated @ugrc/utah-design-system from v1.33.1 to v1.34.0
package-lock.json Updated lock file to reflect design system upgrade and peer dependency changes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <[email protected]>
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