From f7ce369c60d9ea1132bb1fc5f148debb74338913 Mon Sep 17 00:00:00 2001 From: "mgalli%geckonnection.com" Date: Wed, 7 Dec 2005 22:21:52 +0000 Subject: [PATCH] Code cleanup and sync button fix. Using getBrowser().webNavigation instead. --- minimo/chrome/content/minimo.js | 34 ++++++++++++++++++++------------- 1 file changed, 21 insertions(+), 13 deletions(-) diff --git a/minimo/chrome/content/minimo.js b/minimo/chrome/content/minimo.js index e7622f6fdc6..0156dea57b8 100755 --- a/minimo/chrome/content/minimo.js +++ b/minimo/chrome/content/minimo.js @@ -180,20 +180,9 @@ nsBrowserStatusHandler.prototype = if (domWindow == domWindow.top) { this.urlBar.value = aLocation.spec; } - - if(aWebProgress.canGoBack) { - document.getElementById("back-button").disabled=false; - } else { - document.getElementById("back-button").disabled=true; - } - - if(aWebProgress.canGoForward) { - document.getElementById("forward-button").disabled=false; - } else { - document.getElementById("forward-button").disabled=true; - } - } + + BrowserUpdateBackForwardState(); BrowserUpdateFeeds(); }, @@ -410,6 +399,25 @@ function BrowserUpdateFeeds() { } } +/* + * For now, this updates via DOM the top menu. Context menu should be here as well. + */ +function BrowserUpdateBackForwardState() { + + if(getBrowser().webNavigation.canGoBack) { + document.getElementById("back-button").setAttribute("disabled","false"); + } else { + document.getElementById("back-button").setAttribute("disabled","true"); + } + + if(getBrowser().webNavigation.canGoForward) { + document.getElementById("forward-button").setAttribute("disabled","false"); + } else { + document.getElementById("forward-button").setAttribute("disabled","true"); + } + +} + function findChildShell(aDocument, aDocShell, aSoughtURI) { aDocShell.QueryInterface(Components.interfaces.nsIWebNavigation);