Martin Raifer

Adding a warning would be too confusing and disturbing regular workflows. If you're working with huge multiselections, you should probable split the work up into slightly smaller chunks.

Yeah, adding layer tags to overlapping buildings rarely properly addresses the underlying map data quality issue of building features. It might actually be better to omit those _fixes_ in this...

I've occasionally seen (semi-)underground parking mapped as overlapping buildings, like here: https://www.openstreetmap.org/way/180842312

I can't easily reproduce the bug: just moving the photo outside of the viewport does not seem to be enough to trigger the issue (see screencast below). Can you please...

PS: If you can reproduce a bug also on the preview instance (the development version at ideditor.netlify.app), then please post the error messages you see from that version of iD:...

this seems to be the line where the crash happened: https://github.com/openstreetmap/iD/blob/6bfcc17e2d43525511e7a206b8c74e60834771b5/modules/ui/photoviewer.js#L128 It could also be possible that the bug was fixed "accidentally" already in the dev instance by some of...

This should be now hopefully working as intended: the click handler was not properly registered, thus it could fail when the button was rendered for the first time.

Yeah, such a message would be useful when starting at an intersection vertex to cycle through the nodes with `[` / `]`.

This seems to work now. Maybe some of the other recent bugfixes fixed this problem as well. Can you please confirm whether this is fixed on https://preview.ideditor.com/ ?

Oh. Yeah, this does not seem optimal. The text mode raw tag editor does appear to scramble the whole tag around in rtl mode, making it really hard to figure...