From 16e65edd0d1cfb7b5e350c5a588dfbdd1c424e35 Mon Sep 17 00:00:00 2001 From: M Fahru Date: Wed, 20 Nov 2024 10:42:58 -0700 Subject: [PATCH] Merge pull request #21995 from overleaf/mf-disable-footer-tooltip-if-element-is-not-available [web] Only trigger footer tooltip if target element is available GitOrigin-RevId: e0aff735243369a0d2180174231951054f1ee533 --- .../web/frontend/js/features/tooltip/index-bs5.ts | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/services/web/frontend/js/features/tooltip/index-bs5.ts b/services/web/frontend/js/features/tooltip/index-bs5.ts index f748504e4d..62c199e2e6 100644 --- a/services/web/frontend/js/features/tooltip/index-bs5.ts +++ b/services/web/frontend/js/features/tooltip/index-bs5.ts @@ -14,19 +14,18 @@ function getElementWidth(el: Element) { const footerLanguageElement = document.querySelector( '[data-ol-lang-selector-tooltip]' ) as Element +if (footerLanguageElement) { + // eslint-disable-next-line no-new + new Tooltip(footerLanguageElement) +} const allTooltips = document.querySelectorAll('[data-bs-toggle="tooltip"]') - -const possibleBadgeTooltips = document.querySelectorAll('[data-badge-tooltip]') - -// eslint-disable-next-line no-unused-vars -const footLangTooltip = new Tooltip(footerLanguageElement) - allTooltips.forEach(element => { // eslint-disable-next-line no-unused-vars const tooltip = new Tooltip(element) }) +const possibleBadgeTooltips = document.querySelectorAll('[data-badge-tooltip]') possibleBadgeTooltips.forEach(element => { // Put data-badge-tooltip on .badge-content // then tooltip is only shown if content is clipped due to max-width on .badge