commit c499ed333b534629e0b99ee7f668597ca54a6ae3
author Ben Rohlfs <brohlfs@google.com> Fri Jan 24 12:18:58 2020 +0100
committer Ben Rohlfs <brohlfs@google.com> Sat Jan 25 13:51:11 2020 +0000
tree e5974f60d0d4f7a96fbd71ab2514db1bccb81837
parent 8f716d7c02b053f19dbfdc92d1c0d8eef0622128

Fix the alignment of the placeholder in reply dialog

"Add reviewer..." was vertically not correctly positioned.

Also reduce/align the vertical heights and margins a little, and use
a little bit less space for the "Reviewers" and "CC" labels.

Change-Id: I1ac7a64fb426218ee1ba0b4e999b7b29f4b4275d

polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.html

diff --git a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.html b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.html
index c177283..d54669f 100644
--- a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.html
+++ b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.html
@@ -86,19 +86,17 @@
       }
       .peopleList {
         display: flex;
-        padding-top: var(--spacing-xxs);
       }
       .peopleListLabel {
         color: var(--deemphasized-text-color);
         margin-top: var(--spacing-xs);
-        min-width: 7em;
+        min-width: 6em;
         padding-right: var(--spacing-m);
       }
       gr-account-list {
         display: flex;
         flex-wrap: wrap;
         flex: 1;
-        min-height: 1.8em;
       }
       #reviewerConfirmationOverlay {
         padding: var(--spacing-l);

polygerrit-ui/app/elements/shared/gr-account-list/gr-account-list.html

diff --git a/polygerrit-ui/app/elements/shared/gr-account-list/gr-account-list.html b/polygerrit-ui/app/elements/shared/gr-account-list/gr-account-list.html
index 6103d6f..37591d8 100644
--- a/polygerrit-ui/app/elements/shared/gr-account-list/gr-account-list.html
+++ b/polygerrit-ui/app/elements/shared/gr-account-list/gr-account-list.html
@@ -32,6 +32,7 @@
         display: flex;
         flex: 1;
         min-width: 10em;
+        margin: var(--spacing-xs) var(--spacing-xs) var(--spacing-xs) 0;
       }
       .group {
         --account-label-suffix: ' (group)';