Fix z-index of map layers settings popup. (#1070)

wilmaier · web-flow · commit f53aea190cda · 2024-10-21T08:32:24.000-05:00
* Fix z-index of map layers settings popup

* Add change log and TODO comment
diff --git a/change/@itwin-map-layers-41f0daa2-132a-4829-98d9-6650960a31ab.json b/change/@itwin-map-layers-41f0daa2-132a-4829-98d9-6650960a31ab.json
@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Fix z-index of map layers settings popup",
+  "packageName": "@itwin/map-layers",
+  "email": "wil.maier@bentley.com",
+  "dependentChangeType": "patch"
+}
diff --git a/packages/itwin/map-layers/src/ui/widget/MapLayerSettingsPopupButton.scss b/packages/itwin/map-layers/src/ui/widget/MapLayerSettingsPopupButton.scss
@@ -7,7 +7,8 @@
 $hovered-icon-color: $buic-foreground-primary;
 
 .maplayers-settings-popup {
-  z-index: 900; // Used to be higher, but we want this be less than modal dialog z-index
+  /* TODO: z-index workaround while map-layers-react is using the deprecated Popup component from core-react */
+  z-index: 6001; /* one more than floating widget */
 }
 
 .maplayers-settings-popup-panel {

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,8 @@`
`7`	`7`	`$hovered-icon-color: $buic-foreground-primary;`
`8`	`8`
`9`	`9`	`.maplayers-settings-popup {`
`10`		`- z-index: 900; // Used to be higher, but we want this be less than modal dialog z-index`
	`10`	`+ /* TODO: z-index workaround while map-layers-react is using the deprecated Popup component from core-react */`
	`11`	`+ z-index: 6001; /* one more than floating widget */`
`11`	`12`	`}`
`12`	`13`
`13`	`14`	`.maplayers-settings-popup-panel {`