<p></p>
<p><b>@1ec5</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6448#discussion_r2461733364">app/helpers/browse_tag_changes_helper.rb</a>:</p>
<pre style='color:#555'>> +    when :unmodified
+      "diff-indicator-cell diff-unmodified"
+    else
+      "diff-indicator-cell"
+    end
+  end
+
+  def format_tag_row_with_change(key, change_info)
+    case change_info[:type]
+    when :modified
+      # Generate two rows for modified tags
+      value_cells = format_tag_value_with_change(key, change_info)
+      [
+        tag.tr(:class => tag_change_class(change_info[:type])) do
+          safe_join([
+                      tag.th(format_key(key), :class => "diff-key-modified", :rowspan => 2),
</pre>
<p dir="auto">How does the rowspan interact with the Wikidata preview, which normally adds a row below the existing row?</p>

<hr>

<p>In <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6448#discussion_r2461763201">app/assets/stylesheets/common.scss</a>:</p>
<pre style='color:#555'>> +.diff-indicator-cell.diff-removed {
+  background-color: var(--bs-danger);
+}
+
+.diff-indicator-cell.diff-unmodified {
+  background-color: var(--bs-border-color);
+  color: var(--bs-body-color);
+}
+
+.diff-cell {
+  padding: 0.25rem 0.5rem !important;
+}
+
+/* Modified tag key cell styling */
+.diff-key-modified {
+  color: var(--bs-body-color) !important;
</pre>
<p dir="auto">What if the rightmost column still has a red or green background like you originally had, while the key column remains yellow? It would make the relationship between the ± column and this column a little clearer. (I realize this is going back and forth to an older design, so feel free to ignore this suggestion if it isn’t a good tradeoff.)</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/6448#pullrequestreview-3378505734">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLIIL2UZH2UMYZWYTML3ZJ5BNAVCNFSM6AAAAACJGVZNZ2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTGNZYGUYDKNZTGQ">unsubscribe</a>.<br />You are receiving this because you are subscribed to this thread.<img src="https://github.com/notifications/beacon/AAK2OLNKFTAUZ77ISFF7SG33ZJ5BNA5CNFSM6AAAAACJGVZNZ2WGG33NNVSW45C7OR4XAZNRKB2WY3CSMVYXKZLTORJGK5TJMV32UY3PNVWWK3TUL5UWJTWJL7UAM.gif" height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; max-width: 0; max-height: 0; mso-hide: all">Message ID: <span><openstreetmap/openstreetmap-website/pull/6448/review/3378505734</span><span>@</span><span>github</span><span>.</span><span>com></span></span></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/6448#pullrequestreview-3378505734",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/6448#pullrequestreview-3378505734",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>