Re: [openstreetmap/openstreetmap-website] add "edit" button to mobile version (#2233)

2024-12-01 Thread Simon Poole via rails-dev
I would end on noting that the button does nothing or after while causes an error for everybody that doesn't have iD selected as default. At least remove the drop down and only start iD as long as the other issues are not resolved. -- Reply to this email directly or view it on GitHub: https:/

Re: [openstreetmap/openstreetmap-website] Show edit and export on small screens (PR #4839)

2024-12-01 Thread Anton Khorev via rails-dev
I guess we don't need this PR now? -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/4839#issuecomment-2510761962 You are receiving this because you are subscribed to this thread. Message ID: ___

Re: [openstreetmap/openstreetmap-website] add "edit" button to mobile version (#2233)

2024-12-01 Thread Anton Khorev via rails-dev
The *Edit* button is visible on all window widths after #5151. How well it works is a different question. For example, I still can't get element selection by tapping to work all of the time in iD, see https://www.openstreetmap.org/user/Anton%20Khorev/diary/401385. On the other hand, opening iD

Re: [openstreetmap/openstreetmap-website] add "edit" button to mobile version (#2233)

2024-12-01 Thread Anton Khorev via rails-dev
Closed #2233 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/2233#event-15488190406 You are receiving this because you are subscribed to this thread. Message ID: ___ rai

Re: [openstreetmap/openstreetmap-website] Move export tab to right hand tool pane (#3040)

2024-12-01 Thread Anton Khorev via rails-dev
Closed #3040 as not planned. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/3040#event-15487701533 You are receiving this because you are subscribed to this thread. Message ID: ___ r

Re: [openstreetmap/openstreetmap-website] Move export tab to right hand tool pane (#3040)

2024-12-01 Thread Anton Khorev via rails-dev
*Export* was moved in #5151, therefore this issue may no longer be relevant. If you still want to move *Export* elsewhere or rename it, it's better to open a new issue. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/3040#issue

Re: [openstreetmap/openstreetmap-website] Add "view" tab to main navigation (#2420)

2024-12-01 Thread Anton Khorev via rails-dev
Since we merged https://github.com/openstreetmap/openstreetmap-website/pull/5151 we are less likely to merge this because the goal was to save space in the header. Although it still may be possible, then it's going to be a trade of *History* and *Export* for *View* which may still save some spa

[openstreetmap/openstreetmap-website] Dynamic "more" dropdown (PR #5363)

2024-12-01 Thread Anton Khorev via rails-dev
![image](https://github.com/user-attachments/assets/5172b87d-e22d-483f-81fe-012bd194f704) ![image](https://github.com/user-attachments/assets/429240de-8762-4bed-9da5-643b13bd7ac6) ![image](https://github.com/user-attachments/assets/2d1cdcd7-f7f8-4edc-abc4-e380faddd6c6) ![image](https://github.com/u

Re: [openstreetmap/openstreetmap-website] Fixed small mode breakpoint + dynamic "more" dropdown (PR #4862)

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

Re: [openstreetmap/openstreetmap-website] Dark Mode bugs/improvements for the UI (Issue #5329)

2024-12-01 Thread Roman Deev via rails-dev
Although iD doesn't have a dark theme, it seems that you can reduce the annoyance of blinding when loading it by setting at least some dark or transparent background for the iframe. https://github.com/user-attachments/assets/8132c4d3-b55a-435f-8b7c-47042b43828f -- Reply to this email direct

Re: [openstreetmap/openstreetmap-website] Bump eslint from 9.15.0 to 9.16.0 (PR #5360)

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