From 6fe4b3a5ae5fb132043ba97ea8c4a068aac7993e Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Thu, 3 Feb 2022 22:16:56 +0100 Subject: [PATCH] Simplify the `findResultsCount` span toggling, by using the same approach as with the `findMsg` span --- web/pdf_find_bar.js | 1 - web/viewer.css | 2 ++ web/viewer.html | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/web/pdf_find_bar.js b/web/pdf_find_bar.js index 54fda4ddb..c004017a7 100644 --- a/web/pdf_find_bar.js +++ b/web/pdf_find_bar.js @@ -163,7 +163,6 @@ class PDFFindBar { } matchCountMsg.then(msg => { this.findResultsCount.textContent = msg; - this.findResultsCount.classList.toggle("hidden", !total); // Since `updateResultsCount` may be called from `PDFFindController`, // ensure that the width of the findbar is always updated correctly. this._adjustWidth(); diff --git a/web/viewer.css b/web/viewer.css index a7a1188ef..b7b4fc792 100644 --- a/web/viewer.css +++ b/web/viewer.css @@ -691,6 +691,8 @@ html[dir="ltr"] .doorHangerRight:before { #findMsg { color: rgba(251, 0, 0, 1); } + +#findResultsCount:empty, #findMsg:empty { display: none; } diff --git a/web/viewer.html b/web/viewer.html index 16e0ccefa..1317e0deb 100644 --- a/web/viewer.html +++ b/web/viewer.html @@ -142,7 +142,7 @@ See https://github.com/adobe-type-tools/cmap-resources
- +