diff options
| author | Chris Poucet <poucet@google.com> | 2024-04-09 15:25:30 +0200 |
|---|---|---|
| committer | Paladox none <thomasmulhall410@yahoo.com> | 2024-04-09 14:16:01 +0000 |
| commit | 133a676ecda316f8cfe2325af7a6a8de4deb1b3c (patch) | |
| tree | 1410861dbf76d541acbd74e1c6df975892a16d60 | |
| parent | ba68993add5b7a122b9961fd3bcf7ca2811cfee5 (diff) | |
Fix a bug with paper-tooltip.
We need to set properties, not attributes for paper-tooltip.
Google-Bug-Id: b/333416564
Release-Notes: skip
Change-Id: I48d41184d10aa0b2fd719717c38664792fa8bc54
(cherry picked from commit 4f7449246c08c9d6066ebbf5dca54423d07a9afc)
| -rw-r--r-- | polygerrit-ui/app/elements/checks/gr-checks-results.ts | 4 | ||||
| -rw-r--r-- | polygerrit-ui/app/elements/checks/gr-checks-results_test.ts | 9 |
2 files changed, 3 insertions, 10 deletions
diff --git a/polygerrit-ui/app/elements/checks/gr-checks-results.ts b/polygerrit-ui/app/elements/checks/gr-checks-results.ts index afb9d0fa9f..1057664a9a 100644 --- a/polygerrit-ui/app/elements/checks/gr-checks-results.ts +++ b/polygerrit-ui/app/elements/checks/gr-checks-results.ts @@ -495,7 +495,7 @@ export class GrResultRow extends LitElement { return html` <div class="label ${status}"> <span>${label} ${valueStr}</span> - <paper-tooltip offset="5" ?fitToVisibleBounds=${true}> + <paper-tooltip offset="5" .fitToVisibleBounds=${true}> The check result has (probably) influenced this label vote. </paper-tooltip> </div> @@ -609,7 +609,7 @@ export class GrResultRow extends LitElement { @click=${(e: MouseEvent) => this.tagClick(e, tag.name)} > <span>${tag.name}</span> - <paper-tooltip offset="5" ?fitToVisibleBounds=${true}> + <paper-tooltip offset="5" .fitToVisibleBounds=${true}> ${tag.tooltip ?? 'A category tag for this check result. Click to filter.'} </paper-tooltip> diff --git a/polygerrit-ui/app/elements/checks/gr-checks-results_test.ts b/polygerrit-ui/app/elements/checks/gr-checks-results_test.ts index 19205c4750..a349b58b4e 100644 --- a/polygerrit-ui/app/elements/checks/gr-checks-results_test.ts +++ b/polygerrit-ui/app/elements/checks/gr-checks-results_test.ts @@ -45,12 +45,7 @@ suite('gr-result-row test', () => { /* HTML */ ` <div class="approved label"> <span> test-label +1 </span> - <paper-tooltip - fittovisiblebounds="" - offset="5" - role="tooltip" - tabindex="-1" - > + <paper-tooltip offset="5" role="tooltip" tabindex="-1"> The check result has (probably) influenced this label vote. </paper-tooltip> </div> @@ -95,7 +90,6 @@ suite('gr-result-row test', () => { <button class="tag"> <span> OBSOLETE </span> <paper-tooltip - fittovisiblebounds="" offset="5" role="tooltip" tabindex="-1" @@ -106,7 +100,6 @@ suite('gr-result-row test', () => { <button class="tag"> <span> E2E </span> <paper-tooltip - fittovisiblebounds="" offset="5" role="tooltip" tabindex="-1" |