Closed Bug 289385 Opened 20 years ago Closed 16 years ago

Links for which you've cancelled a drag should not act as clicked

Categories

(Core :: DOM: UI Events & Focus Handling, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: sfraser_bugs, Unassigned)

Details

There's some funky behavior with link clicking, which appears to be Mac-only. On any page, mouse down on a link, and move the mouse a little to start a drag. Now, while still over some part of the link, release the mouse. It will act as if you clicked the link, and load it, which is wrong. If we've started a drag for the link, we should never behave as if it was clicked at the end of the drag. On Firefox, it's also possible to get the click-hold context menu to appear and remain visible, even as we load the target page. This appears to be Mac-only (could not reproduce on Linux).
I'm not sure if this is related, or not, but I've noticed that sometimes when I click links Gecko acts as if I've just been dragging them slightly, and so I get an outline slideback when there shouldn't be one. (I've also been noticing some slight lag in DOM mouse events, but that's not as serious and probably unrelated.)
Simon, are we actually generating a link click, or are we managing to drag the link to the page itself and fail the "don't load link if it's dragged to the same document" check?
The click is coming out of CheckForAndDispatchClick(), so it's thinking it's a click (because the mouse up is over the same element as the mouse down, according to the comment).
Do the other widget sets not send the mouseup native event if a drag has started? I don't get the mouseup event handler for the node I stop dragging over firing on Linux....
does this still occur on Mac?
Assignee: events → nobody
QA Contact: ian → events
Seems to WFM on Mac now.
(In reply to comment #6) > Seems to WFM on Mac now. => WFM then :)
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WORKSFORME
Component: Event Handling → User events and focus handling
You need to log in before you can comment on or make changes to this bug.