New issue
Advanced search Search tips

Issue 894831 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

Negative kerning is misrendered and creates positive space in neighboring glyphs

Project Member Reported by dcrossland@google.com, Oct 12

Issue description

Chrome Version: 

Google Chrome	69.0.3497.100 (Official Build) (64-bit)

Revision	8920e690dd011895672947112477d10d5c8afb09-refs/branch-heads/3497@{#948}

OS	Mac OS X

JavaScript	V8 6.9.427.23

What steps will reproduce the problem?

(1) Visit https://jsbin.com/zuvamif/edit?html,output

See how when two dash/minus characters precede a word with a <wbr> character inside, that creates a gap between the letters, but when one dash is used, it does not.

What is the expected result?

Inside Google Sans, we have a negative kern value of `-128` between two dashes `--`. 

This is to merge the dashes in order to create one continuous line. 

What happens instead?

This should not add any amount of space around other glyphs; having a negative kern value on this is then applied as a positive value on the adjacent character of the `<wbr>` spacing operator. 

Removing the `--` at the beginning of the string removes the issue, as does removing the kern value of the dashes; however, the kerning in GS is needed for brand purposes.

 
Labels: Fixed-In-LayoutNG
Owner: kojii@chromium.org
Status: Assigned (was: Untriaged)
Thanks for the detailed report and test case. That is very odd indeed and has to do with how handle optional whitespace.

The good news is that this is fixed in LayoutNG. The bad news is that it'll be another release or two before that launches.

Koji, would you mind looking into this to see if it's something we could address in legacy?
I wonder if this is related to https://bugs.chromium.org/p/chromium/issues/detail?id=767631 (although that is marked as fixed)
#2: yeah, looks like the same problem. It's marked as fixed in LayoutNG, not in the layout engine we ship today.

Sign in to add a comment