Closed tp-cast-ai closed 7 months ago
Thanks for the report. I was also able to confirm the same problem.
I identified that this issue was caused by #55219, which decided to merge all origins.
However, regarding font size, I think it should not be merged with the default origin, but should be completely replaced by the theme.json definition.
Also, I found that the same problem occurs with SpacingControl. Here is an example theme.json:
Expected | Actual |
---|---|
Only the space sizes defined in theme.json and the default, zero, are displayed. | Both the space sizes defined in theme.json and the default space size (0, 1, 2...) are displayed. |
cc: @matiasbenedetto, @pbking
I’ve noticed this too. I’m glad the cause has been identified.
For spacing, this seems to be the same issue: https://github.com/WordPress/gutenberg/issues/55636
I'm searching for what solution would be best, but haven't found an answer yet. Therefore, I give it the "Needs Technical Feedback" label.
I discovered that a similar problem also occurs with inline highlighting.
WP6.4.1 with Gutenberg trunk enabled (Unexpected behavior) | WP6.4.1 with Gutenberg trunk disabled (Expected behavior) |
---|---|
In TT4, the default palette is displayed even though it is disabled. If you temporarily revert the changes made in ##55219 on the current trunk, this issue will not occur.
Since the suggestion on #55636 is to add settings.spacing.defaultScale
perhaps doing a similar settings.typography.defaultFontSizes
here would make sense.
I think this issue a high priority because I believe this behavior affects many users. So I'll add it to the WP6.5 Editor Tasks board.
I've got the same problem.
Is there someway I can help with this issue - reproduction steps, testing, development?
Actually there is a defaultFontSizes
property in theme.json
(see #52200 and theme.json reference) but setting it to false does not provide the expected result.
An interesting aspect:
theme.json > settings > typography > fontSizes
that has the same slug as a core default font-size:
{
"fluid": false,
"name": "Groß",
"size": "1.375rem",
"slug": "large"
},
Noting that @ajlende has a PR open for this https://github.com/WordPress/gutenberg/pull/58409
This issue has been fixed in #58951. #58951 has been given a backport label, so it should be reflected in WP6.5 Beta2.
Description
Font Size Picker seems to be broken since v16.9.0 release. See the issue below.
Step-by-step reproduction instructions
Add these font sizes to theme.json:
Try selecting a font size with Gutenberg plugin disabled (or with Gutenberg plugin v16.8.1), you will see Font Size Picker displayed correctly:
Enable Gutenberg 16.9.0, Font Size Picker displays nonsense - order is incorrect, default sizes displays incorrect values (in pixels, however it still works fine on the frontend)
Please confirm that you have searched existing issues in the repo.
Yes
Please confirm that you have tested with all plugins deactivated except Gutenberg.
Yes