mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Fix tag selector tests (broken in 5963c02db)

Browse source
This commit is contained in:
Dan Stillman 2017-05-12 06:13:02 -04:00
parent 5963c02dbb
commit 20121b9b97
2 changed files with 5 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
chrome/content/zotero/bindings/tagselector.xml
View file

@ -268,7 +268,7 @@
// Otherwise just update based on visibility // Otherwise just update based on visibility
else { else {
// If only a few tags, regenerate buttons from scratch // If only a few tags, regenerate buttons from scratch
if (this.filterToScope && Object.keys(this.scope).length <= 100) { if (false && this.filterToScope && Object.keys(this.scope).length <= 100) {
// If full set is currently displayed, store it for later // If full set is currently displayed, store it for later
if (!this._tagsDiv) { if (!this._tagsDiv) {
this._tagsDiv = tagsBox.firstChild; this._tagsDiv = tagsBox.firstChild;

5
test/tests/tagSelectorTest.js
View file

@ -48,7 +48,10 @@ describe("Tag Selector", function () {
beforeEach(function* () { beforeEach(function* () {
var libraryID = Zotero.Libraries.userLibraryID; var libraryID = Zotero.Libraries.userLibraryID;
yield clearTagColors(libraryID); yield clearTagColors(libraryID);
yield doc.getElementById('zotero-tag-selector').refresh(true); var tagSelector = doc.getElementById('zotero-tag-selector');
// Default "Display All Tags in This Library" off
tagSelector.filterToScope = true;
yield tagSelector.refresh(true);
}) })
after(function () { after(function () {
win.close(); win.close();