Bug 23839

Summary: REGRESSION(r40254): Definition terms with display style of run-in display :after content like :before content
Product: WebKit Reporter: Chasen Le Hara <rendezvouscp>
Component: CSSAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: hyatt, mrowe
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: Mac (Intel)   
OS: OS X 10.5   
URL: https://ironmoney.com/api/statuses/
Attachments:
Description Flags
Test case
none
Patch sam: review+

Chasen Le Hara
Reported 2009-02-08 22:27:27 PST
Content defined with the :after pseudo-element appears before the term in a definition list when the display is set to run-in. It should appear after the term instead of before the term. This is a regression (works in current versions of Safari on the Mac and was working in nightlies until sometime early last week).
Attachments
Test case (405 bytes, text/html)
2009-02-08 22:30 PST, Chasen Le Hara 		no flags
Details
Patch (22.12 KB, patch)
2009-02-08 23:12 PST, Dave Hyatt 		sam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Chasen Le Hara
Comment 1 2009-02-08 22:30:02 PST
Created attachment 27477 [details] Test case The term should have a colon after it instead of before it.
Mark Rowe (bdash)
Comment 2 2009-02-08 22:36:49 PST
<rdar://problem/6567717>
Mark Rowe (bdash)
Comment 3 2009-02-08 22:43:15 PST
<http://trac.webkit.org/changeset/40254>
Dave Hyatt
Comment 4 2009-02-08 23:12:34 PST
Created attachment 27478 [details] Patch
Sam Weinig
Comment 5 2009-02-08 23:13:56 PST
Comment on attachment 27478 [details] Patch r=me > =================================================================== > --- LayoutTests/fast/runin/002.html (revision 0) > +++ LayoutTests/fast/runin/002.html (revision 0) > @@ -0,0 +1,17 @@ > +<!DOCTYPE html> > +<html> > + <head> > + <title>WebKit Bug 23839: Definition terms with display style of run-in display :after content like :before content</title> > + <link href="http://chasenlehara.com/" rel="author" title="Chasen Le Hara" /> > + <style type="text/css"> > +dt {display: run-in} > +dt:after {content: ":"} > + </style> > + </head> > + <body> > + <dl> > + <dt>Term</dt> > + <dd>Definition</dd> > + </dl> > + </body> > +</html> > \ No newline at end of file Missing newline.
Dave Hyatt
Comment 6 2009-02-08 23:17:09 PST
Fixed in r40778.
Note You need to log in before you can comment on or make changes to this bug.