-
Notifications
You must be signed in to change notification settings - Fork 0
Clean up, dep bumps, fixes, place search, and bulk download snippets #294
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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 PR introduces bulk download functionality for raster tiles with command-line tool support (wget, curl, aria2c), adds a place search widget to the map, fixes UI overlap issues, and includes dependency updates. It also removes a previously required react-aria patch that is no longer needed.
Key Changes:
- New bulk download feature allowing users to copy download commands for multiple tiles at once
- Place search component integrated into the map for address and location lookup
- UI improvements preventing overlap between preview controls and tile count displays
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
vite.config.ts |
Updates import to use vitest/config instead of vite (recommended pattern for Vite+Vitest projects) |
tests/interaction/bulk-download.spec.ts |
Comprehensive E2E tests for new bulk download feature covering tool selection, clipboard operations, and UI feedback |
tests/edge-cases/error-handling.spec.ts |
Simplifies test by removing flaky coordinate-based clicking and using direct map interaction |
src/components/utils.ts |
Adds generateCommands function to create download snippets for wget, curl, and aria2c tools |
src/components/utils.test.ts |
Unit tests for generateCommands covering all three tools, URL construction, and edge cases |
src/components/TilesControls.tsx |
Removes absolute positioning and updates text styling for better integration with new container |
src/components/PreviewControls.tsx |
Removes absolute positioning to allow parent container control |
src/components/MapContainer.tsx |
Adds search widget with Utah geocoder, wraps controls in shared container to prevent overlap, updates widget positioning |
src/components/Download.tsx |
Integrates bulk download UI with RadioGroup for tool selection, copy button with feedback, and analytics |
patches/@react-aria__disclosure.patch |
Removes patch file (no longer needed with updated react-aria-components) |
package.json |
Bumps multiple dependencies including React 19.2.3, Vite 7.3.0, Playwright 1.57.0, and removes patch configuration |
CHANGELOG.md |
Documents new features, bug fixes, and dependency updates for v3.0.2-rc.1 |
.vscode/settings.json |
Adds "Manti" to spellcheck dictionary (used in search placeholder examples) |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Waiting for acceptance testing from @rkelson before this get's merged.