[openstreetmap/openstreetmap-website] Use close button instead of geolink in sidebar content (PR #3770)
Anton Khorev
notifications at github.com
Wed Oct 26 13:32:35 UTC 2022
This is https://github.com/openstreetmap/openstreetmap-website/pull/3682/commits/cc9bcce5f0531243c34ffab8d43bc2885ff106ef + https://github.com/openstreetmap/openstreetmap-website/pull/3682/commits/b9f05806dbda4c33fcad5c1d68cb59623c432989
When I was changing close buttons to Bootstrap's `class="btn-close"`, I kept the one in the left sidebar as a link. It was a "geolink" and I didn't want to remove it because "geolinks" are updated when map view changes and I didn't know exactly how it was happening. Later in https://github.com/openstreetmap/openstreetmap-website/pull/3682 I finally replaced it with a button. I think I did it so I could cache the entire sidebar without worrying about map view updates.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/3770
-- Commit Summary --
* Use close button instead of geolink in sidebar content
-- File Changes --
M app/assets/javascripts/index.js (4)
M app/views/application/_sidebar_header.html.erb (2)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/3770.patch
https://github.com/openstreetmap/openstreetmap-website/pull/3770.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/3770
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/3770 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20221026/8f4b05c0/attachment.htm>
More information about the rails-dev
mailing list