Added grouping of items by library and separators for libraries to QuickFormat dialog

This commit is contained in:
Mikko Rönkkö 2011-11-03 16:31:31 +02:00
parent 9fa7dc5249
commit 3d928ce55f

View file

@ -265,19 +265,38 @@ var Zotero_QuickFormat = new function () {
while(referenceBox.hasChildNodes()) referenceBox.removeChild(referenceBox.firstChild);
var firstSelectableIndex = 0;
if(ids.length) {
if(ids.length > 50) ids = ids.slice(0, 50);
var items = Zotero.Items.get(ids);
firstSelectableIndex = 1;
//TODO: sort the currently used items in before any other items
items.sort(function(a, b) {return a.libraryID > b.libraryID})
var previousLibrary = -1;
for(var i=0, n=items.length; i<n; i++) {
referenceBox.appendChild(_buildListItem(items[i]));
if(previousLibrary!=items[i].libraryID){
//TODO: Make localized labels and resolve the library name based on library ID
referenceBox.appendChild(_buildListSeparator("Items from "+items[i].libraryID));
}
referenceBox.appendChild(_buildListItem(items[i]),false);
previousLibrary=items[i].libraryID
}
}
_resize();
referenceBox.selectedIndex = 0;
referenceBox.ensureIndexIsVisible(0);
referenceBox.selectedIndex = firstSelectableIndex;
referenceBox.ensureIndexIsVisible(firstSelectableIndex);
}
/**
* Builds a string describing an item. We avoid CSL here for speed.
@ -374,6 +393,25 @@ var Zotero_QuickFormat = new function () {
return rll;
}
function _buildListSeparator(labelText) {
var titleNode = document.createElement("label");
//TODO: CSS style needed for this class
titleNode.setAttribute("class", "quick-format-separator");
titleNode.setAttribute("flex", "1");
titleNode.setAttribute("crop", "end");
titleNode.setAttribute("value", labelText);
// add to rich list item
var rll = document.createElement("richlistitem");
rll.setAttribute("orient", "vertical");
rll.setAttribute("flex", "1");
rll.setAttribute("disabled", true);
rll.setAttribute("class", "quick-format-separator");
rll.appendChild(titleNode);
return rll;
}
/**
* Builds the string to go inside a bubble
@ -719,10 +757,24 @@ var Zotero_QuickFormat = new function () {
} else if(keyCode === event.DOM_VK_UP) {
var selectedItem = referenceBox.selectedItem;
var previousSibling;
if((previousSibling = selectedItem.previousSibling)) {
//Seek the closet previous sibling that is not disabled
while((previousSibling = selectedItem.previousSibling) && previousSibling.getAttribute("disabled")){
selectedItem = previousSibling;
}
//If found, change to that
if(previousSibling) {
referenceBox.selectedItem = previousSibling;
referenceBox.ensureElementIsVisible(previousSibling);
//If there are separators before this item, ensure that they are visible
var visibleItem = previousSibling;
while(visibleItem.previousSibling && visibleItem.previousSibling.getAttribute("disabled")){
visibleItem = visibleItem.previousSibling;
}
referenceBox.ensureElementIsVisible(visibleItem);
event.preventDefault();
};
} else if(keyCode === event.DOM_VK_DOWN) {
@ -768,7 +820,15 @@ var Zotero_QuickFormat = new function () {
} else {
var selectedItem = referenceBox.selectedItem;
var nextSibling;
if((nextSibling = selectedItem.nextSibling)) {
//Seek the closet next sibling that is not disabled
while((nextSibling = selectedItem.nextSibling) && nextSibling.getAttribute("disabled")){
selectedItem = nextSibling;
}
//If found, change to that
if(nextSibling){
referenceBox.selectedItem = nextSibling;
referenceBox.ensureElementIsVisible(nextSibling);
event.preventDefault();