Closed
Bug 143031
Opened 22 years ago
Closed 22 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)
1.48 KB,
patch
|
Details | Diff | Splinter Review | |
768 bytes,
patch
|
bugs
:
review+
blizzard
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
20.08 KB,
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•22 years ago
|
||
Sounds like bug 143029 (reported nine minutes earlier...)?
Reporter | ||
Comment 2•22 years ago
|
||
*** Bug 143029 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 3•22 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•22 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•22 years ago
|
||
(builds from the branch)
Comment 10•22 years ago
|
||
*** Bug 143009 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 11•22 years ago
|
||
Patch that makes sure that drags don't start on unix.
Comment 12•22 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•22 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•22 years ago
|
||
*** Bug 143227 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 15•22 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•22 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•22 years ago
|
||
Pierre, does bug 139471 patch fix this problem?
Assignee | ||
Comment 18•22 years ago
|
||
*** Bug 143247 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 19•22 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•22 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•22 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•22 years ago
|
||
Comment on attachment 82916 [details] [diff] [review] patch sr=blizzard
Attachment #82916 -
Flags: superreview+
Comment 23•22 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•22 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•22 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•22 years ago
|
||
sr=jag for rc2 (not trunk).
Comment 27•22 years ago
|
||
Comment on attachment 82916 [details] [diff] [review] patch r=ben@netscape.com
Attachment #82916 -
Flags: review+
Comment 28•22 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•22 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•22 years ago
|
||
*** Bug 143439 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 31•22 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•22 years ago
|
||
*** Bug 143639 has been marked as a duplicate of this bug. ***
Comment 33•22 years ago
|
||
BUILD ID: 2002051010 I've got two crashes after clicking on the bookmark link at the personal toolbar: TB6168782Y TB6169524K
Comment 34•22 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•22 years ago
|
||
Comment 36•22 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•22 years ago
|
||
*** Bug 143885 has been marked as a duplicate of this bug. ***
Comment 38•22 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•22 years ago
|
||
*** Bug 144046 has been marked as a duplicate of this bug. ***
Comment 40•22 years ago
|
||
*** Bug 144096 has been marked as a duplicate of this bug. ***
Comment 41•22 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•22 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•22 years ago
|
||
*** Bug 144267 has been marked as a duplicate of this bug. ***
Comment 44•22 years ago
|
||
*** Bug 143785 has been marked as a duplicate of this bug. ***
Comment 45•22 years ago
|
||
*** Bug 144251 has been marked as a duplicate of this bug. ***
Comment 46•22 years ago
|
||
*** Bug 143479 has been marked as a duplicate of this bug. ***
Comment 47•22 years ago
|
||
*** Bug 144973 has been marked as a duplicate of this bug. ***
Comment 48•22 years ago
|
||
*** Bug 145314 has been marked as a duplicate of this bug. ***
Comment 49•22 years ago
|
||
I wonder if http://bugzilla.mozilla.org/show_bug.cgi?id=127839#c6 is related?
Comment 50•22 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•22 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•22 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•22 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•22 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•22 years ago
|
||
*** Bug 146103 has been marked as a duplicate of this bug. ***
Comment 56•22 years ago
|
||
*** Bug 146254 has been marked as a duplicate of this bug. ***
Comment 57•22 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•22 years ago
|
||
Comment 51 checked in Win98SE. Fixes my win problem. Also thanks for extended support. pi
Comment 59•22 years ago
|
||
just thought I'd report that the bug still exists in build 2002052307
Assignee | ||
Comment 60•22 years ago
|
||
*** Bug 146501 has been marked as a duplicate of this bug. ***
Comment 61•22 years ago
|
||
Same problem on Sun/Solaris with any nightly build most recent than 20020502. It makes Mozilla unusable !
Comment 62•22 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•22 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•22 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•22 years ago
|
||
*** Bug 146759 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•22 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•22 years ago
|
||
*** Bug 147700 has been marked as a duplicate of this bug. ***
Comment 67•22 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•22 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•22 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•22 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•22 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: 22 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 72•22 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•22 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•22 years ago
|
||
*** Bug 148325 has been marked as a duplicate of this bug. ***
Comment 75•22 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
•