Closed Bug 544047 Opened 13 years ago Closed 13 years ago
Places drop indicators are not dismissed on drop (revise code to get rid of ns
Drag And Drop in Places)
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:126.96.36.199pre) Gecko/20100203 Firefox/3.6.2 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20100202 Minefield/3.7a1pre ID:20100202053418 After drop a link onto Bookmarks menu dropdown, menupopup-drop-indicator-bar is still present after closing and reopening the menu. After drop a link onto Bookmarks toolbar,toolbar-drop-indicator is still present. See https://bugzilla.mozilla.org/show_bug.cgi?id=541520#c21 Reproducible: Always Steps to Reproduce: 1.Drag a link onto Bookmarks menupoup 2. 3. OR 1.Drag a link onto Bookmarks toolbar 2. 3. Actual Results: drop-indicator-bar/toolbar-drop-indicator is still present. Expected Results: drop-indicator-bar/toolbar-drop-indicator should disappear.
regression window: works: http://hg.mozilla.org/mozilla-central/rev/f57b95afb57e Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20100201 Minefield/3.7a1pre ID:20100201043011 broken: http://hg.mozilla.org/mozilla-central/rev/6e3003aeea75 Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a1pre) Gecko/20100201 Minefield/3.7a1pre ID:20100201085418 pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=f57b95afb57e&tochange=6e3003aeea75 edit: candidate regress bug: Bug 541520 - dragleave fired at successful drop
Summary: After drop a link, menupopup-drop-indicator-bar/toolbar-drop-indicator is still present after closing and reopening. → After drop a link, menupopup-drop-indicator-bar/toolbar-drop-indicator is still present
Now that dragleave does not fire on successful drop, I suspect that the bookmarks code just needs to do what the dragleave listener does on drop as well.
confirmed, places d&d code still needs a cleanup, hope to find some time in near future.
Status: UNCONFIRMED → NEW
Ever confirmed: true
let's see if i can bring back our d&d code in the 20th century.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Ugh there are actually 2 regression here. This one causes the drop indicator to not disappear, and i've fixed it locally. The other one is making it not appear. I've reduced the regrange between 9 and 13 of Jan, still working on a better regrange.
the regrange is http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=120667a01fd2&tochange=a43e2f7eda8f in this range there are a tracemonkey merge and Roc's big rewrite. Excluding the TM merge (for now) the most plausible bugs are bug 526394 or bug 529670.
attaching current patch to avoid losing work. i'm actually investigating the other regression, filed as bug 545049.
Summary: After drop a link, menupopup-drop-indicator-bar/toolbar-drop-indicator is still present → Places drop indicators are not dismissed on drop (revise code to get rid of nsDragAndDrop in Places)
afaict, this is working pretty well. I can't tell this is perfect, we could still move helpers and make code more elegant, but it's out of the scope of this bug and i want to avoid regressions as far as i can. This is a nice first step killing all nsDragAndDrop dependencies, after some baking time on nightlies we can further improve the code.
PS: the move of nsDragAndDrop from PlacesOverlay.xuk to browser.xul looks correct to me (since Places is no more depending on it while browser.xul depends on it), but actually does not bring real improvements, since it's just a move. So i could avoid that change and just wait to be able to get rid of the inclusion completely.
Attachment #425985 - Flags: review?(mano) → review+
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
OS: Windows 7 → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a2
Notice there is still bug 545049 that is causing the drop indicator to not appear sometimes, but it is due to some kind of layout regression.
You need to log in before you can comment on or make changes to this bug.