-
Notifications
You must be signed in to change notification settings - Fork 0
feat(arm): add abstract-resource-map #191
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 adds a comprehensive map feature using Shadcn-inspired components built on top of Leaflet. It introduces a new /map route that displays resources on an interactive map with layer controls, allowing users to visualize objects across different map layers.
Key Changes:
- Integrated Leaflet and react-leaflet libraries for interactive mapping functionality
- Created a comprehensive map component system with drawing tools, layer management, and location controls
- Added new route
/mapwith permissions for users and admins
Reviewed Changes
Copilot reviewed 14 out of 15 changed files in this pull request and generated 17 comments.
Show a summary per file
| File | Description |
|---|---|
src/components/ui/map.tsx |
Core map component with full Leaflet integration including markers, layers, drawing tools, and controls |
src/components/ui/button-group.tsx |
New button group component for organizing map control buttons |
src/components/ui/button/variants.ts |
Added input, icon-sm, and icon-lg button variants for map controls |
src/components/ui/dropdown-menu.tsx |
Updated dropdown menu with new icons and styling fixes for layer selection |
src/features/abstract-resource-collection/components/abstract-resource-map.tsx |
Map wrapper component for displaying resource collections |
src/features/abstract-resource-collection/constants.ts |
Added PWR coordinates constant for map centering |
src/features/abstract-resource-collection/index.ts |
Exported new AbstractResourceMap component |
src/app/(private)/map/page.tsx |
Map page component implementation |
src/app/(private)/map/layout.tsx |
Layout wrapper for map route |
src/app/(private)/page.tsx |
Added map navigation button to dashboard |
src/features/authentication/data/route-permissions.ts |
Added /map route permissions |
src/app/globals.css |
Leaflet-specific CSS customizations and theme integration |
package.json |
Added leaflet, leaflet-draw, and react-leaflet dependencies |
components.json |
Added Shadcn map registry configuration |
Comments suppressed due to low confidence (1)
src/components/ui/dropdown-menu.tsx:12
- Unused imports CircleDotDashedIcon, CircleDotIcon, DotIcon.
import {
CheckIcon,
ChevronRightIcon,
CircleDotDashedIcon,
CircleDotIcon,
CircleIcon,
CircleSmallIcon,
DotIcon,
} from "lucide-react";
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
f7300bd to
e33baa0
Compare
2d0169c to
90df30c
Compare
c3ee3cc to
1ea2f72
Compare
1ea2f72 to
4c7de70
Compare
zmienimy panele zarządzania obiektami na mapę która wyświetla wszystkie zasoby w osobnych warstwach
https://shadcn-map.vercel.app
czeka na #96