trails-cool/trails
1
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix no-go button click creating a waypoint

Browse source 
The button was rendered as a React component, not a Leaflet control,
so clicks propagated to the map. Use L.DomEvent.disableClickPropagation
on the container — same approach Leaflet's built-in controls use.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Ullrich Schäfer 2026-04-03 10:37:43 +01:00
parent 596d96d8cb
commit 16568ffa45
No known key found for this signature in database
GPG key ID: A32FF691A0F752D9
1 changed files with 8 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

9
apps/planner/app/components/PlannerMap.tsx
View file

@ -170,8 +170,15 @@ function CursorTracker({ awareness }: { awareness: YjsState["awareness"] }) {
} }
function NoGoAreaButton({ active, onClick }: { active: boolean; onClick: () => void }) { function NoGoAreaButton({ active, onClick }: { active: boolean; onClick: () => void }) {
const ref = useRef<HTMLDivElement>(null);
// Prevent clicks from reaching the Leaflet map (same as built-in controls)
useEffect(() => {
if (ref.current) L.DomEvent.disableClickPropagation(ref.current);
}, []);
return ( return (
<div className="leaflet-top leaflet-left" style={{ marginTop: 80 }}> <div ref={ref} className="leaflet-top leaflet-left" style={{ marginTop: 80 }}>
<div className="leaflet-control leaflet-bar"> <div className="leaflet-control leaflet-bar">
<a <a
href="#" href="#"