Closed
Bug 63698
Opened 24 years ago
Closed 23 years ago
Context menu with empty submenu for personal toolbar items
Categories
(SeaMonkey :: Bookmarks & History, defect, P2)
SeaMonkey
Bookmarks & History
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9
People
(Reporter: bugzilla, Assigned: bugs)
References
Details
(Keywords: regression)
Attachments
(1 file)
Build ID: 12/24 trunk (tip) Steps to Reproduce: (1) Right click on any item in the personal toolbar. Result: you get a context menu with one empty submenu. Alec, someone on IRC said your inclusion of nsTreeUtils.js in bookmarks.xul caused this somehow.
Comment 2•24 years ago
|
||
Comment 3•24 years ago
|
||
the contextual menu was never used anyway, because fillContextMenu() would always verify that the target of the context menu was a tree.. also in this patch is the removal of bookmarks.js - we no longer need it because I have re-factored the bookmarks code. looking for a r/sr, adding ben
Status: NEW → ASSIGNED
Updated•24 years ago
|
Whiteboard: fix in hand
Target Milestone: --- → mozilla0.8
Reporter | ||
Comment 4•24 years ago
|
||
r=blake
Comment 5•24 years ago
|
||
pardon the spam: personal toolbar bugs should go into the Bookmarks component. so, i'm moving the existing ones from xp apps/xp apps: gui to there...
Component: XP Apps: GUI Features → Bookmarks
Updated•24 years ago
|
QA Contact: sairuh → claudius
Comment 8•24 years ago
|
||
carrying over nav triage team comment from bug 64021 in case anyone cares. ------- Additional Comments From knous@netscape.com 2001-01-04 16:11 ------- Netscape Nav triage team: this is a Netscape beta stopper.
Assignee | ||
Comment 9•24 years ago
|
||
hm. I'll be needing to monkey around with this function a bit for bookmarks, as the commands that need to be in the context menus aren't all stored in the bookmarks service. I may define a different mechanism for creating context menus for bookmarks, which will be shared by the bookmarks window, panel and toolbar.
Assignee | ||
Comment 10•24 years ago
|
||
a=ben@netscape.com
Comment 11•24 years ago
|
||
*** Bug 64976 has been marked as a duplicate of this bug. ***
Comment 12•24 years ago
|
||
fix is in.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 13•24 years ago
|
||
Seems like Ben never fixed this in his rewritten bookmarks stuff, I guess it's not too hard to make the same changes as this patch does on the new BM stuff? Adding regression keyword and reopening.
Comment 14•24 years ago
|
||
dammit. reassigning to ben then.
Assignee: alecf → ben
Status: REOPENED → NEW
Comment 15•24 years ago
|
||
WORKSFORME Platform: PC OS: Windows 98 Mozilla Build: 2001022311 drbrain sees them on FreeBSD thus marking FIXED. Scott sees it as well
Status: NEW → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
Comment 16•23 years ago
|
||
I'm still seeing an empty menu on current Linux builds (2001-03-14-08 and a debug build from 2001-03-18). Reopening.
Comment 17•23 years ago
|
||
*** Bug 72445 has been marked as a duplicate of this bug. ***
Comment 18•23 years ago
|
||
clobber milestone, clear status.
Whiteboard: fix in hand
Target Milestone: mozilla0.8 → ---
Comment 19•23 years ago
|
||
Marking nsbeta1+, p2, mozilla0.9
Comment 20•23 years ago
|
||
*** Bug 73585 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 21•23 years ago
|
||
*** Bug 65174 has been marked as a duplicate of this bug. ***
Comment 22•23 years ago
|
||
Verified this behavior with build id 2001041305 output generated on this action: JavaScript error: line 0: fillContextMenu is not defined
Assignee | ||
Comment 23•23 years ago
|
||
This bug is fixed by the patch in 17920.
Status: REOPENED → ASSIGNED
Depends on: 17920
Assignee | ||
Comment 24•23 years ago
|
||
fixed.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago → 23 years ago
Resolution: --- → FIXED
Comment 25•23 years ago
|
||
VERIFIED Fixed with 200104260x builds on all platforms
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•