fix: remove nested scroll in popup by limiting scroll to device list only

Copilot · Jackass4life · Copilot · commit 1e74c2e48a84 · 2026-03-24T07:34:33.000Z
Remove max-height and overflow-y:auto from .leaflet-popup-content to eliminate the outer scrollbar. Add max-height and overflow-y:auto to .coloc-popup to preserve co-located popup scroll + sticky header behavior. Fixes #26 Co-authored-by: Jackass4life <94110786+Jackass4life@users.noreply.github.com> Agent-Logs-Url: https://github.com/Jackass4life/Nautobot-maps/sessions/27b334cc-ad36-405d-9d1c-8530169cdf30
diff --git a/static/css/map.css b/static/css/map.css
@@ -372,8 +372,7 @@ html, body {
 }
 
 .leaflet-popup-content {
-  max-height: min(500px, 70vh);
-  overflow-y: auto;
+  overflow-y: visible;
 }
 
 .leaflet-popup-tip {
@@ -575,6 +574,11 @@ html, body {
   box-shadow: 0 0 4px rgba(0,0,0,.3);
 }
 
+.coloc-popup {
+  max-height: min(500px, 70vh);
+  overflow-y: auto;
+}
+
 .coloc-header {
   font-size: 0.78rem;
   font-weight: 600;

Original file line number	Diff line number	Diff line change
`@@ -372,8 +372,7 @@ html, body {`
`372`	`372`	`}`
`373`	`373`
`374`	`374`	`.leaflet-popup-content {`
`375`		`- max-height: min(500px, 70vh);`
`376`		`- overflow-y: auto;`
	`375`	`+ overflow-y: visible;`
`377`	`376`	`}`
`378`	`377`
`379`	`378`	`.leaflet-popup-tip {`
`@@ -575,6 +574,11 @@ html, body {`
`575`	`574`	`box-shadow: 0 0 4px rgba(0,0,0,.3);`
`576`	`575`	`}`
`577`	`576`
	`577`	`+.coloc-popup {`
	`578`	`+ max-height: min(500px, 70vh);`
	`579`	`+ overflow-y: auto;`
	`580`	`+}`
	`581`	`+`
`578`	`582`	`.coloc-header {`
`579`	`583`	`font-size: 0.78rem;`
`580`	`584`	`font-weight: 600;`