[openstreetmap/openstreetmap-website] [WIP] Load js controller modules only when needed (PR #7147)

Pablo Brasero notifications at github.com
Wed Jun 10 16:53:09 UTC 2026


pablobm left a comment (openstreetmap/openstreetmap-website#7147)

This seems like a good idea to me. I think it's reasonable to assume that most visitors will only use a couple of these modules in each visit.

Does `app/assets/javascripts/index_modules/history-changesets-layer.js` need to get the `export` line too?

> do you think keeping the separation of concerns when cut down so far is still something to worry about when it bloats the flattened diff size so much?

Sorry, I don't understand the question 😓 Can you rephrase, please?

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

Message ID: <openstreetmap/openstreetmap-website/pull/7147/c4672414810 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260610/2813c6a5/attachment.htm>


More information about the rails-dev mailing list