RESOLVED FIXED 167340
Proclaim that we are considering font-display, rhythmic sizing, and text decoration level 4 
https://bugs.webkit.org/show_bug.cgi?id=167340
Summary Proclaim that we are considering font-display, rhythmic sizing, and text deco...
Myles C. Maxfield
Reported 2017-01-23 16:12:14 PST
Proclaim that we are considering font-display, rhythmic sizing, and text decoration level 4
Attachments
Patch (3.32 KB, patch)
2017-01-23 16:12 PST, Myles C. Maxfield 		no flags
DetailsFormatted DiffDiff
Patch (3.32 KB, patch)
2017-01-27 12:00 PST, Myles C. Maxfield 		saam: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Myles C. Maxfield
Comment 1 2017-01-23 16:12:41 PST
Created attachment 299551 [details] Patch
Sam Weinig
Comment 2 2017-01-24 08:41:32 PST
Comment on attachment 299551 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=299551&action=review > Source/WebCore/features.json:139 > + "description": "Allows block content to set a simple grid to which lines will adhere", The other two descriptions you added have periods at the end of the sentence in the description. We should be consistent. > Source/WebCore/features.json:179 > + "keywords": ["css text decoration", "text decoration"], Do we really need to add strict subset phrases? That seems unfortunate.
Myles C. Maxfield
Comment 3 2017-01-27 12:00:09 PST
Created attachment 299948 [details] Patch
Simon Fraser (smfr)
Comment 4 2017-01-27 12:06:12 PST
Comment on attachment 299948 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=299948&action=review > Source/WebCore/features.json:51 > + "description": "Allows content to specify timelines of what browsers display during font loading.", This text is pretty confusing.
Myles C. Maxfield
Comment 5 2017-01-27 14:41:07 PST
Committed r211303: <http://trac.webkit.org/changeset/211303>
Note You need to log in before you can comment on or make changes to this bug.