From 9b58fea0efc0068133629d09bfc4c6974c7e2ab5 Mon Sep 17 00:00:00 2001 From: Ben Rohlfs Date: Mon, 25 Nov 2019 10:13:35 +0100 Subject: [PATCH] Fix branch overflow in change-list-item Screenshots: https://imgur.com/a/LpkWUOC 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) --- .../change-list/gr-change-list-item/gr-change-list-item.html | 1 - 1 file changed, 1 deletion(-) 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 568fe12b41..23746770ea 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 @@ limitations under the License. a { color: inherit; cursor: pointer; - display: inline-block; text-decoration: none; } a:hover {