Bug 203860 – REGRESSION: font-variant-numeric puts large space after period

• Component: CSS
• Status: RESOLVED CONFIGURATION CHANGED

[Ross Kirsling]

Created attachment 382843 [details]
Screenshot

Repro:
1. Visit https://developer.mozilla.org/en-US/docs/Web/CSS/font-variant-numeric.
2. Choose a value other than `normal` in the demo.

Result:
In addition to the intended effect, a very large space is inserted after the period.
(See attached screenshot.)

Notes:
No repro in STP 95, so this regression was introduced sometime after r251210.

[Ross Kirsling]

Ugh, the STP 95 range is supposed to be r250947 - r251210, but building r251209 (https://github.com/WebKit/webkit/commit/e146947e3de2db01460172509f724ad9a1d99ced) on Mojave still exhibits the problem (with run-minibrowser as well as run-safari). Not sure how that's possible.

[Ross Kirsling]

This is pretty deeply confusing. Builds take too long so I started grabbing archives from here:
https://s3-us-west-2.amazonaws.com/minified-archives.webkit.org/mac-mojave-x86_64-release/######.zip

But even if I go ten thousand revisions back (e.g. to 241001), I cannot seem to find a point "before" this regression.

I wonder if there's something that local builds (and their archived versions) depend upon that's different from Safari and STP releases (since releases do not repro)?

---

Just for the record, it's definitely not machine-specific, though it could be Mojave-specific, as I don't currently have a machine running Catalina. Also, repro is pretty darn noticeable as it affects every period in Web Inspector's timeline and layers data grids.

[Myles C. Maxfield]

It doesn't seem to reproduce for me on Catalina. Do you have a Catalina machine to test with?

[Ross Kirsling]

(In reply to Myles C. Maxfield from comment #3)
> It doesn't seem to reproduce for me on Catalina. Do you have a Catalina
> machine to test with?

I'm not able to upgrade my work machine at the moment, but there's nothing keeping me from upgrading my home machine -- I can do that soon and report back.

[Ross Kirsling]

Confirmed that r252217 reproes on Mojave and does not repro on Catalina.

[Ahmad Saleem]

It seems to be Mojave specific issue but not present in Catalina and current Safari 16.1 on Ventura also does not display any space etc. as seen in the attached screenshot after period.

I am marking this as "RESOLVED CONFIGURATION CHANGED" but if it is reproducible or if other status would be appropriate. Please tag / mark or change accordingly. Thanks!