Closed Bug 341881 Opened 19 years ago Closed 19 years ago

Disable "reload" menu item when page is loading

Categories

(Camino Graveyard :: Toolbars & Menus, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED INVALID
Camino1.5

People

(Reporter: froodian, Assigned: froodian)

Details

Attachments

(1 file)

Currently the toolbar item gets disabled, but the menu item stays enabled.
Attached patch PatchSplinter Review
Note that 90% of this is cosmetic cleanup. The meat of the patch is if (action == @selector(doReload:)) return (![[browserController getBrowserWrapper] isBusy] && ![browserController bookmarkManagerIsVisible]);
Attachment #226020 - Flags: review?(bugzilla)
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.
Attachment #226020 - Flags: review?(bugzilla)
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → INVALID
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: