New issue
Advanced search Search tips

Issue 809956 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 809920



Sign in to add a comment

Fix font-weight bolder, lighter behavior for variable weights

Project Member Reported by drott@chromium.org, Feb 7 2018

Issue description

MS WPT Test
wpt/css/css-fonts/variations/font-weight-lighter-bolder.html

What we return for font-weight should match the table in 
https://drafts.csswg.org/css-fonts-4/#font-weight-prop

for lighter and bolder keyword properties.


 
Owner: cnardi@chromium.org
Status: Started (was: Assigned)
Fixing this with  bug 828747 .
Owner: drott@chromium.org
Status: Available (was: Started)
Better wait for https://chromium-review.googlesource.com/c/chromium/src/+/995452 to land first.
Status: Assigned (was: Available)

Comment 4 by f...@opera.com, Apr 5 2018

Cc: drott@chromium.org
Owner: f...@opera.com
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bc916f13fed4d1fa474c8fe7922ccb21be180227

commit bc916f13fed4d1fa474c8fe7922ccb21be180227
Author: Fredrik Söderquist <fs@opera.com>
Date: Fri Apr 06 12:11:09 2018

Update 'lighter' and 'bolder' computations to match spec

Update FontDescription::BolderWeight and LighterWeight to match the table
in https://drafts.csswg.org/css-fonts-4/#font-weight-prop .

Bug:  809956 
Change-Id: I18cb9a10ef3d86f68113bb0663e5364d0120a739
Reviewed-on: https://chromium-review.googlesource.com/997592
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Reviewed-by: Dominik Röttsches <drott@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548741}
[modify] https://crrev.com/bc916f13fed4d1fa474c8fe7922ccb21be180227/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/bc916f13fed4d1fa474c8fe7922ccb21be180227/third_party/WebKit/Source/platform/fonts/FontDescription.cpp

Comment 6 by f...@opera.com, Apr 6 2018

Status: Fixed (was: Assigned)

Sign in to add a comment