The default bug view has changed. See this FAQ.

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

RESOLVED FIXED in seamonkey2.0a1

Status

SeaMonkey
Bookmarks & History
RESOLVED FIXED
15 years ago
9 years ago

People

(Reporter: Christophe Thalet, Assigned: sgautherie)

Tracking

({pp})

Trunk
seamonkey2.0a1
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

15 years ago
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
(Reporter)

Comment 1

15 years ago
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
<http://www.mozilla.org/quality/help/bugzilla-helper.html> to report bugs.
->bookmarks, pch
Assignee: Matti → pierrechanial
Blocks: 133604
Severity: major → normal
Status: UNCONFIRMED → NEW
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

Comment 3

15 years ago
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

Comment 4

13 years ago
http://lxr.mozilla.org/seamonkey/source/xpfe/components/bookmarks/resources/bookmarksMenu.js#334
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")
    return;
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.

Comment 6

11 years ago
Reassigning as per Bug #32644
Assignee: p_ch → nobody
(Assignee)

Updated

9 years ago
Blocks: 296651
(Assignee)

Comment 7

9 years ago
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).
(Assignee)

Comment 8

9 years ago
(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 ?
(Assignee)

Comment 9

9 years ago
Reminder:
This bug is about bypassing/removing, on Linux and/or MacOSX,
<http://bonsai.mozilla.org/cvsblame.cgi?file=/mozilla/suite/common/bookmarks/bookmarksMenu.js&rev=1.32&mark=366-372#357>

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.
}}
(Assignee)

Updated

9 years ago
QA Contact: imajes-qa → bookmarks
(Assignee)

Comment 10

9 years ago
Created attachment 309760 [details] [diff] [review]
(Av1) <bookmarksMenu.js>
[Checkin: Comment 14]

(I use Windows only.)
Andrew, Stefan, can you test this patch on Linux and MacOSX ?
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #309760 - Flags: superreview?(neil)
Attachment #309760 - Flags: review?(stefanh)
Attachment #309760 - Flags: review?(ajschult)

Comment 11

9 years ago
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 12

9 years ago
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+

Updated

9 years ago
Attachment #309760 - Flags: superreview?(neil) → superreview+

Updated

9 years ago
Target Milestone: --- → seamonkey2.0alpha
(Assignee)

Comment 13

9 years ago
(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

Comment 14

9 years ago
landed attachment 309760 [details] [diff] [review].  FIXED?
Keywords: checkin-needed
(Assignee)

Updated

9 years ago
Attachment #309760 - Attachment description: (Av1) <bookmarksMenu.js> → (Av1) <bookmarksMenu.js> [Checkin: Comment 14]
(Assignee)

Comment 15

9 years ago
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
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.