[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