WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 91723
58422
[REGRESSION] A::before content not clickable
https://bugs.webkit.org/show_bug.cgi?id=58422
Summary
[REGRESSION] A::before content not clickable
Pavel Feldman
Reported
2011-04-13 02:26:06 PDT
Open webkit.org, navigate to a css using sidebar. Broken!
Attachments
[HTML] Reduced test case
(228 bytes, text/html)
2011-04-13 07:31 PDT
,
Alexander Pavlov (apavlov)
no flags
Details
Proposed Patch.
(8.09 KB, patch)
2011-05-23 18:34 PDT
,
Kulanthaivel Palanichamy
no flags
Details
Formatted Diff
Diff
Proposed Patch.
(8.91 KB, patch)
2011-05-24 12:33 PDT
,
Kulanthaivel Palanichamy
webkit.review.bot
: commit-queue-
Details
Formatted Diff
Diff
Archive of layout-test-results from ec2-cr-linux-02
(1.20 MB, application/zip)
2011-05-24 14:32 PDT
,
WebKit Review Bot
no flags
Details
Proposed Patch.
(8.91 KB, patch)
2011-05-24 17:43 PDT
,
Kulanthaivel Palanichamy
hyatt
: review-
Details
Formatted Diff
Diff
Proposed Patch V2.
(7.20 KB, patch)
2011-05-26 11:12 PDT
,
Kulanthaivel Palanichamy
hyatt
: review-
Details
Formatted Diff
Diff
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
Alexander Pavlov (apavlov)
Comment 1
2011-04-13 07:29:55 PDT
This is a regression from
http://trac.webkit.org/changeset/83075
Alexander Pavlov (apavlov)
Comment 2
2011-04-13 07:31:17 PDT
Created
attachment 89374
[details]
[HTML] Reduced test case
Alexander Pavlov (apavlov)
Comment 3
2011-04-13 07:35:38 PDT
Mouse events on A::before content should be handled the same way as on the A content.
Alexey Proskuryakov
Comment 4
2011-04-13 11:24:49 PDT
<
rdar://problem/9279324
>
Alexander Pavlov (apavlov)
Comment 5
2011-04-14 05:13:58 PDT
As a regression, it should be P1.
Julien Chaffraix
Comment 6
2011-05-23 18:09:05 PDT
***
Bug 60543
has been marked as a duplicate of this bug. ***
Kulanthaivel Palanichamy
Comment 7
2011-05-23 18:34:21 PDT
Created
attachment 94539
[details]
Proposed Patch.
Kulanthaivel Palanichamy
Comment 8
2011-05-24 12:33:53 PDT
Created
attachment 94667
[details]
Proposed Patch. Missed out a layout test image resource in my previous patch.
WebKit Review Bot
Comment 9
2011-05-24 14:32:08 PDT
Comment on
attachment 94667
[details]
Proposed Patch.
Attachment 94667
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://queues.webkit.org/results/8727694
New failing tests: fast/css/before-after-pseudo-element-selector-empty-anchor.html
WebKit Review Bot
Comment 10
2011-05-24 14:32:13 PDT
Created
attachment 94689
[details]
Archive of layout-test-results from ec2-cr-linux-02 The attached test failures were seen while running run-webkit-tests on the chromium-ews. Bot: ec2-cr-linux-02 Port: Chromium Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Kulanthaivel Palanichamy
Comment 11
2011-05-24 17:43:52 PDT
Created
attachment 94722
[details]
Proposed Patch. Fixed the expected test result.
Dave Hyatt
Comment 12
2011-05-25 11:20:47 PDT
Comment on
attachment 94722
[details]
Proposed Patch. I don't think this should be necessary. If you don't have a node you should just be moving up the line box tree to parents (and/or the RenderObject tree to parents). You shouldn't need a special bit just to identify generated content.
Kulanthaivel Palanichamy
Comment 13
2011-05-26 11:12:08 PDT
Created
attachment 95007
[details]
Proposed Patch V2.
Alexander Pavlov (apavlov)
Comment 14
2011-06-14 09:05:42 PDT
@reviewers: ping. This bug breaks a few bits of Web Inspector...
Pavel Feldman
Comment 15
2011-06-14 09:06:01 PDT
There is another report on this issue:
http://code.google.com/p/chromium/issues/detail?id=84256
This time, error circle next to the error count in console is not clickable (it is also implemented as ::before).
Alexander Pavlov (apavlov)
Comment 16
2011-07-05 01:20:58 PDT
@reviewers: anyone?
Dave Hyatt
Comment 17
2011-07-19 17:47:19 PDT
Comment on
attachment 95007
[details]
Proposed Patch V2. This isn't quite right. You want something narrower. I suspect you could move the code from addNodeToRectBasedTestResult to updateHitTestResult to fix this bug.
Alexander Pavlov (apavlov)
Comment 18
2011-10-26 05:08:50 PDT
Chromium issue:
http://code.google.com/p/chromium/issues/detail?id=82888
luctiemessen
Comment 19
2012-08-28 01:24:57 PDT
Is this bug still under development? I am pretty new to this bug filing stuff but it seems the previous comment is referring to a chromium bug report that seems to be fixed but I am still getting this bug.
Alexander Pavlov (apavlov)
Comment 20
2012-08-28 04:52:39 PDT
The Chromium bugfix is a workaround of this issue, so this one is still valid and is waiting for someone to fix it.
Martin Robinson
Comment 21
2013-03-01 18:06:43 PST
*** This bug has been marked as a duplicate of
bug 91723
***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug