RESOLVED FIXED 155826
Web Inspector: Use font-variant-numeric: tabular-nums instead of -apple-system-monospaced-numbers
https://bugs.webkit.org/show_bug.cgi?id=155826
Summary Web Inspector: Use font-variant-numeric: tabular-nums instead of -apple-syste...
Timothy Hatcher
Reported 2016-03-23 19:17:31 PDT
We should be a good citizen and use font-variant-numeric. Myles plans to remove -apple-system-monospaced-numbers at some point.
Attachments
Patch (3.56 KB, patch)
2016-03-26 23:58 PDT, Nikita Vasilyev
mmaxfield: review+
Patch for Landing (3.56 KB, patch)
2016-03-28 10:01 PDT, Timothy Hatcher
no flags
Radar WebKit Bug Importer
Comment 1 2016-03-23 19:17:47 PDT
Nikita Vasilyev
Comment 2 2016-03-26 23:53:53 PDT
Should I remove -apple-system-monospaced-numbers from CSS autocompletion list as well?
Nikita Vasilyev
Comment 3 2016-03-26 23:56:57 PDT
We currently don't autocomplete font-variant-numeric at all. Which values do we support?
Nikita Vasilyev
Comment 4 2016-03-26 23:58:28 PDT
Myles C. Maxfield
Comment 5 2016-03-27 07:17:58 PDT
(In reply to comment #2) > Should I remove -apple-system-monospaced-numbers from CSS autocompletion > list as well? Yes, Please.
Myles C. Maxfield
Comment 6 2016-03-27 07:20:15 PDT
(In reply to comment #3) > We currently don't autocomplete font-variant-numeric at all. > Which values do we support? All of them. https://developer.mozilla.org/en-US/docs/Web/CSS/font-variant-numeric
Nikita Vasilyev
Comment 7 2016-03-27 21:20:17 PDT
I created bug 155941: Web Inspector: Add font-variant-numeric to CSS autocompletions
Myles C. Maxfield
Comment 8 2016-03-28 08:51:58 PDT
Comment on attachment 274997 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=274997&action=review > Source/WebInspectorUI/ChangeLog:3 > + Web Inspector: Use font-variant-numeric: tabular-nuns instead of -apple-system-monospaced-numbers I hope I don't cross any tabular-nuns during church! I think you mean tabular-nums.
Timothy Hatcher
Comment 9 2016-03-28 10:01:50 PDT
Created attachment 275026 [details] Patch for Landing
WebKit Commit Bot
Comment 10 2016-03-28 11:01:40 PDT
Comment on attachment 275026 [details] Patch for Landing Clearing flags on attachment: 275026 Committed r198751: <http://trac.webkit.org/changeset/198751>
WebKit Commit Bot
Comment 11 2016-03-28 11:01:45 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.