this fixes default tab selection for Firefox

This commit is contained in:
gorhill 2015-01-13 11:01:33 -05:00
parent 580d3a81db
commit 5ec300602a
1 changed files with 17 additions and 13 deletions

View File

@ -36,22 +36,26 @@ var messager = vAPI.messaging.channel('stats.js');
var renderPageSelector = function(targetTabId) { var renderPageSelector = function(targetTabId) {
var selectedTabId = targetTabId || uDom('#pageSelector').val(); var selectedTabId = targetTabId || uDom('#pageSelector').val();
var onTabReceived = function(tabId, tabTitle) { var onDataReceived = function(pageTitles) {
uDom('#pageSelector').append('<option value="' + tabId + '">' + tabTitle); if ( pageTitles.hasOwnProperty(selectedTabId) === false ) {
if ( tabId.toString() === selectedTabId ) { selectedTabId = pageTitles[0];
uDom('#pageSelector').val(tabId);
} }
}; var select = uDom('#pageSelector').empty();
var onDataReceived = function(pageDetails) { var option;
uDom('#pageSelector option').remove(); for ( var tabId in pageTitles ) {
if ( pageDetails.hasOwnProperty(selectedTabId) === false ) { if ( pageTitles.hasOwnProperty(tabId) === false ) {
selectedTabId = pageDetails[0]; continue;
}
for ( var tabId in pageDetails ) {
if ( pageDetails.hasOwnProperty(tabId) ) {
onTabReceived(tabId, pageDetails[tabId]);
} }
option = uDom('<option>').text(pageTitles[tabId])
.prop('value', tabId);
if ( tabId === selectedTabId ) {
option.prop('selected', true);
}
select.append(option);
} }
// This must be done after inserting all option tags, or else Firefox
// will refuse values which do not exist yet.
select.prop('value', selectedTabId);
selectPage(); selectPage();
}; };
messager.send({ what: 'getPageDetails' }, onDataReceived); messager.send({ what: 'getPageDetails' }, onDataReceived);