Closed Bug 23161 Opened 25 years ago Closed 25 years ago

[dogfood][EVENTTARG] Can't click on right hand URLs on CNET

Categories

(Core :: Layout, defect, P3)

x86
Windows 98
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: chriss, Assigned: joki)

References

()

Details

(Whiteboard: [PDT+])

Attachments

(1 file)

Build: 2000010408 Steps to reproduce: 1) Go to http://www.news.com 2) Click on any news article. e.g.: http://news.cnet.com/news/0-1003-200-1513804.html?tag=st.ne.1002.thed.1003-200-1513804 4) Notice that you cannot click on any of the URLs on the right side under "Latest Headlines" Note that I filed a similar bug (13275) but I don't think they are related as Amazon links seem to work OK for me.
Whiteboard: [PDT+]
PDT+
I think I have a fix for this, but it's more of a band-aid than a real fix. It's just less likely to be wrong in common situations. (I haven't tested it thouroughly yet.) I'll attach it if someone wants to look at it...
Attached patch possible fix (deleted) — Splinter Review
Summary: [dogfood] Can't click on right hand URLs on CNET → [dogfood][EVENTTARG] Can't click on right hand URLs on CNET
This would be wrong in the case where inline content overlaps a floater (which can happen for various reasons, all of them rather obscure, for example, negative margins, negative text-indent, really long words or other overflow, etc.). GetFrameForPoint really needs to be the exact reverse of Paint...
*** Bug 22759 has been marked as a duplicate of this bug. ***
*** Bug 23505 has been marked as a duplicate of this bug. ***
Note that events are also stolen from form widgets in a left|right float. (see attachment to bug #23505). [But joki and dbaron knew that already].
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
I have committed dbaron's fix. I also added some additional bits which should fix a number of other targetting issues.
*** Bug 20557 has been marked as a duplicate of this bug. ***
*** Bug 21678 has been marked as a duplicate of this bug. ***
*** Bug 23344 has been marked as a duplicate of this bug. ***
*** Bug 22812 has been marked as a duplicate of this bug. ***
*** Bug 23795 has been marked as a duplicate of this bug. ***
Fixed in the Jan 2Oth build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: