fix: prevent bottom-left layer control from overlapping attribution

[Copilot]

On narrow screens, the multi-line Leaflet attribution can cover the bottom of the layer selector. The fix mirrors the attribution height into a CSS variable and offsets the bottom-left control stack accordingly.

• Runtime measurement
  • Observe attribution size changes with ResizeObserver.
  • Update --attribution-height on the map container.
• Control positioning
  • Apply the variable height to .leaflet-bottom.leaflet-left.

new ResizeObserver(() => {
    map.getContainer().style.setProperty(
        '--attribution-height',
        attribution.offsetHeight + 'px'
    );
}).observe(attribution);

[github-actions]

PR Preview Action v1.8.1
🚀 View preview at https://Sp3EdeR.github.io/huroutes/pr/pr-267/
Built to branch gh-pages at 2026-09-19 14:31 UTC. Preview will be ready when the GitHub Pages deployment is complete.

[Copilot]

Copilot review overview

🟢 Approval recommended

No unresolved issues were identified, and the review supports approval.

Review effort: Lite
Findings: None

What changed in this PR

Prevents Leaflet’s bottom-left controls from overlapping multi-line attribution on narrow screens.

Changes:

• Tracks attribution height with ResizeObserver.
• Offsets bottom-left controls using a CSS variable.

File	Description
res/​huroutes.js	Updates the attribution-height CSS variable dynamically.
res/​huroutes.css	Applies the variable to bottom-left Leaflet controls.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🟢 Approval recommended

The focused changes correctly adapt control positioning as attribution height changes.

Review effort: Balanced
Findings: None

[Sp3EdeR]

This selector looks too brittle to me. Please update to this to make sure it applies to all bottom-placed controls except the one that has attribution, but controls only:

.leaflet-control-container .leaflet-bottom:not(:has(.leaflet-control-attribution))

[nbali]

Wouldn't this break the UI as it would apply on the right as well?

[Sp3EdeR]

No, it has :not(:has(.leaflet-control-attribution)), so it doesn't apply this for any controls that have an attribution within