Merge pull request #10462 from Snuffleupagus/pr-10152

Tweak the `DOMTokenList.toggle` polyfill (issue 10460)
This commit is contained in:
Tim van der Meij 2019-01-16 22:44:20 +01:00 committed by GitHub
commit f711df583a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 3 deletions

View File

@ -101,14 +101,17 @@ const hasDOM = typeof window === 'object' && typeof document === 'object';
if (div.classList.toggle('test', 0) === false) {
return;
}
const originalDOMTokenListToggle = DOMTokenList.prototype.toggle;
DOMTokenList.prototype.toggle = function(token) {
if (arguments.length > 1) {
const force = !!arguments[1];
return (this[force ? 'add' : 'remove'](token), force);
}
return originalDOMTokenListToggle(token);
if (this.contains(token)) {
return (this.remove(token), false);
}
return (this.add(token), true);
};
})();

View File

@ -180,7 +180,7 @@ class PDFFindBar {
}
Promise.resolve(matchesCountMsg).then((msg) => {
this.findResultsCount.textContent = msg;
this.findResultsCount.classList[!total ? 'add' : 'remove']('hidden');
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();