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
diff --git a/polygerrit-ui/app/embed/diff/gr-diff/gr-diff.ts b/polygerrit-ui/app/embed/diff/gr-diff/gr-diff.ts
index 34e9d5a..1c5d594 100644
--- a/polygerrit-ui/app/embed/diff/gr-diff/gr-diff.ts
+++ b/polygerrit-ui/app/embed/diff/gr-diff/gr-diff.ts
@@ -316,7 +316,6 @@
}
.diffContainer {
max-width: var(--diff-max-width, none);
- display: flex;
font-family: var(--monospace-font-family);
}
table {
