Skip to content

Conversation

@kguzek
Copy link
Member

@kguzek kguzek commented Nov 16, 2025

zmienimy panele zarządzania obiektami na mapę która wyświetla wszystkie zasoby w osobnych warstwach

  • Ola/Szymek chcą żeby móc mieć też opcję wyświetlania w liście
  • Możnaby też dodać mapę jako alternatywny sposób inputu w samym formularzu

https://shadcn-map.vercel.app

czeka na #96

@kguzek kguzek self-assigned this Nov 16, 2025
Copilot AI review requested due to automatic review settings November 16, 2025 16:01
@kguzek kguzek added the enhancement New feature or request label Nov 16, 2025
@kguzek kguzek changed the title feat(arc): add shadcn map feat(arc): add abstract-resource-map Nov 16, 2025
Copy link

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 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 /map with 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.

@kguzek kguzek force-pushed the feat/resource-map branch 3 times, most recently from 2d0169c to 90df30c Compare December 8, 2025 22:46
@kguzek kguzek force-pushed the feat/resource-map branch 4 times, most recently from c3ee3cc to 1ea2f72 Compare December 29, 2025 20:55
@kguzek kguzek changed the title feat(arc): add abstract-resource-map feat(arm): add abstract-resource-map Dec 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size/XXL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants