Closed
Bug 647665
Opened 14 years ago
Closed 13 years ago
Url bar at bottom left does not disappear if link is removed from dom
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: j, Assigned: mounir)
References
()
Details
Attachments
(1 file)
(deleted),
patch
|
bzbarsky
:
review+
Gavin
:
approval-mozilla-central+
mounir
:
checkin+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; Linux i686; rv:2.0) Gecko/20100101 Firefox/4.0
Build Identifier: Mozilla/5.0 (X11; Linux i686; rv:2.0) Gecko/20100101 Firefox/4.0
If you hover over a link, the url shows up at the bottom right,
if now the link element is removed from the dom(or the div around it), the url preview is not removed and sticks around forever.
Reproducible: Always
Steps to Reproduce:
1. open http://firefogg.org/j/bugs/linkpreview.html
2. move mouse over 'link'
Actual Results:
url preview stays around at bottom
Expected Results:
url preview should be removed if the link that is previewed is removed from the dom
Comment 1•14 years ago
|
||
I have reproduced the alleged bug in:
Mozilla/5.0 (Windows NT 6.0; rv:2.2a1pre) Gecko/20110403 Firefox/4.2a1pre
Mozilla/5.0 (X11; Linux x86_64; rv:2.2a1pre) Gecko/20110403 Firefox/4.2a1pre
Comment 2•14 years ago
|
||
Mozilla/5.0 (X11; Linux i686; rv:2.2a1pre) Gecko/20110407 Firefox/4.2a1pre
Confirming issue on latest trunk.
Present ever since Firefox 3.6.16
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 3•13 years ago
|
||
Same Issue/Dupe of 629024?
Like that one it's repro'able against 3.5. and thus not related to Bug 541656.
It's just more obvious now ;-).
Comment 5•13 years ago
|
||
The front-end actually just reacts to setOverLink() calls from the core. That ends up being triggered by nsGenericElement::LeaveLink in the link-hover case - seems like we should make it fire when links are removed from the DOM, somehow.
Component: General → DOM
Product: Firefox → Core
QA Contact: general → general
Assignee | ||
Comment 6•13 years ago
|
||
Comment 7•13 years ago
|
||
Comment on attachment 615276 [details] [diff] [review]
Patch v1
r=me
Attachment #615276 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #615276 -
Flags: approval-mozilla-central?
Updated•13 years ago
|
Attachment #615276 -
Flags: approval-mozilla-central? → approval-mozilla-central+
Assignee | ||
Comment 8•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
Assignee | ||
Comment 9•13 years ago
|
||
Backed out because of excess of stupidity. I put some stuff that shouldn't be in the patch or even pushed the wrong patch (didn't check in details).
Backout rev:
https://hg.mozilla.org/mozilla-central/rev/f9fbee492a5c
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 10•13 years ago
|
||
Comment on attachment 615276 [details] [diff] [review]
Patch v1
Pushed to inbound.
Attachment #615276 -
Flags: checkin+
Comment 11•13 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•