[openstreetmap/openstreetmap-website] Use MapLibre class extensions to DRY code (PR #6726)

Marwin Hochfelsner notifications at github.com
Mon Jan 19 21:08:36 UTC 2026


hlfan left a comment (openstreetmap/openstreetmap-website#6726)

> I think it's probably a bit silly moving `CombinedControlGroup` to `maplibre/dom_util` in one commit and then moving it again to `maplibre/controls` in the next one - why not just move it directly to the final location in the first commit?

I wanted to get reviews in early; I also felt it was silly when committing this but figured I'd clean it up together if other restructures were suggested.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6726/c3770123418 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260119/3a1fa3a0/attachment.htm>


More information about the rails-dev mailing list