@hlfan commented on this pull request.


> +  .overlay-sidebar.overlay-right-sidebar {
+    #sidebar {
+      position: absolute;
+      width: 350px;
+      height: auto;
+      overflow: hidden;
+    }
+
+    #map {
+      height: 100%;
+    }
+  }

Why do we even re-style a sidebar of zero height depending on the `map-ui`?

```suggestion
  .overlay-sidebar.overlay-right-sidebar #map {
    height: 100%;
  }
```

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/7197#pullrequestreview-4606064334
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/7197/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to