Bug 412019 - Applications pref pane: sorting by application only partially remembered. r=gavin, a1.9=beltzner

This commit is contained in:
florian@queze.net 2008-01-22 20:35:04 -08:00
parent 102194e4a7
commit e7d864aae2

View File

@ -850,10 +850,16 @@ var gApplicationsPane = {
// Figure out how we should be sorting the list. We persist sort settings // Figure out how we should be sorting the list. We persist sort settings
// across sessions, so we can't assume the default sort column/direction. // across sessions, so we can't assume the default sort column/direction.
// XXX should we be using the XUL sort service instead? // XXX should we be using the XUL sort service instead?
if (document.getElementById("typeColumn").hasAttribute("sortDirection")) if (document.getElementById("actionColumn").hasAttribute("sortDirection")) {
this._sortColumn = document.getElementById("typeColumn");
else if (document.getElementById("actionColumn").hasAttribute("sortDirection"))
this._sortColumn = document.getElementById("actionColumn"); this._sortColumn = document.getElementById("actionColumn");
// The typeColumn element always has a sortDirection attribute,
// either because it was persisted or because the default value
// from the xul file was used. If we are sorting on the other
// column, we should remove it.
document.getElementById("typeColumn").removeAttribute("sortDirection");
}
else
this._sortColumn = document.getElementById("typeColumn");
// Load the data and build the list of handlers. // Load the data and build the list of handlers.
// By doing this in a timeout, we let the preferences dialog resize itself // By doing this in a timeout, we let the preferences dialog resize itself