tree 3cf940a934c5e04e53f6d21c33a1ff18389f50f2
parent 25c47811fd4f6cfee8cf164517b835f7c255ae76
author Ben Rohlfs <brohlfs@google.com> 1679399806 +0100
committer Ben Rohlfs <brohlfs@google.com> 1679399806 +0100

Remove `display:flex` from diff container element

This was misplacing the "whitespace only" message:
https://imgur.com/a/YOxZBRt

Digging into the history, the `flex` display came from a time where the
left and right sides each had their own element. Today we just have one
diff table, so `block` is more appropriate than `flex`.

This was only recently broken, because for a long while the diff
container had pseudo elements as their only child such as
`gr-diff-builder`. Those elements were converted from elements to just
classes. And when they got removed from the DOM hiearchy, the bug
manifested, because then the diff container started having two
children: the diff table and the "whitespace only" div.

Release-Notes: skip
Google-Bug-Id: b/274572407
Change-Id: I6d63acc74254860194c6d5e5e7d1226c59093aa6