Closed
Bug 134620
Opened 23 years ago
Closed 22 years ago
Can't add bookmark folder from personal toolbar
Categories
(SeaMonkey :: Bookmarks & History, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: pawel.worach, Assigned: p_ch)
References
Details
(Keywords: regression, useless-UI, Whiteboard: [adt2])
Attachments
(1 file)
(deleted),
patch
|
caillon
:
review+
bugzilla
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
Steps to reproduce:
1) right click on the "Personal Toolbar"
2) click on "New Folder..." in the context menu
3) nothing happens
This appears in the JS console:
Error: this.getBestItem is not a function
Source File: chrome://communicator/content/bookmarks/bookmarksOverlay.js
Line: 384
Comment 1•23 years ago
|
||
works for me. win98 build 2002032803.
please, always add your build id with every bug that you submit.
Reporter | ||
Comment 2•23 years ago
|
||
ops, missed that. buildid: 2002040111
Reporter | ||
Comment 3•23 years ago
|
||
same problem om win32 nigtly mozilla-win32.zip from 01-Apr-2002 11:58
buildid: 2002040103
js console:
Error: this.getBestItem is not a function
Source File: chrome://communicator/content/bookmarks/bookmarksOverlay.js
Line: 384
Comment 4•23 years ago
|
||
Confirming on 2002040103 on WinXP.
OS -> All
Severity: normal → major
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression
OS: Linux → All
I have a fix for this bug.
I have a fix for bug 138523, which also fixes this bug. I'm trying to figure
out how to check my fix in. I'm having trouble creating a patch diff on Win
2000, any help with that would be appreciated.
Comment 7•23 years ago
|
||
*** Bug 135336 has been marked as a duplicate of this bug. ***
have you tried using windiff?
Alternatively, download a decent text editor like www.textpad.com which produces
document diffs
Comment 9•23 years ago
|
||
*** Bug 140872 has been marked as a duplicate of this bug. ***
Comment 10•23 years ago
|
||
I posted a fix patch for this bug. See bug 138523.
Comment 11•23 years ago
|
||
*** Bug 141185 has been marked as a duplicate of this bug. ***
Comment 12•23 years ago
|
||
*** Bug 142761 has been marked as a duplicate of this bug. ***
Comment 13•23 years ago
|
||
*** Bug 143495 has been marked as a duplicate of this bug. ***
Comment 14•23 years ago
|
||
Looks like a dup of bug 138523 by bug 143962.
Comment 15•22 years ago
|
||
This bug appears all over. I've found at least three instances [<A
HREF="http://bugzilla.mozilla.org/show_bug.cgi?id=138523" TITLE="Personal
Toolbar context menu for the empty area does not work"></A>, <A
HREF="http://bugzilla.mozilla.org/show_bug.cgi?id=146692" TITLE="personal
toolbar folder doesn't work">bug 146692</A> and <A
HREF="http://bugzilla.mozilla.org/show_bug.cgi?id=147249" TITLE="New Folder...
menuitem in personal toolbar context menu is broken">bug 147249</A>]. I don't
know how to set dup's, but these are all the same - 138523 seems to be the most
all-encompassing though.
-bZj
Comment 16•22 years ago
|
||
*** Bug 150000 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 17•22 years ago
|
||
*** Bug 147249 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•22 years ago
|
Keywords: useless-UI
To Nav triange team: we should at least remove the non-working UI, doing so
should be trivial and safe
Keywords: nsbeta1
this one is a bit worse then just not working. If you have the bookmarks sidebar
open and do "New Folder" in the personal toolbar you end up creating the folder
at the position currently selected in the sidebar.
Comment 20•22 years ago
|
||
Nav triage team: nsbeta1+/adt2
Assignee | ||
Comment 22•22 years ago
|
||
getBestItem is a remnant from bookmarksTree.js
This file should be removed (please, please r/rs bug 142236) along with
bookmarksDD.js (bug 140414)
r/sr, please.
Assignee | ||
Updated•22 years ago
|
Status: NEW → ASSIGNED
Comment 23•22 years ago
|
||
Comment on attachment 93628 [details] [diff] [review]
Patch v1.0
sr=blake
Attachment #93628 -
Flags: superreview+
Comment 24•22 years ago
|
||
Comment on attachment 93628 [details] [diff] [review]
Patch v1.0
r=caillon
Attachment #93628 -
Flags: review+
Comment 26•22 years ago
|
||
Comment on attachment 93628 [details] [diff] [review]
Patch v1.0
a=asa (on behalf of drivers) for checkin to 1.1. This will need to land on the
1.1 branch as soon as it is created.
Attachment #93628 -
Flags: approval+
Assignee | ||
Comment 27•22 years ago
|
||
checked in by timeless on trunk and the m1.1final branch
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 28•22 years ago
|
||
*** Bug 161114 has been marked as a duplicate of this bug. ***
Comment 29•22 years ago
|
||
*** Bug 162111 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 30•22 years ago
|
||
*** Bug 163115 has been marked as a duplicate of this bug. ***
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•