Use gr-tooltip in size cell
Gr-tooltip provides a number of advantages over native browser titles --
this change swaps out the use of the title attribute for gr-tooltip.
Bug: Issue 8487
Change-Id: I277b40343dc5ec8e33185c7e6c9d5df159912e09
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 1a9099d..d0286be 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
@@ -26,6 +26,7 @@
<link rel="import" href="../../shared/gr-change-status/gr-change-status.html">
<link rel="import" href="../../shared/gr-date-formatter/gr-date-formatter.html">
<link rel="import" href="../../shared/gr-limited-text/gr-limited-text.html">
+<link rel="import" href="../../shared/gr-tooltip-content/gr-tooltip-content.html">
<link rel="import" href="../../../styles/shared-styles.html">
<dom-module id="gr-change-list-item">
@@ -81,6 +82,9 @@
.status .comma:first-of-type {
display: none;
}
+ .size gr-tooltip-content {
+ max-width: 2.5rem;
+ }
a {
color: var(--default-text-color);
cursor: pointer;
@@ -93,9 +97,6 @@
.positionIndicator {
visibility: hidden;
}
- .size {
- text-align: center;
- }
:host([selected]) .positionIndicator {
visibility: visible;
}
@@ -205,9 +206,12 @@
date-str="[[change.updated]]"></gr-date-formatter>
</td>
<td class="cell size"
- title$="[[_computeSizeTooltip(change)]]"
hidden$="[[isColumnHidden('Size', visibleChangeTableColumns)]]">
- <span>[[_computeChangeSize(change)]]</span>
+ <gr-tooltip-content
+ has-tooltip
+ title="[[_computeSizeTooltip(change)]]">
+ <span>[[_computeChangeSize(change)]]</span>
+ </gr-tooltip-content>
</td>
<template is="dom-repeat" items="[[labelNames]]" as="labelName">
<td title$="[[_computeLabelTitle(change, labelName)]]"
