Bug 167340

Summary: Proclaim that we are considering font-display, rhythmic sizing, and text decoration level 4
Product: WebKit Reporter: Myles C. Maxfield <mmaxfield>
Component: New BugsAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch saam: review+

Description Myles C. Maxfield 2017-01-23 16:12:14 PST 
Proclaim that we are considering font-display, rhythmic sizing, and text decoration level 4
Comment 1 Myles C. Maxfield 2017-01-23 16:12:41 PST 
Created attachment 299551 [details]
Patch
Comment 2 Sam Weinig 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.
Comment 3 Myles C. Maxfield 2017-01-27 12:00:09 PST 
Created attachment 299948 [details]
Patch
Comment 4 Simon Fraser (smfr) 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.
Comment 5 Myles C. Maxfield 2017-01-27 14:41:07 PST 
Committed r211303: <http://trac.webkit.org/changeset/211303>