From a1ca71d0e5d672cbc480e6500d3cebb047faece4 Mon Sep 17 00:00:00 2001
From: Kasper Nilsson <kaspern@google.com>
Date: Mon, 11 Sep 2017 14:07:46 -0700
Subject: [PATCH] Move overflow-y to gr-account-list internal list

The reply dialog set the max height of the gr-account-list component.
Because the autocomplete dropdown is positioned absolutely, it appeared
to not exist on the input as the account-list required more and more
scroll.

Applying a max-height and overflow-y to the list of account chips within
the account-list fixes the autocomplete dropdown placement.

Bug: Issue 7112
Change-Id: I5714f8f08c24d4ed2048b75dbc9ccf0e69b17774
---
 .../elements/change/gr-account-list/gr-account-list.html    | 5 ++++-
 .../elements/change/gr-reply-dialog/gr-reply-dialog.html    | 6 ++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/polygerrit-ui/app/elements/change/gr-account-list/gr-account-list.html b/polygerrit-ui/app/elements/change/gr-account-list/gr-account-list.html
index 7b18b5e69b..3d98ecb7ef 100644
--- a/polygerrit-ui/app/elements/change/gr-account-list/gr-account-list.html
+++ b/polygerrit-ui/app/elements/change/gr-account-list/gr-account-list.html
@@ -37,12 +37,15 @@ limitations under the License.
       .pending-add {
         font-style: italic;
       }
+      .list {
+        @apply --account-list-style;
+      }
     </style>
     <!--
       NOTE(Issue 6419): Nest the inner dom-repeat template in a div rather than
       as a direct child of the dom-module's template.
     -->
-    <div>
+    <div class="list">
       <template id="chips" is="dom-repeat" items="[[accounts]]" as="account">
         <gr-account-chip
             account="[[account]]"
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 68c7fd7751..2f9f78cf74 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
@@ -76,8 +76,10 @@ limitations under the License.
         display: flex;
         flex-wrap: wrap;
         flex: 1;
-        max-height: 12em;
-        overflow-y: auto;
+        --account-list-style: {
+          max-height: 12em;
+          overflow-y: auto;
+        }
       }
       #reviewerConfirmationOverlay {
         padding: 1em;