Closed
Bug 458696
Opened 16 years ago
Closed 16 years ago
Dropping a tab on Tabbar causes reload the page
Categories
(Firefox :: Tabbed Browser, defect)
Firefox
Tabbed Browser
Tracking
()
VERIFIED
FIXED
Firefox 3.1b1
People
(Reporter: hidenosuke, Unassigned)
References
Details
(Keywords: regression, verified1.9.1, Whiteboard: [fixed by bug 458070])
Bug 458048 has been fixed then we can drag and drop a tab again. By the way dragging and dropping a tab cauases reloading the page. This seems regression. Steps to reproduce: 1. Open two tabs or more. 2. Dragging a tab and drop into any places on the tabbar. Actual result: Reloading the page. Expeected result: Not reload the page. I observed with 20081006 Linux and Windows build.
Comment 1•16 years ago
|
||
Confirmed also on Vista HP SP1
Updated•16 years ago
|
Updated•16 years ago
|
Keywords: regression
Comment 2•16 years ago
|
||
one line fix..... add aEvent.preventDefault() to _onDrop method <method name="_onDrop"> <parameter name="aEvent"/> <body> <![CDATA[ + aEvent.preventDefault(); var dt = aEvent.dataTransfer; var dropEffect = dt.dropEffect; var draggedTab; if (dropEffect != "link") { // copy or move draggedTab = dt.mozGetDataAt("application/x-moz-node", 0); NS_ASSERT(draggedTab && draggedTab.localName == "tab", "copy or move action without a tab"); }
Comment 3•16 years ago
|
||
I'm sure I've seen a duplicate of this bug somewhere.
Comment 4•16 years ago
|
||
(In reply to comment #3) > I'm sure I've seen a duplicate of this bug somewhere. Sounds similar to bug 458070
Comment 5•16 years ago
|
||
Given that one of the things we want people to test is the new tab d&d, and that it sounds like this bug makes it not happen at all, should this be blocking beta?
Comment 7•16 years ago
|
||
Landed patch in bug 458070: http://hg.mozilla.org/mozilla-central/rev/060b7352c2ad
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Flags: blocking-firefox3.1?
Hardware: PC → All
Whiteboard: [fixed by bug 458070]
Reporter | ||
Comment 9•16 years ago
|
||
I confired to fix the problem with today's Linux build. Thanks!
Status: RESOLVED → VERIFIED
Comment 10•16 years ago
|
||
Verified fix on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b1pre) Gecko/20081007 Minefield/3.1b1pre
Updated•16 years ago
|
Target Milestone: --- → Firefox 3.1b1
Updated•16 years ago
|
Keywords: verified1.9.1
You need to log in
before you can comment on or make changes to this bug.
Description
•