[openstreetmap/openstreetmap-website] Use flex container instead of floats in header (PR #4296)
Andy Allan
notifications at github.com
Wed Nov 15 16:03:49 UTC 2023
@gravitystorm requested changes on this pull request.
Thanks @AntonKhorev , I definitely want to get rid of all of the floats.
However, in this PR the menus aren't working for me when I change the window size. For example, everything is fine at wide screen widths:

Then as the screen is narrowed there is a positioning error:

Then slightly narrower it looks OK again:

Then during the next transition there is also an error:

Then it looks OK again:

Other notes:
* The same errors are shown as the width increases (vs decreases)
* The same errors are shown when refreshing the page at the given error width
* UA: Firefox 119.0.1 (Ubuntu)
* If there's some way to avoid the absolute positioning (of the menu icon in small-nav mode) that would be great. Perhaps it can be positioned using flex too?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4296#pullrequestreview-1732380679
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4296/review/1732380679 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231115/3f00cd14/attachment.htm>
More information about the rails-dev
mailing list