Re: [openstreetmap/openstreetmap-website] Inline SVGs with helper module (PR #5938)

2025-06-18 Thread Marwin Hochfelsner via rails-dev
@hlfan pushed 2 commits. 1fdd276fa9167250171014cf769040146d0872d0 Move social link icons to partials 0a0c179849a5e60081d68f19281452df5d14ab22 Move more icons to partials -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5938/files/3423c3e26235eccf327c089037ae74f

Re: [openstreetmap/openstreetmap-website] Newer/Older Version buttons in element history (PR #6117)

2025-06-18 Thread Anton Khorev via rails-dev
@AntonKhorev pushed 4 commits. 09ee78696b87840775faf933660928d34f439587 Allow to specify cursor column in get_page_items 3261cbefbef108d46fee92646ece9a7822ddb2e5 Add older/newer element versions buttons a719e9f87795f8b91be74c85efa3784816e48f58 Add before/after params to version history links

[openstreetmap/openstreetmap-website] Newer/Older Version buttons in element history (PR #6117)

2025-06-18 Thread Anton Khorev via rails-dev
![image](https://github.com/user-attachments/assets/24a039f9-e2f3-41d9-8e26-5b7c25ffbc3c) You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6117 -- Commit Summary -- * Allow to specify cursor column in get_page_items

Re: [openstreetmap/openstreetmap-website] Add OpenMapTiles vector map (PR #4042)

2025-06-18 Thread Paul Norman via rails-dev
pnorman left a comment (openstreetmap/openstreetmap-website#4042) My PR won't touch OpenMapTiles stuff. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/4042#issuecomment-2985592436 You are receiving this because you are subscribed

Re: [openstreetmap/openstreetmap-website] Use a bootstrap dropdown for language selection (PR #6103)

2025-06-18 Thread Anton Khorev via rails-dev
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6103) Those single-column dropdowns are probably not the best idea. I made a version with [Bootstrap Modal](https://getbootstrap.com/docs/5.3/components/modal/) and up to four columns: ![image](https://github.com/user-attachments/a

Re: [openstreetmap/openstreetmap-website] Make flash alerts dismissable (PR #6115)

2025-06-18 Thread Anton Khorev via rails-dev
@AntonKhorev pushed 1 commit. 1f9e7b36de28e693d4902f2c7498f3f25a44aac2 Make flash alerts dismissable -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6115/files/fc928f8589de6e975435bdecd95294a25a5d11e6..1f9e7b36de28e693d4902f2c7498f3f25a44aac2 You are receiving

[openstreetmap/openstreetmap-website] Move user list javascript to its own file (PR #6116)

2025-06-18 Thread Anton Khorev via rails-dev
Continues #5972 by moving things out of `user.js`. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6116 -- Commit Summary -- * Move user list javascript to its own file -- File Changes -- M app/assets/javascrip

Re: [openstreetmap/openstreetmap-website] Make flash alerts dismissable (PR #6115)

2025-06-18 Thread Marwin Hochfelsner via rails-dev
@hlfan commented on this pull request. > @@ -1,13 +1,17 @@ <% [[:error, :danger], [:warning, :warning], [:notice, :success]].each do |flash_type, bootstrap_type| %> <% if flash[flash_type] %> -<%= tag.div :class => "alert alert-#{bootstrap_type} row mx-0 mb-0 p-3 rounded-0 align-items-

[openstreetmap/openstreetmap-website] Make flash alerts dismissable (PR #6115)

2025-06-18 Thread Anton Khorev via rails-dev
![image](https://github.com/user-attachments/assets/4ab8971d-fd08-4649-bd59-7b47f79e145f) You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6115 -- Commit Summary -- * Make flash alerts dismissable -- File Changes --

Re: [openstreetmap/openstreetmap-website] Add OpenMapTiles vector map (PR #4042)

2025-06-18 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#4042) I think @pnorman has also been working on it. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/4042#issuecomment-2983176180 You are receiving this because you are

Re: [openstreetmap/openstreetmap-website] Add OpenMapTiles vector map (PR #4042)

2025-06-18 Thread petr-hajek via rails-dev
petr-hajek left a comment (openstreetmap/openstreetmap-website#4042) > Are the maintainers planning to make the final changes here, or should > someone from the community do it? I am planning to it in the next 2 weeks -- Reply to this email directly or view it on GitHub: https://github.com/ope