This is 
https://github.com/openstreetmap/openstreetmap-website/pull/5866#issuecomment-2764695781
 for borders. You want horizontal/vertical line boundaries to coincide with 
pixel boundaries. If the line is drawn at integer coordinates, it means that 
its width has to be an even number. The line shape boundaries are width/2 to 
one side and width/2 to another and you want the original coords +- width/2 to 
still be integers.

Before, notice the blurry changeset bbox:
![image](https://github.com/user-attachments/assets/b6dddfdd-898a-4031-94bd-b225d90897db)

This will be noticeable on 1x pixel density devices, and technically on 3x or 
other odd-numbered density, although physical pixels probably get too small to 
notice.

After:
![image](https://github.com/user-attachments/assets/90f2a812-eb54-45fb-ba22-a91c7a97d900)

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

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

-- Commit Summary --

  * Change highlighted bbox border width to 4px on history pages

-- File Changes --

    M app/assets/javascripts/index/history-changesets-layer.js (2)

-- Patch Links --

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

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5936
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