Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrading to Catalina makes Code use wrong italic font on non-default font weights #82122

Closed
Kataract opened this issue Oct 8, 2019 · 9 comments
Assignees
Labels
font-rendering Font rendering issues info-needed Issue requires more information from poster

Comments

@Kataract
Copy link

Kataract commented Oct 8, 2019

Issue Type: Bug

  1. Starting on macOS Mojave, have a default font for your workspace/editor that has an italic weight and oblique weight installed (I use Victor Mono: https://rubjo.github.io/victor-mono/), along with a theme that supports italics (currently using Night Owl). Italics should render with cursive by default.
  2. Use a different font weight than the default (I use 700 because the default is too thin to see for me)
  3. Upgrade from Mojave to Catalina
  4. Files in editor no longer use the italic weight and instead use the oblique weight - I noticed this because it wasn't showing in cursive anymore
  5. Set font weight to "normal" (default), observe that the right italic font is used

Screenshots:

Default font weight:
default font weight with cursive font

Font weight 700:
font weight 700 with fake italics

VS Code version: Code 1.38.1 (b37e54c, 2019-09-11T13:31:32.854Z)
OS version: Darwin x64 19.0.0

System Info
Item Value
CPUs Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)
GPU Status 2d_canvas: enabled
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: enabled
oop_rasterization: disabled_off
protected_video_decode: unavailable_off
rasterization: enabled
skia_deferred_display_list: disabled_off
skia_renderer: disabled_off
surface_synchronization: enabled_on
video_decode: enabled
viz_display_compositor: disabled_off
webgl: enabled
webgl2: enabled
Load (avg) 4, 4, 4
Memory (System) 32.00GB (0.39GB free)
Process Argv
Screen Reader no
VM 0%
Extensions (12)
Extension Author (truncated) Version
bracket-pair-colorizer Coe 1.0.61
vscode-eslint dba 1.9.1
gitlens eam 10.1.0
vscode-npm-script eg2 0.3.9
prettier-vscode esb 2.3.0
vscode-pull-request-github Git 0.11.2
MagicPython mag 1.1.0
vscode-less mrm 0.6.3
vscode-docker ms- 0.8.1
python ms- 2019.9.34911
material-icon-theme PKi 3.9.1
vscodeintellicode Vis 1.1.9

(1 theme extensions excluded)

@alexdima
Copy link
Member

alexdima commented Oct 9, 2019

Duplicate of #82109

@alexdima alexdima marked this as a duplicate of #82109 Oct 9, 2019
@alexdima alexdima closed this as completed Oct 9, 2019
@alexdima alexdima added the *duplicate Issue identified as a duplicate of another issue(s) label Oct 9, 2019
@sschultze
Copy link

I have referenced this issue in https://bugs.chromium.org/p/chromium/issues/detail?id=1005969 ("Narrow kerning of system font in macOS Catalina") because this seems to be the underlying issue of #82109. Not sure whether they are really related though.

@rubjo
Copy link

rubjo commented Oct 15, 2019

@alexandrudima Are you sure this is a duplicate of #82109?

screenrec.zip

Issues shown in screen recording:

  • Oblique style chosen over italic style for all weights apart from 600.
  • Weights 200 and 300 are wrong for the oblique text, seems to use 100.
  • Weights 100 and 200 are wrong for the normal text, seems to use 300.

(Weights 800 and 900 are not installed, so that's fine.)

@alexdima
Copy link
Member

No, you're right, I'm not sure it is a duplicate... But it is similar...

@alexdima alexdima reopened this Oct 15, 2019
@alexdima alexdima removed the *duplicate Issue identified as a duplicate of another issue(s) label Oct 15, 2019
@alexdima alexdima assigned deepak1556 and unassigned alexdima Oct 15, 2019
@rubjo
Copy link

rubjo commented Feb 26, 2020

On latest macOS 10.15.4 beta (19E234g) with latest VS Code Insiders (1.43.0-insider), this has become decidedly less weird.

Skjermbilde 2020-02-26 kl  21 42 03

Everything works for me now, apart from this: weights 100, 200 and 300 all seem to employ the 100 (thin) variant.

@alexdima alexdima added the font-rendering Font rendering issues label Feb 26, 2020
@cy-by
Copy link

cy-by commented Mar 12, 2020

For me, font weight 600 is applying oblique instead of the correct italic face:

https://www.loom.com/share/03f0417b445e4296ada958ddf747d14c

@deepak1556
Copy link
Collaborator

Is this still an issue with latest stable ?

@deepak1556 deepak1556 added the info-needed Issue requires more information from poster label Nov 5, 2020
@Kataract
Copy link
Author

Kataract commented Nov 9, 2020

I'm no longer having this issue, no

@deepak1556
Copy link
Collaborator

Thanks for confirming, closing per above comment.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
font-rendering Font rendering issues info-needed Issue requires more information from poster
Projects
None yet
Development

No branches or pull requests

6 participants