diff --git a/chrome/content/zotero/xpcom/itemTreeView.js b/chrome/content/zotero/xpcom/itemTreeView.js index bd792862b..2ab36bdfc 100644 --- a/chrome/content/zotero/xpcom/itemTreeView.js +++ b/chrome/content/zotero/xpcom/itemTreeView.js @@ -2332,69 +2332,73 @@ Zotero.ItemTreeView.prototype.onColumnPickerShowing = function (event) { Zotero.debug(e, 1); } + // // Secondary Sort menu - try { - let id = prefix + 'sort-menu'; - let primaryField = this.getSortField(); - let sortFields = this.getSortFields(); - let secondaryField = false; - if (sortFields[1]) { - secondaryField = sortFields[1]; - } - - // Get localized names from treecols, since the names are currently done via .dtd - let treecols = menupopup.parentNode.parentNode; - let primaryFieldLabel = treecols.getElementsByAttribute('id', - 'zotero-items-column-' + primaryField)[0].getAttribute('label'); - - let sortMenu = doc.createElementNS(ns, 'menu'); - sortMenu.setAttribute('label', - Zotero.getString('pane.items.columnChooser.secondarySort', primaryFieldLabel)); - sortMenu.setAttribute('anonid', id); - - let sortMenuPopup = doc.createElementNS(ns, 'menupopup'); - sortMenuPopup.setAttribute('anonid', id + '-popup'); - - // Generate menuitems - let sortOptions = [ - 'title', - 'firstCreator', - 'itemType', - 'date', - 'year', - 'publisher', - 'publicationTitle', - 'dateAdded', - 'dateModified' - ]; - for (let i=0; i