commit 2b092152397c6fe06a75260627ddc861ebfc0d6a
author Ben Rohlfs <brohlfs@google.com> Tue Nov 10 10:29:58 2020 +0100
committer Ben Rohlfs <brohlfs@google.com> Wed Nov 11 08:40:02 2020 +0000
tree b1fe683668689ed131dbe17ad5b3f292280df763
parent 2dade92c502eea97673878b496d32b7ec2107626

Specify line-height for code in terms of rem

We currently see 1px horizontal lines that originate from some <tr>s
being 17px instead of 16px. Apparently changing line-height to use `rem`
instead of a multiplier fixes the issue.

Change-Id: I4eec06f6fb02cf344e4ced1a6087810b3fa6512a

polygerrit-ui/app/styles/themes/app-theme.ts

diff --git a/polygerrit-ui/app/styles/themes/app-theme.ts b/polygerrit-ui/app/styles/themes/app-theme.ts
index 9b62718..61e1a7b 100644
--- a/polygerrit-ui/app/styles/themes/app-theme.ts
+++ b/polygerrit-ui/app/styles/themes/app-theme.ts
@@ -122,7 +122,7 @@
     --font-size-h3: 1.143rem;   /* 16px */
     --font-size-h2: 1.429rem;   /* 20px */
     --font-size-h1: 1.714rem;   /* 24px */
-    --line-height-code: 1.334;      /* 16px */
+    --line-height-code: 1.143rem;   /* 16px */
     --line-height-mono: 1.286rem;   /* 18px */
     --line-height-small: 1.143rem;  /* 16px */
     --line-height-normal: 1.429rem; /* 20px */