@CommanderStorm approved this pull request.
Since the refactoring moved the icon-handling to
`OSM.MapLibre.CombinedControlGroup`, I think this is a good fix.
Having the geolocator here is nice to have anyhow likely, maybe.
I have never written a diary, so might be wrong on that one.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6749#pullrequestreview-3717937097
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6749/review/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
