[openstreetmap/openstreetmap-website] Lessen code repetition (PR #5633)

Marwin Hochfelsner notifications at github.com
Sat Feb 8 18:49:50 UTC 2025


Simplifying if-else structures and condensing repeated expressions.
At least the things I found with my regex search.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5633

-- Commit Summary --

  * Reduce condition complexity
  * Lessen code repetition
  * Remove IE<10 workaround

-- File Changes --

    M app/assets/javascripts/application.js (8)
    M app/assets/javascripts/index.js (8)
    M app/assets/javascripts/index/changeset.js (10)
    M app/assets/javascripts/index/directions.js (39)
    M app/assets/javascripts/leaflet.map.js (19)
    M app/assets/javascripts/leaflet.note.js (10)
    M app/assets/javascripts/leaflet.query.js (14)
    M app/assets/javascripts/router.js (7)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5633.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5633.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/5633 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250208/998432a1/attachment.htm>


More information about the rails-dev mailing list