- Replaces `new URLSearchParams(location.search)` with `OSM.params()` when it 
yields more concise code.
  - All remaining URLSearchParams that aren't initialized empty or with an 
object use .toString().
- Adds `location.hash` as a default parameter for `OSM.parseHash`.
  - Moves the fallback for `OSM.params` to a default parameter too, which 
removes the check needed in #3571.

You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Shorten OSM.params and URLSearchParams

-- File Changes --

    M app/assets/javascripts/edit/id.js.erb (5)
    M app/assets/javascripts/heatmap.js (2)
    M app/assets/javascripts/index.js (8)
    M app/assets/javascripts/index/changeset.js (2)
    M app/assets/javascripts/index/directions.js (10)
    M app/assets/javascripts/index/history.js (6)
    M app/assets/javascripts/index/new_note.js (6)
    M app/assets/javascripts/index/note.js (2)
    M app/assets/javascripts/index/query.js (6)
    M app/assets/javascripts/index/search.js (10)
    M app/assets/javascripts/osm.js.erb (17)

-- Patch Links --

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

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

Message ID: <openstreetmap/openstreetmap-website/pull/5...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to