commit | 9b58fea0efc0068133629d09bfc4c6974c7e2ab5 | [log] [tgz] |
---|---|---|
author | Ben Rohlfs <brohlfs@google.com> | Mon Nov 25 10:13:35 2019 +0100 |
committer | David Pursehouse <dpursehouse@collab.net> | Thu Nov 28 07:16:35 2019 +0000 |
tree | 8a26fee541b0648101a9f23b9bd329bc8e67dce4 | |
parent | c8a5f9c187c47e87cfbe783252cabda93e993abc [diff] |
Fix branch overflow in change-list-item Screenshots: https://imgur.com/a/LpkWUOC <a> should really have display:inline as usual. You can argue for display:block, because it increases the click target, but display:inline-block is never a great idea. I have researched the origin of this and apparently there was a misunderstanding in Change 119891 that display:inline-block would still increase the click target, but that is not the case. Change-Id: I76d19d312920110d5c5b6221e3ff197fc6159112 (cherry picked from commit 1bcdba68c54b1bf055cb58d35f38fccc9ebd9f8c)
diff --git a/polygerrit-ui/app/elements/change-list/gr-change-list-item/gr-change-list-item.html b/polygerrit-ui/app/elements/change-list/gr-change-list-item/gr-change-list-item.html index 568fe12..2374677 100644 --- a/polygerrit-ui/app/elements/change-list/gr-change-list-item/gr-change-list-item.html +++ b/polygerrit-ui/app/elements/change-list/gr-change-list-item/gr-change-list-item.html
@@ -90,7 +90,6 @@ a { color: inherit; cursor: pointer; - display: inline-block; text-decoration: none; } a:hover {