[openstreetmap/openstreetmap-website] input_marker checked by default (PR #6031)

2025-05-19 Thread Martin Monperrus via rails-dev
fix #6025 You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6031 -- Commit Summary -- * input_marker checked by default -- File Changes -- M app/assets/javascripts/leaflet.share.js (1) -- Patch Links -- https:/

Re: [openstreetmap/openstreetmap-website] Don't draw compass needle for user's location (Issue #6030)

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

Re: [openstreetmap/openstreetmap-website] Don't draw compass needle for user's location (Issue #6030)

2025-05-19 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#6030) I have no idea if that is even possible, but the location control is provided by https://github.com/domoritz/leaflet-locatecontrol so you would need to raise it there. -- Reply to this email directly or view it on GitHub: http

[openstreetmap/openstreetmap-website] Don't draw compass needle for user's location (Issue #6030)

2025-05-19 Thread Boris Verkhovskiy via rails-dev
verhovsky created an issue (openstreetmap/openstreetmap-website#6030) When I click the show my location button on the homepage, it also has a compass needle that points north. I'm not looking north, I dont think this data is even available in the browser. It should just be a blue circle without

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

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

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

2025-05-19 Thread Anton Khorev via rails-dev
@AntonKhorev pushed 3 commits. 1ed00e36177de2da9dd36b20a75e6290ea748762 Test if max_id is present in heatmap data d4be383971f9c10f119637c5698f16352561cd93 Split heatmap rendering test c65a529009d77c0af14210ca9317032a04d1f948 Show heatmap only to logged in users -- View it on GitHub: https://

[openstreetmap/openstreetmap-website] Bump @types/leaflet from 1.9.17 to 1.9.18 (PR #6029)

2025-05-19 Thread dependabot[bot] via rails-dev
Bumps [@types/leaflet](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/leaflet) from 1.9.17 to 1.9.18.
Commits

Re: [openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread wolfy1339 via rails-dev
wolfy1339 left a comment (openstreetmap/openstreetmap-website#6028) This pretty much sounds exactly the same as https://github.com/openstreetmap/openstreetmap-website/issues/5983. I experienced it today as well -- Reply to this email directly or view it on GitHub: https://github.com/openstreetm

Re: [openstreetmap/openstreetmap-website] Company Name on Profile Page (PR #5987)

2025-05-19 Thread Amy Bordenave via rails-dev
uw-amy-bordenave left a comment (openstreetmap/openstreetmap-website#5987) > We're not going to have the wiki driving checkmarks, for many reasons, of > which it not proving anything at all is just the cherry on the cake. Makes sense - that's probably for the best. Regardless, I support this add

Re: [openstreetmap/openstreetmap-website] Company Name on Profile Page (PR #5987)

2025-05-19 Thread Minh Nguyễn via rails-dev
@1ec5 commented on this pull request. > @@ -153,6 +153,14 @@ <%= @user.home_location_name %> <% end %> +<% if @user.company&.strip.present? %> + +http://www.w3.org/2000/svg"; width="16" height="16" fill="curre

Re: [openstreetmap/openstreetmap-website] Company Name on Profile Page (PR #5987)

2025-05-19 Thread Minh Nguyễn via rails-dev
@1ec5 commented on this pull request. > @@ -153,6 +153,14 @@ <%= @user.home_location_name %> <% end %> +<% if @user.company&.strip.present? %> + +http://www.w3.org/2000/svg"; width="16" height="16" fill="curre

Re: [openstreetmap/openstreetmap-website] Company Name on Profile Page (PR #5987)

2025-05-19 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#5987) We're not going to have the wiki driving checkmarks, for many reasons, of which it not proving anything at all is just the cherry on the cake. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetm

Re: [openstreetmap/openstreetmap-website] Company Name on Profile Page (PR #5987)

2025-05-19 Thread Amy Bordenave via rails-dev
uw-amy-bordenave left a comment (openstreetmap/openstreetmap-website#5987) (Note that above mention of "@[mmd-osm](https://github.com/mmd-osm) mentioned this pull request: Intermittent CORS issues with editing API #6028" is just due to a link typo which they've fixed, so disregard that) --- I

Re: [openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread elevenchars via rails-dev
elevenchars left a comment (openstreetmap/openstreetmap-website#6028) I'm also using Firefox and regularly get treated poorly by Cloudflare so I would not be terribly surprised to hear it is related. There's a user in the OSM World discord reporting the same behavior on Chrome on Windows, though

Re: [openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread Lumikeiju via rails-dev
Lumikeiju left a comment (openstreetmap/openstreetmap-website#6028) Seconding, same exact issue. Also reported by multiple other people in the [OSM World Discord server](https://discord.com/channels/413070382636072960/413070502580453387/1374075517782724648) and [OSM US Slack server](https://osm

Re: [openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread mmd via rails-dev
mmd-osm left a comment (openstreetmap/openstreetmap-website#6028) Isn't that similar to #5987? -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/6028#issuecomment-2892302476 You are receiving this because you are subscribed to thi

Re: [openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread elevenchars via rails-dev
elevenchars left a comment (openstreetmap/openstreetmap-website#6028) Here's some other people reporting it on the community forums: https://community.openstreetmap.org/t/actual-problems-with-id-editor/130320/2 as well as a handful of users on Discord and Slack. I haven't seen this behavior befor

Re: [openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#6028) If it's intermittent then I'm afraid I have no idea - there's absolutely nothing at our end that would give inconsistent results and yours are the only reports we have ever had to my knowledge. -- Reply to this email directly

Re: [openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread Will via rails-dev
elevenchars left a comment (openstreetmap/openstreetmap-website#6028) It's happening for me again - seems intermittent -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/6028#issuecomment-2892195481 You are receiving this because y

[openstreetmap/openstreetmap-website] Intermittent CORS issues with editing API (Issue #6028)

2025-05-19 Thread Will via rails-dev
elevenchars created an issue (openstreetmap/openstreetmap-website#6028) ### URL https://openstreetmap.org/edit ### How to reproduce the issue? 1. Open iD (or Rapid) 2. Observe that no map data loads ### Screenshot(s) or anything else? ![Image](https://github.com/user-attachments/assets/7a3bca

Re: [openstreetmap/openstreetmap-website] Remove .display_name from user path helper calls (PR #6026)

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

Re: [openstreetmap/openstreetmap-website] Restore the original heatmap cell colors (PR #6027)

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

Re: [openstreetmap/openstreetmap-website] Restore the original heatmap cell colors (PR #6027)

2025-05-19 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#6027) I had kind of assumed that the new implementation and didn't really check that is was but once deployed it was fairly obviously not right, so thanks for the fix! -- Reply to this email directly or view it on GitHub: https://g

Re: [openstreetmap/openstreetmap-website] Remove .display_name from user path helper calls (PR #6026)

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

Re: [openstreetmap/openstreetmap-website] CORS issues in Firefox with API (Issue #5983)

2025-05-19 Thread wolfy1339 via rails-dev
wolfy1339 left a comment (openstreetmap/openstreetmap-website#5983) ``` HTTP/2 200 date: Mon, 19 May 2025 17:17:38 GMT content-type: text/plain access-control-allow-origin: https://www.openstreetmap.org access-control-allow-methods: POST,OPTIONS access-control-max-age: 86400 vary: Origin access-c

Re: [openstreetmap/openstreetmap-website] CORS issues in Firefox with API (Issue #5983)

2025-05-19 Thread wolfy1339 via rails-dev
wolfy1339 left a comment (openstreetmap/openstreetmap-website#5983) I just happened to experience this issue again today. Whatever is going on seems to be intermittent -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/5983#issuec

[openstreetmap/openstreetmap-website] Remove .display_name from user path helper calls (PR #6026)

2025-05-19 Thread Anton Khorev via rails-dev
Replaces ``` user_path(user.display_name) ``` and likes with ``` user_path(user) ``` They mostly were replaced already but heatmap and friends-follows brought some back. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/

[openstreetmap/openstreetmap-website] Restore the original heatmap cell colors (PR #6027)

2025-05-19 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/6027 -- Commit Summary -- * Use css properties of heatmap cells for everything except colors * Use spans instead of divs in all heatmap cells * Restore the origina

Re: [openstreetmap/openstreetmap-website] Implement JSON support for the changeset download API (PR #5973)

2025-05-19 Thread Kyℓe Hensel via rails-dev
k-yle left a comment (openstreetmap/openstreetmap-website#5973) If we decide to get rid of the `create`/`update`/`delete` blocks, then there are two options (broadly speaking): 1. allow attributes to be specified some some elements only. Requires a nested array: ```jsonc { "osmChange": [

Re: [openstreetmap/openstreetmap-website] link by gps coordinates does not show a pin (Issue #6025)

2025-05-19 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#6025) Right, but where did you get the idea that particular URL would do that? It never has so it sounds like there is some documentation our there which is wrong? If you just open the share panel on the right hand side there are all

Re: [openstreetmap/openstreetmap-website] link by gps coordinates does not show a pin (Issue #6025)

2025-05-19 Thread Martin Monperrus via rails-dev
monperrus left a comment (openstreetmap/openstreetmap-website#6025) OK, there is the option, then my issue is about the default. The default, shortest URL, could be with marker. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/6

Re: [openstreetmap/openstreetmap-website] link by gps coordinates does not show a pin (Issue #6025)

2025-05-19 Thread Martin Monperrus via rails-dev
monperrus left a comment (openstreetmap/openstreetmap-website#6025) Because this is the URL I get when I click on "share" :) -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/6025#issuecomment-2890242921 You are receiving this b

Re: [openstreetmap/openstreetmap-website] link by gps coordinates does not show a pin (Issue #6025)

2025-05-19 Thread Martin Monperrus via rails-dev
monperrus left a comment (openstreetmap/openstreetmap-website#6025) thanks @tomhughes for the fast answer. the use case is to send a URL to somebody to meet, the positioning fragment is meant to be this location and is expected to be shown on the map. -- Reply to this email directly or view i