Closed Bug 151336 Opened 18 years ago Closed 12 years ago

[LINUX] can not drag bookmarks from a folder in the personal toolbar


(SeaMonkey :: Bookmarks & History, defect)

Not set


(Not tracked)



(Reporter: christophe.thalet, Assigned: sgautherie)



(Keywords: platform-parity)


(1 file)

to open a page from personal toolbar, you can either click it or
drag it into the browser (or tab) window. OK so far.

But if the bookmark resides in a folder, you can *not* drag it there.

Please fix
additional explication:

the folder is part of the personal toolbar,
and in that folder the bookmark resides
Confirmed linux trunk cvs 2002-07-08, don't see an open dup so marking new. Bug
118141 except for the 'open' part; propably fixed on windows on trunk. Reporter,
always include Mozilla's build id (on the title bar) when reporting bugs,
consider using bugzilla helper
<> to report bugs.
->bookmarks, pch
Assignee: Matti → pierrechanial
Blocks: 133604
Severity: major → normal
Component: Browser-General → Bookmarks
Ever confirmed: true
OS: Windows NT → All
Summary: can drag bookmark from Personal Tb. to open, but *cannot* for bookmarks in a folder → can drag bookmark from Personal Toolbar to open, but *cannot* for bookmarks in a folder
unfortunately, the patch in bug 145439 won't fix this issue.
Keywords: pp
OS: All → Linux
Summary: can drag bookmark from Personal Toolbar to open, but *cannot* for bookmarks in a folder → [LINUX] can not drag bookmarks from a folder in the personal toolbar
explains why this is disabled intentionally:
"on Linux, this causes an X-server crash"

You can easily test this by removing the first condition from
  if (navigator.platform != "Win32" && target.localName != "toolbarbutton")
in the lines below the cited one.
Product: Browser → Seamonkey
I know there was a fork in the bookmark code for Firefox but reverting the
changes and using that feature no crash happens with Mozilla/5.0 (X11; U; Linux
i686; en-US; rv:1.8b2) Gecko/20050604 Firefox/1.0+. I can't test with Seamonkey
cause I don't have a trunk developer build available. If the crashes don't
appear anymore we could reactive this nice feature for Linux. For Firefox I
filed bug 296651.
Reassigning as per Bug #32644
Assignee: p_ch → nobody
Blocks: 296651
From comment 5 and bug 296651 comment 1 (and bug 420341 comment 14),
it seems we could allow Linux too to bypass this test,
(which landed with bug 196756).
(In reply to comment #7)
> it seems we could allow Linux too to bypass this test,

Who could check and actually decide on this matter ?
This bug is about bypassing/removing, on Linux and/or MacOSX,

Linux: see comment 7.

MacOSX: bug 420341 comment 24 reads
{{ Stefan   2008-03-11 09:16:29 PDT
> > doesn't seem to change anything on mac (the folder still doesn't  open).
> Could you check and comment on bug 151336, about "Mac bypassing" that other
> line too ?
> (Would it work ? Would that behavior be wanted ?)

I haven't checked, but it might work. Seems that mac Minefield suport this.
QA Contact: imajes-qa → bookmarks
(I use Windows only.)
Andrew, Stefan, can you test this patch on Linux and MacOSX ?
Assignee: nobody →
Attachment #309760 - Flags: superreview?(neil)
Attachment #309760 - Flags: review?(stefanh)
Attachment #309760 - Flags: review?(ajschult)
Comment on attachment 309760 [details] [diff] [review]
(Av1) <bookmarksMenu.js>
[Checkin: Comment 14]

This makes drag and drop work on Linux, although it closes the menu after drop (as in bug 420341)
Attachment #309760 - Flags: review?(ajschult) → review+
Comment on attachment 309760 [details] [diff] [review]
(Av1) <bookmarksMenu.js>
[Checkin: Comment 14]

Yes, this works except that the menu closes. Note that this doesn't work in mac native menus and I don't think it's a "macish" behaviour. But I don't see anything evil in having it for mac.
Attachment #309760 - Flags: review?(stefanh) → review+
Attachment #309760 - Flags: superreview?(neil) → superreview+
Target Milestone: --- → seamonkey2.0alpha
(Would there be any release note or help to update about the new Linux/MacOSX behaviors from this bug and bug 420341 ?)
Keywords: checkin-needed
Attachment #309760 - Attachment description: (Av1) <bookmarksMenu.js> → (Av1) <bookmarksMenu.js> [Checkin: Comment 14]
R.Fixed, WRT comment 4.

Reopen if comment 0 + comment 1 is not fixed as wanted.
(As I can't test Linux/MacOSX platforms.)

I'll try and look into the other (cross platform) D&D related issues (like tooltip "annoyance" and menu closing after drop) in seperate bugs.
No longer blocks: 296651
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.