Closed Bug 702297 Opened 13 years ago Closed 13 years ago

Recently Closed Tabs/Windows in the Go menu are no longer disabled when empty

Categories

(SeaMonkey :: Bookmarks & History, defect)

Product:
SeaMonkey
Component:
Bookmarks & History
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(seamonkey2.5 wontfix, seamonkey2.6- wontfix, seamonkey2.7+ fixed, seamonkey2.8 fixed)

Status:
RESOLVED FIXED
Milestone:
seamonkey2.8
Tracking Flags:
Tracking Status
seamonkey2.5 --- wontfix
seamonkey2.6 - wontfix
seamonkey2.7 + fixed
seamonkey2.8 --- fixed

People

(Reporter: rsx11m.pub, Assigned: ewong)

References

Details

(Keywords: regression)

Attachments

(2 files, 1 obsolete file)

Disable Recently Closed Tabs/Windows in the Go menu when empty. (v2)
(deleted), patch
iannbugzilla
: review+
Callek
: approval-comm-aurora-
Callek
: approval-comm-beta+
Details | Diff | Splinter Review
Removed unneccessary code from original patch.
(deleted), patch
neil
: review+
Details | Diff | Splinter Review
At least since the 2011-10-31 nightly trunk build, the Recently Closed Tabs and Recently Closed Windows menu items remain available even if those lists are empty. Only an empty submenu shows up when selected, no Error Console messages.
I'm seeing this in 2.5 beta 4 already, both on Windows 7 and - to a lesser extent (menu items enabled but no empty submenu) - on Linux as well. Maybe that's what Neil was referring to in bug 599731 comment #14? > The code to enable and disable the menus was unfortunately hidden away at > the end of updateCloseItems. This needs to be moved into a new function so > that it can be called from the go popup.
status-seamonkey2.5: --- → affected
status-seamonkey2.6: --- → affected
OS: Windows 7 → All
Hardware: x86_64 → All
Bug 599731 was pushed Tue Jul 12 06:52:48 2011 -0700; the 20110710 Windows comm-central build does not exhibit the problem whereas the 20110713 build does. Thus, it seems that this bug is indeed the cause of the regression but apparently was neither resolved in bug 675672 nor filed as a separate follow-up bug (which is this one here now). Further information from Ian in bug 599731 comment #15: > The relevant code to be moved starts at > http://mxr.mozilla.org/comm-central/source/suite/browser/navigator.js#1500 > You will need to add a onpopupshowing to the "history-menu" > http://mxr.mozilla.org/comm-central/source/suite/browser/navigatorOverlay.xul#380
Blocks: 599731
Component: UI Design → Bookmarks & History
Keywords: regressionwindow-wanted
QA Contact: ui-design → bookmarks
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #574801 - Flags: review?(iann_bugzilla)
This is now listed as a Known Issue starting with SM 2.5. Please try to get Aurora/Beta approval once you have r+.
status-seamonkey2.8: --- → affected
tracking-seamonkey2.6: --- → ?
tracking-seamonkey2.7: --- → ?
Comment on attachment 574801 [details] [diff] [review] Disable Recently Closed Tabs/Windows in the Go Menu when empty. >+function updateRecentMenuItems() >+{ >+ var browser = getBrowser(); >+ var ss = Components.classes["@mozilla.org/suite/sessionstore;1"] >+ .getService(Components.interfaces.nsISessionStore); You've added the sessionstore service here, but not removed it from earlier in updateCloseItems function. r- for the moment
Attachment #574801 - Flags: review?(iann_bugzilla) → review-
Attachment #574801 - Attachment is obsolete: true
Attachment #576730 - Flags: review?(iann_bugzilla)
Attachment #576730 - Flags: review?(iann_bugzilla) → review+
Pushed to http://hg.mozilla.org/comm-central/rev/1d4e052858df.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
status-seamonkey2.8: affected → fixed
Target Milestone: --- → seamonkey2.8
Comment on attachment 576730 [details] [diff] [review] Disable Recently Closed Tabs/Windows in the Go menu when empty. (v2) > function updateCloseItems() ... >+ updateRecentMenuItems(); Probably didn't need to do this here, just calling it from the Go menu suffices.
I don't know what the cutoff for 2.6 is, but are you going to request approval-aurora/beta, with or without the change Neil suggested in comment #8?
Attachment #576730 - Flags: approval-comm-beta?
Attachment #576730 - Flags: approval-comm-aurora?
v2 of the patch introduced a superfluous line. This patch removes it and the blank line.
Attachment #579986 - Flags: review?(neil)
Attachment #579986 - Flags: review?(neil) → review+
Edmund, can you please push the second patch to c-c. IanN can you please evaluate the risk of both of these patches going into aurora and beta and handle approvals respectively? (Code freeze for 2.6 is late monday)
tracking-seamonkey2.6: ? → +
tracking-seamonkey2.7: ? → +
Comment on attachment 576730 [details] [diff] [review] Disable Recently Closed Tabs/Windows in the Go menu when empty. (v2) Not sure how we missed this for 2.6 cycle, sorry. But we should take this for 2.7
Attachment #576730 - Flags: approval-comm-beta?
Attachment #576730 - Flags: approval-comm-beta+
Attachment #576730 - Flags: approval-comm-aurora?
Attachment #576730 - Flags: approval-comm-aurora-
Keywords: checkin-needed
Whiteboard: c-n for comm-beta
Comment on attachment 576730 [details] [diff] [review] Disable Recently Closed Tabs/Windows in the Go menu when empty. (v2) http://hg.mozilla.org/releases/comm-beta/rev/1430d8e9f084
status-seamonkey2.7: affected → fixed
Keywords: checkin-needed
Whiteboard: c-n for comm-beta
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: