[openstreetmap/openstreetmap-website] Show tag changes in object history view (PR #6448)

[Waldir Pimenta]

waldyrious left a comment (openstreetmap/openstreetmap-website#6448)

> This option wasn't abandoned or moved away from. It was never attempted.

Ah, my bad — I totally overlooked the author of [this comment](https://github.com/openstreetmap/openstreetmap-website/pull/6448#issuecomment-3410364736) :sweat_smile:

(And responding to your question, my suggestion would be precisely that one — not sure if implementing it would bring more value than the mock-up in that comment, though, at least until a decision is made about whether it's desirable, or concrete use cases where its usage may not be optimal are identified.)

> Do you suggest using `<del>` and `<ins>` tags in addition to the + and − indicators? Or do you mean we should return to inlining a changed tag, using `<del>` and `<ins>` to distinguish old and new values? 

IMHO the latter should be sufficient, since for regular vision users the colors alone would convey the necessary information, and that way we'd avoid the extra visual complexity.

> Inlining could be useful if we later introduce per-word or per-character diffing, to make subtle changes easier to spot.

Agreed, but to be clear I'd argue it'd still be useful with line-based diffing.

> Without that more granular diffing, this more structured design would be more effective at highlighting subtle changes.

By "more structured design" do you mean the +/- indicators? And if so, can you explain what is it that makes it more effective at highlighting subtle changes? I'm not sure I follow.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6448#issuecomment-3540695994
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6448/c3540695994 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251117/2f7ed2ac/attachment.htm>