Default line height is very tight

[kjellr]

Description

When a theme does not provide editor styles, the default line height appears very tight:

This stems from the reset value here:

If I remove that, it inherits the WP-Admin default of 1.5, which looks a little nicer. 1.6 or so would be more comfortable, but I'm not sure exactly what our "reset" strategy is here... the resets are in place so as to not override theme-provided styles, but I imagine we're also aiming to provide some nice defaults for when themes don't provide any editor styles, right? Do we handle those defaults somewhere else?

Possibly related to #31211

Step-by-step reproduction instructions

1. Activate the Gutenberg plugin
2. Activate a theme that does not provide editor styles (I used Isola).
3. Edit a post.
4. Note the tight line height.

Environment info

Gutenberg 11.3.0

[kjellr]

cc @jasmussen when you're around — I feel like you have more knowledge of these resets than I do.

[jasmussen]

Hey, thanks for the ping. There's a bit of context in this comment:

The intent of serif is to override the font that comes by default from wp-admin common styles and restore the browser default.
and if I just checked WP core's code the default font is not added anymore unless the theme doesn't provide a theme.json file. So my question would be, what WP version are you testing with?

The comment seems to suggest that the system fonts are only output for classic themes. Which means we might be able to output a line-height there as well. But this would likely require a core patch to output that same line-height in context of the rule that outputs system fonts.

[kjellr]

Hmm ok. Core does provide a default line height, but Gutenberg's reset styles override it. 😕 Presumably we'd need to have core provide a more specific rule?