Re: [openstreetmap/openstreetmap-website] Don't interpolate type in issue title (PR #5382)

2024-12-08 Thread Tom Hughes via rails-dev
Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5382#issuecomment-2525582308 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [openstreetmap/openstreetmap-website] Let header username expand to screen width in small nav mode (PR #5383)

2024-12-08 Thread Tom Hughes via rails-dev
Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5383#issuecomment-2525586108 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [openstreetmap/openstreetmap-website] Remove logger.debug from relation update action (PR #5384)

2024-12-08 Thread Tom Hughes via rails-dev
Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5384#issuecomment-2525586297 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [openstreetmap/openstreetmap-website] Write browse element icons as (PR #5385)

2024-12-08 Thread Tom Hughes via rails-dev
@tomhughes requested changes on this pull request. So to summarise on the question of what happens when there are multiple icons the old logic is that priority is defined by the order of the rules in the CSS which is mostly alphabetical except that the generic building rule is with the way rul

Re: [openstreetmap/openstreetmap-website] Let header username expand to screen width in small nav mode (PR #5383)

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

Re: [openstreetmap/openstreetmap-website] Allow specifying extra layer options in layers.yml (PR #5386)

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

Re: [openstreetmap/openstreetmap-website] Allow specifying extra layer options in layers.yml (PR #5386)

2024-12-08 Thread Tom Hughes via rails-dev
Nice - making it easier for people reusing the software for other sites to change the map layers has been a long standing problem. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5386#issuecomment-2525668785 You are receiving this

Re: [openstreetmap/openstreetmap-website] Remove logger.debug from relation update action (PR #5384)

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

Re: [openstreetmap/openstreetmap-website] Don't interpolate type in issue title (PR #5382)

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

Re: [openstreetmap/openstreetmap-website] Use fetch() instead of iframe to make remote control requests (PR #5375)

2024-12-08 Thread Tom Hughes via rails-dev
Looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5375#issuecomment-2526151435 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [openstreetmap/openstreetmap-website] Use $.ajax instead of iframe to make remote control requests (PR #3760)

2024-12-08 Thread Tom Hughes via rails-dev
Closed #3760. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/3760#event-15573179182 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-dev mailing

Re: [openstreetmap/openstreetmap-website] Use fetch() instead of iframe to make remote control requests (PR #5375)

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

Re: [openstreetmap/openstreetmap-website] Use $.ajax instead of iframe to make remote control requests (PR #3760)

2024-12-08 Thread Tom Hughes via rails-dev
Superseded by #5375. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/3760#issuecomment-2526152212 You are receiving this because you are subscribed to this thread. Message ID: ___ rails