Closed
Bug 143031
Opened 23 years ago
Closed 23 years ago
opening the bookmarks item on the personal toolbar starts a drag on linux and Sun
Categories
(Core :: DOM: Copy & Paste and Drag & Drop, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: blizzard, Assigned: p_ch)
References
Details
(Keywords: qawanted, Whiteboard: qawanted for *nix and other platforms)
Attachments
(3 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
bugs
:
review+
blizzard
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
(deleted),
image/jpeg
|
Details |
If I open the 'Bookmarks' item on my personal toolbar and start to move the
mouse down over the items in that new popup a drag is started. Oops! You also
can't get rid of the drag as near as I can tell.
Comment 1•23 years ago
|
||
Sounds like bug 143029 (reported nine minutes earlier...)?
Reporter | ||
Comment 2•23 years ago
|
||
*** Bug 143029 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 3•23 years ago
|
||
Yeah, but this has more people on it and it's already marked as dup.
*** Bug 142821 has been marked as a duplicate of this bug. ***
Seeing the problem with Solaris 8/9/etc...
This is really irritating since it makes the browser unusable
once the drop marker is posted. It effectively has permanent
focus from the window manager and you can barely click on any
other window to even kill the mozilla process.
This is 100% reproduceable in the nightly CVS builds that we
do within Sun.
This problem only started happening since 20020505 (4th or 5th
May) builds onwards, so we should easily be able to track it
down.
Forgot to add, that this doesn't appear to a window manager
bug since i can reproduce it on the WindowMaker, CDE, and
GNOME (sawfish) window managers.
Assignee | ||
Comment 8•23 years ago
|
||
This is unrelated with bug 96504 since build 2002-05-06-15 has the patch and is ok.
The regression occurred between:
build 2002-05-07-09
and build 2002-05-08-10
This leads to the following checkins:
http://bonsai.mozilla.org/cvsquery.cgi?module=SeaMonkeyAll&branch=MOZILLA_1_0_0_BRANCH&hours=2&date=explicit&mindate=05%2F07%2F2002+09%3A00&maxdate=05%2F08%2F2002+10%3A00
Assignee | ||
Comment 9•23 years ago
|
||
(builds from the branch)
Comment 10•23 years ago
|
||
*** Bug 143009 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 11•23 years ago
|
||
Patch that makes sure that drags don't start on unix.
Comment 12•23 years ago
|
||
I can confirm that patch 82887 (attached) fixes the problem on Solaris (probably
all platforms since there is no architecture dependent code here). Ok
to close once it is integrated in the cvs tree.
This may be a different issue, but we also appear to have lost the
ability to drag a URL from the location field and drop onto the personal
toolbar. I'm sure this used to work before. Can anyone confirm that this
behavior has suddenly disappeared ? Actually this is not related to the
posted patch since the drag appears to work, but the toolbar is not accepting
drops anymore...
Comment 13•23 years ago
|
||
Actually, this is not entirely fixed.
Try dragging from the location entry icon in the Navigation
toolbar to the "Bookmarks" pulldown on the personal toolbar
and the drop sticks as before, rendering the browser unuseable.
Comment 14•23 years ago
|
||
*** Bug 143227 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 15•23 years ago
|
||
Chris, your patch "fixes" the draggesture problem because it throws an
javascript exception: event is not passed in CheckDragGesture.
Also, nsDragAndDrop.startDrag was called for the X11 platform, therefore, it
was preventing also drag for the other platforms.
I attached another patch for the branch that prevents draggestures in the
bookmark button from bubbling to the toolbar. Drag and drop is still possible
in the PT.
This patch, like the previous one, only fixes the symptoms, not the real bug.
Dave... bug 96504 is fixed on the branch only!!!
Reporter | ||
Comment 16•23 years ago
|
||
Seems good enough to me, although I have to defend my patch saying that I never
saw the drag happen on my platform and I never saw an exception. But maybe I
just missed it.
Anyway, someone with stronger xul-fu should review that patch.
Comment 17•23 years ago
|
||
Pierre, does bug 139471 patch fix this problem?
Assignee | ||
Comment 18•23 years ago
|
||
*** Bug 143247 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 19•23 years ago
|
||
Asa: sorry for the delay, I had to update my tree and figure out how to work
around bug 142853 (grr...).
I tried to confirm this bug with and without my patch on the trunk, and although
I would have bet that my patch does not fix this weirdness, I had to convince
myself that it does, indeed.
Don't ask me why! :)
I'll update the patch in bug 142853 per timeless review, right now.
I am pulling the branch, now. I will test if the bug disappears, too. But it
will take a lot of time.
Keywords: nsbeta1
Comment 20•23 years ago
|
||
Pierre, I think we're going to go with another bandaid here and try for your
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=82916 patch.
Assignee | ||
Comment 21•23 years ago
|
||
Asa, I agree we should not put the patch in bug 139471 for RC2. I am just
checking what happens in the branch with this patch, looking forward at the next
branch release.
Reporter | ||
Comment 22•23 years ago
|
||
Comment on attachment 82916 [details] [diff] [review]
patch
sr=blizzard
Attachment #82916 -
Flags: superreview+
Comment 23•23 years ago
|
||
Piere, how about removing the check for X11? There's no reason to drag the
bookmarks menu around, and it has the added benefit that you can click-drag the
menu open (so you press the left mouse button on the menu, drag your mouse down
and release it on the bookmark you want).
Comment 24•23 years ago
|
||
Since there seems to be some confusion about what I'm really asking here, let me
make it a little more clear:
Can/should we call preventBubble for all platforms instead of just for X11?
Reporter | ||
Comment 25•23 years ago
|
||
I don't know if we want to remove that functionality or not, but we know that it
breaks linux pretty badly. Can we talk about removing the functionality some
other time and just get this in for RC2?
Comment 26•23 years ago
|
||
sr=jag for rc2 (not trunk).
Comment 27•23 years ago
|
||
Attachment #82916 -
Flags: review+
Comment 28•23 years ago
|
||
Comment on attachment 82916 [details] [diff] [review]
patch
a=rjesup@wgate.com for drivers as per Asa
Attachment #82916 -
Flags: approval+
Comment 29•23 years ago
|
||
Fix checked into branch as per Asa's request. Not closing because there are
still other issues to discuss/fix here for trunk.
Keywords: fixed1.0.0
Comment 30•23 years ago
|
||
*** Bug 143439 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 31•23 years ago
|
||
Jag: your approach is more correct, we must be careful at event bubbling
thought, since menu/menuitem startdrag bubble throught it.
I have implemented your approach in bug 139471.
However, the user will not notice the change, since container drag is prevented
on all platforms. But when we will relax it, we will hit this issue again.
Assignee: blaker → pierrechanial
Comment 32•23 years ago
|
||
*** Bug 143639 has been marked as a duplicate of this bug. ***
Comment 33•23 years ago
|
||
BUILD ID: 2002051010
I've got two crashes after clicking on the bookmark link at the personal toolbar:
TB6168782Y
TB6169524K
Comment 34•23 years ago
|
||
Not sure if this is the same bug but please Confirm or not Confirm if that is a
different one. Since today 11th May Whenever I enter the Bookmarks menu on the
Personal Toolbar I completley loose the focus, the mouse pointer is changed (as
described) into the drag mouse pointer. If I click additional then on Manage
Bookmarks the Bookmark Menu doesn't go away. This is even so bad that I have to
kill mozilla on the console as the whole focus for the complete WindowManager is
up to Mozilla.
This happened on both Gnome and in IceWM.
I attached a screenshot (NOTE: I had to repaint the Mousepointer in GIMP because
it was hidden by the screenshot program)
This does *not* occur when I go to the Bookmarks Main Menu
Comment 35•23 years ago
|
||
Comment 36•23 years ago
|
||
Carsten, yes, this sounds like the same bug. Also related is bug 96504,
which I believe has been "fixed" by disabling the facility on Linux.
How the original feature ever landed I don't understand :(
Comment 37•23 years ago
|
||
*** Bug 143885 has been marked as a duplicate of this bug. ***
Comment 38•23 years ago
|
||
If you are trapped inside of the bookmarks menu, you can get back to the normal
by pressing ESC twice.
This prevents from having to kill mozilla from the console.
Comment 39•23 years ago
|
||
*** Bug 144046 has been marked as a duplicate of this bug. ***
Comment 40•23 years ago
|
||
*** Bug 144096 has been marked as a duplicate of this bug. ***
Comment 41•23 years ago
|
||
I don't see why this bug is not marked a blocker, since it makes Mozilla
completely unusable. So the fix should be checked into the trunk, even if it
will be improved later (as comment 29 says).
pi
Comment 42•23 years ago
|
||
Just to let you all know that people still care about this bug :-) And (more
importantly) to help in weeding out the unneeded duplicates. Here are my
candidates to be marked as a duplicate of this bug:
bug 143785
bug 144251
bug 144267
bug 143479(?)
Comment 43•23 years ago
|
||
*** Bug 144267 has been marked as a duplicate of this bug. ***
Comment 44•23 years ago
|
||
*** Bug 143785 has been marked as a duplicate of this bug. ***
Comment 45•23 years ago
|
||
*** Bug 144251 has been marked as a duplicate of this bug. ***
Comment 46•23 years ago
|
||
*** Bug 143479 has been marked as a duplicate of this bug. ***
Comment 47•23 years ago
|
||
*** Bug 144973 has been marked as a duplicate of this bug. ***
Comment 48•23 years ago
|
||
*** Bug 145314 has been marked as a duplicate of this bug. ***
Comment 49•23 years ago
|
||
I wonder if http://bugzilla.mozilla.org/show_bug.cgi?id=127839#c6 is related?
Comment 50•23 years ago
|
||
Applying the patch to the trunk sources, i.e. Mozilla/5.0 (X11; U; Linux i686;
en-US; rv:1.0.0+) Gecko/20020522, solves the problem in Linux. I still think the
trunk should not stay unpatched any longer. Or did the problem vanish for other
reasons? There are no recent dupes to my surprise.
An observation from Windows trunk builds (unpatched): I use to press "Bookmars"
and move the mouse down keeping the left button pressed. This causes a problem,
Mozilla behaves as dragging a bookmark. Is this the same bug?
pi
Assignee | ||
Comment 51•23 years ago
|
||
mrmazda: the current bug is unrelated. This one is due to a mouse up event (that
should prevent DND) not fired when a menupopup opens while the other one seems
to be caused by a particulat bookmark file.
boris: a pach exists in bug 139471 that masks the problem on linux and fixes the
issue you raised on Windows. Fyi, these issues are unrelated. I would be
grateful is you would have time to test (on linux and windows) the last version
of the patch in bug 139471 and bug 145350 and report your findings to accelerate
the review process.
Depends on: 139471
Comment 52•23 years ago
|
||
I reported this about a week ago:
http://bugzilla.mozilla.org/show_bug.cgi?id=144274
I think it may be duplicate, and the comments say so but for some reason
it has not been marked as such. This is on Mac OS X.
Comment 53•23 years ago
|
||
Boris, try making this change on your build from the trunk:
--- xpfe/browser/resources/content/navigator.xul Thu May 9 16:13:43 2002
+++ xpfe/browser/resources/content/navigator.xul Thu May 9 16:14:05 2002
@@ -258,6 +258,7 @@
datasources="rdf:bookmarks rdf:files rdf:localsearch
rdf:internetsearch"
ref="NC:BookmarksRoot" container="true"
flags="dont-test-empty"
oncommand="OpenBookmarkURL(event.target,document.getElementById('BookmarksMenu').database)"
+ ondraggesture="event.preventBubble();"
ondragover="nsDragAndDrop.dragOver(event, folderObserver);"
template="bookmarksMenuTemplate">
<menupopup onpopuphiding="gDidOpen = false;"
Comment 54•23 years ago
|
||
I've accidently found a way to unwedge the menu (under Linux). This doesn't fix
the bug, but lets you get rid of the menu without killing mozilla or quitting X.
Press in 'ALT-<LETTER>' where LETTER is the first letter of some submenu of the
bookmark menu. This causes the submenu to open... Then if you futz with the
mouse a little. (I don't recall the exact fuddlings.) you can get the bookmark
menu to close, unwedging it. Then, you can continue as normal.
This isn't a fix, but it at least it unwedges mozilla to make it easier to
continue development/using it. (Also, can someone mark 146103 as a dup of this?)
Comment 55•23 years ago
|
||
*** Bug 146103 has been marked as a duplicate of this bug. ***
Comment 56•23 years ago
|
||
*** Bug 146254 has been marked as a duplicate of this bug. ***
Comment 57•23 years ago
|
||
jag, your fix works for my windows problem. Thanks also for the personal support
by e-mail.
Next I gonna check out Pierre's patches. As he pointed out this (win) problem is
unrelated, but his patch will take care of both items. I'll be back on this.
pi
Comment 58•23 years ago
|
||
Comment 51 checked in Win98SE. Fixes my win problem. Also thanks for extended
support.
pi
Comment 59•23 years ago
|
||
just thought I'd report that the bug still exists in build 2002052307
Assignee | ||
Comment 60•23 years ago
|
||
*** Bug 146501 has been marked as a duplicate of this bug. ***
Comment 61•23 years ago
|
||
Same problem on Sun/Solaris with any nightly build most recent than 20020502. It
makes Mozilla unusable !
Comment 62•23 years ago
|
||
Unusable? whilst its a MAJOR pain in the rear if you forget and go for the
toolbar button there is a workaround... Use the bookmarks menu item instead.
This bug only shows up for me (on solaris 8 sparc platform) with the toolbar button.
Comment 63•23 years ago
|
||
And in any case you should know that the Bookmarks item is removable. In the
preferences/navigator menu the lower third of the page has a radio button to
deselect it.
Comment 64•23 years ago
|
||
The patches named in comment 51 work under Mozilla/5.0 (X11; U; Linux i686;
en-US; rv:1.0.0+) Gecko/20020527.
pi
Assignee | ||
Comment 65•23 years ago
|
||
*** Bug 146759 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•23 years ago
|
Summary: opening the bookmarks item on the personal toolbar starts a drag on linux → opening the bookmarks item on the personal toolbar starts a drag on linux and Sun
Comment 66•23 years ago
|
||
*** Bug 147700 has been marked as a duplicate of this bug. ***
Comment 67•23 years ago
|
||
Having this problem on Linux with today's build. This problem did not appear in
the RC3 release for me, but is consistantly reproduceable with today's build.
Comment 68•23 years ago
|
||
I'm also still seeing this problem, with
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.0+) Gecko/20020530
Comment 69•23 years ago
|
||
This problem disappeared with build 2002052921 for me. The build I was using
yesterday when I noted the problem was 2002052908.
Comment 70•23 years ago
|
||
I'm still seeing it on this morning's trunk build (2002053008).
I got so tired of this bug I removed the Bookmarks menu from the toolbar a
couple of days ago (Edit -> Preferences -> Navigator, uncheck Bookmarks in the
"Select the buttons you want to see in the toolbars").
This gives me more room in my personal toolbar folder and I can still get to all
of my bookmarks just fine through the Menu Bar.
I actually consider this bug a feature now... ;)
Assignee | ||
Comment 71•23 years ago
|
||
fixed on trunk by the checkin of bug 139471. Thought, the fact that mouse up are
eaten when any menupopup opens on mouse down still remains. The fix only mask it.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 72•23 years ago
|
||
qawanted for platforms other than linux, mac and windows.
If the problem is still not fixed on a platform, then open a new bug report and
assign it to me.
I strongly suspect that for SUN, the problem still exists, but it's a one liner fix.
Keywords: qawanted
Whiteboard: qawanted for *nix and other platforms
Comment 73•23 years ago
|
||
Pierre:
Is there a bug for the item you tell is still not resolved in comment #71? If
there is, please give us a bug number, if there isn't, please file one - it
seems you know better about that issue than e.g. me...
Comment 74•23 years ago
|
||
*** Bug 148325 has been marked as a duplicate of this bug. ***
Comment 75•23 years ago
|
||
Confirm appears fixed on Solaris with nightly build 2002053023.
Previously expected behaviour is restored with both
<click-and-hold><slide-to-bookmark><release> and
<click-and-release><second-click-on-bookmark> resulting in opening the
bookmarked page.
Comment 76•22 years ago
|
||
Verified fixed linux trunk build 2002060308 and branch build 2002060309
Status: RESOLVED → VERIFIED
Keywords: fixed1.0.0 → verified1.0.0
Assignee | ||
Comment 77•22 years ago
|
||
*** Bug 148028 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•