Re: [openstreetmap/openstreetmap-website] Social media icons are not visible in the dark theme of the site (Issue #6043)

2025-05-24 Thread Tom Hughes via rails-dev
Closed #6043 as completed via #6049. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/6043#event-17800693334 You are receiving this because you are subscribed to this thread. Message ID: _

[openstreetmap/openstreetmap-website] Data layer: display text for http 509 rate limiting errors (PR #6051)

2025-05-24 Thread mmd via rails-dev
This pull request adds human readable HTTP 509 rate limiting error messages. Before: ![b2](https://github.com/user-attachments/assets/a068668c-1218-463f-aec6-3635a10347b9) After: ![b3](https://github.com/user-attachments/assets/6abee1b7-24c7-4649-8280-480e3c7a0f07) You can view, comment on, or m

Re: [openstreetmap/openstreetmap-website] Setting for adding social media links conflicts with EasyList – Social Widgets (Issue #6041)

2025-05-24 Thread Mateusz Konieczny via rails-dev
matkoniecz left a comment (openstreetmap/openstreetmap-website#6041) If that filter in third-party extension is supposed to block such stuff then both osm.org and blocker works as expected If that filter in third-party extension is not supposed to block such stuff then adblocker has bug and you

Re: [openstreetmap/openstreetmap-website] Rename "social share buttons"/"social icons" to "share buttons"/"share button icons" (PR #6050)

2025-05-24 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#6050) Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6050#issuecomment-2906903777 You are receiving this because you are subscribed to this t

Re: [openstreetmap/openstreetmap-website] Remove legacy OpenID login (PR #6038)

2025-05-24 Thread mmd via rails-dev
@mmd-osm pushed 1 commit. a52b33ba96918d3523c9c6e91c1cf218b50e9e25 Remove legacy OpenID login -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6038/files/d9c9ea5f1cfa553da2bbc8586a1e83ae60271183..a52b33ba96918d3523c9c6e91c1cf218b50e9e25 You are receiving this be

Re: [openstreetmap/openstreetmap-website] Rename "social share buttons"/"social icons" to "share buttons"/"share button icons" (PR #6050)

2025-05-24 Thread Tom Hughes via rails-dev
Merged #6050 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6050#event-17800693305 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Invert social link icons in dark mode (PR #6049)

2025-05-24 Thread Tom Hughes via rails-dev
Merged #6049 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6049#event-17800693306 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Invert social link icons in dark mode (PR #6049)

2025-05-24 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#6049) Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6049#issuecomment-2906902694 You are receiving this because you are subscribed to this t

Re: [openstreetmap/openstreetmap-website] Search forms redesign (#3123)

2025-05-24 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#3123) For touch devices `title` is not the best choice. `optgroup` looks like this if you tap the select: ![Image](https://github.com/user-attachments/assets/d55a2142-2a74-4a4b-a30b-d0743828a402) -- Reply to this email directly or

[openstreetmap/openstreetmap-website] Rename "social share buttons"/"social icons" to "share buttons"/"share button icons" (PR #6050)

2025-05-24 Thread Anton Khorev via rails-dev
You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6050 -- Commit Summary -- * Rename "social share buttons" to "share buttons" * Remove social-share-buttons css class * Rename share button icons directory -- File

Re: [openstreetmap/openstreetmap-website] Search forms redesign (#3123)

2025-05-24 Thread Marwin Hochfelsner via rails-dev
hlfan left a comment (openstreetmap/openstreetmap-website#3123) > > > * there is no clue of what the dropdown is for, what OSRM, GraphHopper, > > > and Valhallah are > > > > > > Do you think an `optgroup` with a `label` and a `title` on the `select` > > would be enough hints? > > `optgroup` i

[openstreetmap/openstreetmap-website] Invert social link icons in dark mode (PR #6049)

2025-05-24 Thread Anton Khorev via rails-dev
Fixes #6043. ![image](https://github.com/user-attachments/assets/d4b81a35-1061-423d-80f6-8822ce60ce37) You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6049 -- Commit Summary -- * Invert social link icons in dark mo

Re: [openstreetmap/openstreetmap-website] Drop skype from social links (PR #6039)

2025-05-24 Thread Anton Khorev via rails-dev
Merged #6039 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6039#event-17800336943 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Drop skype from social links (PR #6039)

2025-05-24 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6039) Merged, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6039#issuecomment-2906844141 You are receiving this because you are subscribed to this thread.

Re: [openstreetmap/openstreetmap-website] Set length limits on user provided content (PR #6047)

2025-05-24 Thread Anton Khorev via rails-dev
Merged #6047 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6047#event-17800217894 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Add routing provider tips (PR #5956)

2025-05-24 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#5956) Let's suppose I don't know anything about osm. I open the site, I see the button with two arrows, there's a tooltip on it that says "Find directions...". I click it and I see on the top a car, a bicycle and a person walking, t

Re: [openstreetmap/openstreetmap-website] Set length limits on user provided content (PR #6047)

2025-05-24 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6047) Merged, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6047#issuecomment-2906824686 You are receiving this because you are subscribed to this thread.

Re: [openstreetmap/openstreetmap-website] Move heatmap to Turbo frame (PR #6046)

2025-05-24 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6046) > The cells resize to fit into the container without scrolling You get both scrolling and small cells on a phone. > the current min width of 10 px can easily be increased To what? Increase it to 14px and no *responsive* resiz

Re: [openstreetmap/openstreetmap-website] Show heatmap only to logged in users (PR #5829)

2025-05-24 Thread Anton Khorev via rails-dev
@AntonKhorev pushed 1 commit. 3109ca690a93d689a900db21a59575bbdcc238a1 Show heatmap only to logged in users -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5829/files/251599ba6c2592504afd8ef747af8e8f5d110d6b..3109ca690a93d689a900db21a59575bbdcc238a1 You are rec

Re: [openstreetmap/openstreetmap-website] Move heatmap to Turbo frame (PR #6046)

2025-05-24 Thread Marwin Hochfelsner via rails-dev
hlfan left a comment (openstreetmap/openstreetmap-website#6046) The cells resize to fit into the container without scrolling, the current min width of 10 px can easily be increased. There's also the option to assign the rows and cols as variables and switch the axies for small widths, then the c

Re: [openstreetmap/openstreetmap-website] Move heatmap to Turbo frame (PR #6046)

2025-05-24 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6046) > > if it would be possible to reserve a fixed space up front for the frame to > > fill > > The old SVG version had a fixed height, but now with it being responsive it's > more complicated, but with min-height, max-height and