379272 - Dragging and dropping a non-button element leaves it matching :active

Status: NEW

(Core :: Layout, defect)

Description

[Xavier]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-FR; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3 XPCOMViewer/0.9.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-FR; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3 XPCOMViewer/0.9.5

I've set up a drag and drop handler on a label, and used css styles so that the label has a different color when it's ":hover" and ":active"

When I click and hold, it turns to the ":active" defined color, then I move the pointer and drop it anywhere (on the location bar for instance).

The color of the label remains the one defined in the ":active" style (and in DOM inspector, the css rule shows that :active is still taken into account.

Reproducible: Always

Steps to Reproduce:
1. set up and startdrag handler on an element with an :active style
2. Drag and drop the element
3. 
Actual Results:  
The active style remains even when element is no longer selected

Expected Results:  
The element should be reset to its default style

Comment 1

[Steve England [:stevee]]

can you please attach a minimal testcase that shows this problem? ('Add an attachment' above)

Comment 2

[Xavier]

Attachment: Demonstration of the bug. Install, then call URI chrome://test/chrome/test.xul

install the xpi, restart, then call URI chrome://test/chrome/test.xul
Drag the label, drop it, it remains red, as defined in the :active style

(style are declared with a data: protocol, but it is the same with a separate style sheet, I just made it as compact as possible).

Comment 4

[Markus Stange [:mstange]]

Attachment: minimal testcase

Comment 5

[Markus Stange [:mstange]]

I'm not sure what component this bug should be in.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Whatever component owns nsEventStateManager.cpp :-)

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

Bug 593959 should fix attachment 323590 [details], at least on Linux. Haven't tested
OSX/Windows

Comment 9

[Joe Wilson]

It should block bug 675925, just like bug 666864 does.

Comment 10

[Joe Wilson]

oops, read bug 675925 as bug 674925

Comment 12

[Markus Stange [:mstange]]

There is a very old patch in bug 851103 for this but it got stuck on lack of tests.

Comment 18

[BugBot [:suhaib / :marco/ :calixte]]

The severity field for this bug is relatively low, S3. However, the bug has 8 duplicates.
:dholbert, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Comment 19

[BugBot (nomail) [:suhaib / :marco/ :calixte]]

The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.