Carolina Nymark
Carolina Nymark
Nik was faster :) Thank you for working on this @akasunil. I'm glad the initial calculations were not needed. This works well both when the theme sets spacing units with...
Hi @AKSHAT2802 Are you available to continue working on this change? Can you update the PR with the latest changes from trunk? I am not able to build the PR....
I'm sorry I have not found the time to test this. Truthfully I don't understand the code in this PR, so I would not be able to do a code...
I started looking at the font face resolver class, but I have not worked with this feature before so I am not sure if I am approaching this the right...
@matiasbenedetto Hi, are you able to provide some insight on the question above?
I don't understand. How would **combining** the font families from theme.json and styles/style.json prevent fonts from theme.json from loading?
It needs to be solved, because first adding all fonts to theme.json and make all those presets available by default is not a good user experience nor a good developer...
I am not sure that the expected result would be for the "Abel" font to remain unchanged when I switch typography presets. I tried to test this in Twenty Twenty-Four....
I agree that it is never ideal when something is loaded/included and not used. But the same thing will happen if the theme author must include all fonts in theme.json...
The accessibility chapter or section in the theme developer handbook still needs a volunteer to write it. Tracked here: https://github.com/WordPress/Documentation-Issue-Tracker/issues/816