Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Simplify the findResultsCount span toggling, by using the same approach as with the findMsg span

Browse Source
This commit is contained in:
Jonas Jenwald 2022-02-03 22:16:56 +01:00
parent d0354d20b3
commit 6fe4b3a5ae
3 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
web/pdf_find_bar.js
View File

@ -163,7 +163,6 @@ class PDFFindBar {
} }
matchCountMsg.then(msg => { matchCountMsg.then(msg => {
this.findResultsCount.textContent = msg; this.findResultsCount.textContent = msg;
this.findResultsCount.classList.toggle("hidden", !total);
// Since `updateResultsCount` may be called from `PDFFindController`, // Since `updateResultsCount` may be called from `PDFFindController`,
// ensure that the width of the findbar is always updated correctly. // ensure that the width of the findbar is always updated correctly.
this._adjustWidth(); this._adjustWidth();

2
web/viewer.css
View File

@ -691,6 +691,8 @@ html[dir="ltr"] .doorHangerRight:before {
#findMsg { #findMsg {
color: rgba(251, 0, 0, 1); color: rgba(251, 0, 0, 1);
} }
#findResultsCount:empty,
#findMsg:empty { #findMsg:empty {
display: none; display: none;
} }

2
web/viewer.html
View File

@ -142,7 +142,7 @@ See https://github.com/adobe-type-tools/cmap-resources
</div> </div>
<div class="findbarMessageContainer"> <div class="findbarMessageContainer">
<span id="findResultsCount" class="toolbarLabel hidden"></span> <span id="findResultsCount" class="toolbarLabel"></span>
</div> </div>
<div class="findbarMessageContainer"> <div class="findbarMessageContainer">
<span id="findMsg" class="toolbarLabel"></span> <span id="findMsg" class="toolbarLabel"></span>