Bug 57232

Summary: Change default alignment for LI elements to -webkit-match-parent
Product: WebKit Reporter: Jeremy Moskovich <playmobil>
Component: New BugsAssignee: Jeremy Moskovich <playmobil>
Status: RESOLVED FIXED    
Severity: Normal CC: aharon, eric, mitz, rniwa, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch none

Description Jeremy Moskovich 2011-03-28 07:42:30 PDT
Change default alignment for LI elements to -webkit-match-parent
Comment 1 Jeremy Moskovich 2011-03-28 07:48:07 PDT
Created attachment 87143 [details]
Patch
Comment 2 Eric Seidel (no email) 2011-03-28 07:54:11 PDT
Comment on attachment 87143 [details]
Patch

LGTM.  I assume other browsers don't pass this yet?
Comment 3 Eric Seidel (no email) 2011-03-28 07:55:19 PDT
Would be nice if your ChangeLog could give more justification for this change.  (Like citations of the spec, or the style mailing list.)

Has this changed in HTML5?  If not, will you please file a bug against the HTML5 spec and reference it form this bug (and ideally your ChangeLog).
Comment 4 Aharon (Vladimir) Lanin 2011-03-29 03:30:59 PDT
(In reply to comment #3)
> Would be nice if your ChangeLog could give more justification for this change.  (Like citations of the spec, or the style mailing list.)
> 
> Has this changed in HTML5?  If not, will you please file a bug against the HTML5 spec and reference it form this bug (and ideally your ChangeLog).

This has been discussed on the CSS list, see http://lists.w3.org/Archives/Public/www-style/2011Feb/0066.html and keep going in the thread. It includes the status vis-a-vis the HTML5 spec.
Comment 5 Jeremy Moskovich 2011-03-29 06:40:32 PDT
HTML5 bug: http://www.w3.org/Bugs/Public/show_bug.cgi?id=10824
Comment 6 Ryosuke Niwa 2011-04-03 00:13:12 PDT
Has this been landed or not?
Comment 7 Ryosuke Niwa 2011-06-18 10:21:01 PDT
Jeremy, have you landed this patch yet?
Comment 8 WebKit Review Bot 2011-06-18 12:52:47 PDT
Comment on attachment 87143 [details]
Patch

Clearing flags on attachment: 87143

Committed r89199: <http://trac.webkit.org/changeset/89199>
Comment 9 WebKit Review Bot 2011-06-18 12:52:52 PDT
All reviewed patches have been landed.  Closing bug.