Currently the toolbar item gets disabled, but the menu item stays enabled.
Created attachment 226020 [details] [diff] [review] Patch Note that 90% of this is cosmetic cleanup. The meat of the patch is if (action == @selector(doReload:)) return (![[browserController getBrowserWrapper] isBusy] && ![browserController bookmarkManagerIsVisible]);
OK. The real problem here is bug 304061 (rather than disable the menu item, enable the toolbar item), and the fact that reloading while a page is in the "Looking up http://foo.bar" stage does Bad Things (don't know if that's a filed bug yet). INVALID.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.