[openstreetmap/openstreetmap-website] Add Recent Diaries Section to User Profile (Issue #6009)

2025-05-12 Thread Emin Kocan via rails-dev
kcne created an issue (openstreetmap/openstreetmap-website#6009) ### Problem Currently, the OpenStreetMap (OSM) user profile page only shows the user information and contribution heatmap. While it's possible to navigate to a user's diary entries by visiting the /user/diary page, this content re

Re: [openstreetmap/openstreetmap-website] Add locale selector (PR #5201)

2025-05-12 Thread Anton Khorev via rails-dev
@AntonKhorev commented on this pull request. > @@ -0,0 +1,15 @@ + + + + + <% unless disabled %> + + <% Locale.available + .select { |locale| I18n.exists? "shared.language_selector.#{locale}" } + .sort_by { |locale| t(".#{locale}") } +

Re: [openstreetmap/openstreetmap-website] Prerender heatmap in partial (PR #5998)

2025-05-12 Thread Marwin Hochfelsner via rails-dev
hlfan left a comment (openstreetmap/openstreetmap-website#5998) It's always 1px from bootstrap, although I certainly don't like to send 3.5 kb of `rounded-1` I've tried to use bootstrap for most of the styling. The seeming variation comes from the grid scaling to the available width (see how the

Re: [openstreetmap/openstreetmap-website] Prerender heatmap in partial (PR #5998)

2025-05-12 Thread Emin Kocan via rails-dev
kcne left a comment (openstreetmap/openstreetmap-website#5998) Even though I spent quite a bit of time trying to make our custom configuration work alongside cal-heatmap, I have to say - this approach makes a lot more sense. Modifying the library ended up requiring more code than writing the co

Re: [openstreetmap/openstreetmap-website] Refactor Heatmap: Manual Labels & Yearly Template (PR #5906)

2025-05-12 Thread Emin Kocan via rails-dev
kcne left a comment (openstreetmap/openstreetmap-website#5906) Thanks everyone for the valuable feedback. I completely understand the concerns around code complexity. My initial intent was to address the visual inconsistency of showing "empty" days beyond today’s date and to ensure proper local

Re: [openstreetmap/openstreetmap-website] Switch to inline colourable markers (PR #5854)

2025-05-12 Thread Marwin Hochfelsner via rails-dev
@hlfan pushed 4 commits. da77e371a1282dad1a72805c62a0d75f8433d540 Use common note marker definitions 7493f3e5d9283f377207130b11996f61bc7e2023 Generalize leaflet-marker-icon selector cc63535abcd9037a88bf432ac9d0bd0065ff4a14 Update and inline markers 0b153c5ba3e2e7779ee86e4247cafea407bd28ee Cle

Re: [openstreetmap/openstreetmap-website] Prerender heatmap in partial (PR #5998)

2025-05-12 Thread Marwin Hochfelsner via rails-dev
hlfan left a comment (openstreetmap/openstreetmap-website#5998) default (ISO 8601): ![localhost_3000_user_hltest](https://github.com/user-attachments/assets/9ff7eb00-cf4a-4f1e-84c3-35cd9c05fb82) "ar-AF" locale: ![localhost_3000_user_hltest (1)](https://github.com/user-attachments/assets/4760ee77

Re: [openstreetmap/openstreetmap-website] Prerender heatmap in partial (PR #5998)

2025-05-12 Thread Marwin Hochfelsner via rails-dev
@hlfan pushed 1 commit. 699ff148af9e759365c2a8e6904a3bae59283f27 Prerender heatmap in partial -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5998/files/064cf0e6203496f83fc1494259aaafa497dfb608..699ff148af9e759365c2a8e6904a3bae59283f27 You are receiving this be

Re: [openstreetmap/openstreetmap-website] Update to iD v2.34.0 (PR #6008)

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

Re: [openstreetmap/openstreetmap-website] Combine dashboard user popup custom css (PR #6004)

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

Re: [openstreetmap/openstreetmap-website] Use I18n.exists? to filter available locales (PR #6007)

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

Re: [openstreetmap/openstreetmap-website] Use I18n.exists? to filter available locales (PR #6007)

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

Re: [openstreetmap/openstreetmap-website] Combine dashboard user popup custom css (PR #6004)

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

Re: [openstreetmap/openstreetmap-website] Adds note versions and variable note tags (PR #5904)

2025-05-12 Thread Nenad Vujicic via rails-dev
@nenad-vujicic pushed 31 commits. b783f7b0f8061adef115a2aa5b34f3904b5ff13a Adds note versioning and adds note_versions table 32fb169cd8ecc360def808af68ce8d53ea5b6a09 Adds from_* and save_*! methods to notes d4c4557bb6f9dff7b213cefba004423ceb8e7a0a Adds using from_* and save_*! methods 6889212

Re: [openstreetmap/openstreetmap-website] Use I18n.exists? to filter available locales (PR #6007)

2025-05-12 Thread Nenad Vujicic via rails-dev
@nenad-vujicic approved this pull request. Works great on my side, thanks! -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6007#pullrequestreview-2833787655 You are receiving this because you are subscribed to this thread. Mess

Re: [openstreetmap/openstreetmap-website] Update to iD v2.34.0 (PR #6008)

2025-05-12 Thread Martin Raifer via rails-dev
@tyrasd pushed 1 commit. 82c33d461469fd1fe129edba135414b9e11edffc Update to iD v2.34.0 -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6008/files/5e3d6ef224c76e17b736555903ec23c906da6c24..82c33d461469fd1fe129edba135414b9e11edffc You are receiving this because y

Re: [openstreetmap/openstreetmap-website] Update to iD v2.34.0 (PR #6008)

2025-05-12 Thread github-actions[bot] via rails-dev
github-actions[bot] left a comment (openstreetmap/openstreetmap-website#6008) 1 Warning :warning: Number of updated lines of code is too large to be in one PR. Perhaps it should be separated into two or more? Gener

[openstreetmap/openstreetmap-website] Update to iD v2.34.0 (PR #6008)

2025-05-12 Thread Martin Raifer via rails-dev
Upstream changelog: https://github.com/openstreetmap/iD/releases/tag/v2.34.0 You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6008 -- Commit Summary -- * Update to iD v2.34.0 -- File Changes -- A package-lock.js

[openstreetmap/openstreetmap-website] Use I18n.exists? to filter available locales (PR #6007)

2025-05-12 Thread Anton Khorev via rails-dev
The method that checks if a locale exists currently does it by accessing the translation and checking if exception is (not) thrown. It could use `I18n.exists?` instead. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6

Re: [openstreetmap/openstreetmap-website] Add locale selector (PR #5201)

2025-05-12 Thread Anton Khorev via rails-dev
@AntonKhorev commented on this pull request. > @@ -0,0 +1,15 @@ + + + + + <% unless disabled %> + + <% Locale.available + .select { |locale| I18n.exists? "shared.language_selector.#{locale}" } This filters out some languages because of case sensitivity: - be-tar

Re: [openstreetmap/openstreetmap-website] Landing page to promote OSM (Issue #6006)

2025-05-12 Thread Tom Hughes via rails-dev
tomhughes left a comment (openstreetmap/openstreetmap-website#6006) There are already more such pages than we know what to do with, to name some examples: * https://www.openstreetmap.org/fixthemap * https://www.openstreetmap.org/welcome * https://www.openstreetmap.org/copyright (recently enhance

[openstreetmap/openstreetmap-website] Landing page to promote OSM (Issue #6006)

2025-05-12 Thread map-per via rails-dev
map-per created an issue (openstreetmap/openstreetmap-website#6006) Working on the [website](https://www.comaps.app/) for [CoMaps](https://codeberg.org/comaps), I once again realized that OSM has no good place to send people who don't know OSM to. (So we are linking to our own FAQ instead of di

Re: [openstreetmap/openstreetmap-website] API versions, capabilities and permissions resources (PR #5979)

2025-05-12 Thread Minh Nguyễn via rails-dev
1ec5 left a comment (openstreetmap/openstreetmap-website#5979) Oh! My bad. I [reverted the change](https://wiki.openstreetmap.org/wiki/Special:Diff/2852676). -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5979#issuecomment-28717

Re: [openstreetmap/openstreetmap-website] API versions, capabilities and permissions resources (PR #5979)

2025-05-12 Thread Andy Allan via rails-dev
gravitystorm left a comment (openstreetmap/openstreetmap-website#5979) > I [updated the > documentation](https://wiki.openstreetmap.org/wiki/Special:Diff/2852642) to > reflect the removal. It hasn't been removed - see https://www.openstreetmap.org/api/capabilities . This PR was just some refac

Re: [openstreetmap/openstreetmap-website] Add links to external map legends (PR #6005)

2025-05-12 Thread Anton Khorev via rails-dev
@AntonKhorev pushed 1 commit. 44f6f286ca41bf833a2829abf9c01fda6f4ad721 Add links to external map legends -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6005/files/1df2e6bc7c516bd85ab5d45979b409238962c00c..44f6f286ca41bf833a2829abf9c01fda6f4ad721 You are receiv

[openstreetmap/openstreetmap-website] Add links to external map legends (PR #6005)

2025-05-12 Thread Anton Khorev via rails-dev
Add links to map legends on external sites: ![image](https://github.com/user-attachments/assets/a7536bb1-c196-414f-87c4-61345964754a) The key on osm.org may not be complete and maybe it doesn't even make sense to complete it, in case there's a lot of icons. This PR adds links to pages like https

Re: [openstreetmap/openstreetmap-website] API versions, capabilities and permissions resources (PR #5979)

2025-05-12 Thread Minh Nguyễn via rails-dev
1ec5 left a comment (openstreetmap/openstreetmap-website#5979) I [updated the documentation](https://wiki.openstreetmap.org/wiki/Special:Diff/2852642) to reflect the removal. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5979#

[openstreetmap/openstreetmap-website] Combine dashboard user popup custom css (PR #6004)

2025-05-12 Thread Anton Khorev via rails-dev
Part of the css is in `leaflet-all.scss` where it partially overrides another part of the css from `common.scss`. This PR puts everything into `common.scss`. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/6004 -- Comm