Conversation
giswqs
commented
Nov 23, 2025
|
🚀 Deployed on https://692292e8ccee2e3ecbaf6e47--opengeos.netlify.app |
There was a problem hiding this comment.
Pull request overview
This PR updates the layer control icon from a CSS-based design (using box-shadow for stacked lines) to an inline SVG icon representing stacked layers. The change improves visual clarity and robustness across different color schemes.
- Replaces CSS pseudo-element icon with an inline SVG "layers" glyph
- Updates CSS to support the new SVG icon and enforce dark icon color
- Removes the old CSS-based icon implementation
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| anymap/static/maplibre_widget.js | Adds inline SVG icon markup to the layer control button element |
| anymap/static/maplibre_widget.css | Updates styles to support SVG icon, removes old CSS-based icon, and enforces dark icon color |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| justify-content: center; | ||
| cursor: default !important; | ||
| outline: none; | ||
| color: #1f2a37 !important; /* force icon to be dark even if parent sets white */ |
There was a problem hiding this comment.
The hardcoded dark color with !important may cause accessibility issues in dark mode themes where a light icon would be more appropriate. Consider using a CSS variable or data attribute to support theme-aware icon colors instead of forcing a dark color.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>