Move fontSize and lineHeight properties to typography object#1898
Conversation
|
@madskristensen @fabiankaegy just wanted to get some eyes on this, please :) |
|
I wonder if the top level ones should just be marked as deprecated instead of being removed. People using versions of WordPress prior to 5.8 should also be able to use the schema, and removing them will break the schema for them. EDIT: To clarify, as long as the old properties still work, even if they produce a warning, I think they should still be included. When the properties are removed entirely and there is a breaking change in block.json, then we should make a new schema (a block-v2.json schema or something) that has the properties removed. |
|
Worth noting that JSON Schema Draft 2019-09 and later has a |
|
@ZebulanStanphill we talked about that here: |
|
@ryanwelcher I think this is great and the addition looks correct to me. However I agree with @ajlende that we should deprecate the old way and not remove. Also I used the block editor handbook block supports section and that was where the line height etc info was coming from. So I think we should update that also :) |
|
@fabiankaegy @ajlende I've added the original placements back and changed the description to explicitly say they have been deprecated and moved |
This error was because the nesting was incorrect. (Basically just move the |
Thanks @ajlende! I've made the changes. |
|
Is this ready to be merged? |
|
@madskristensen yes 👍 |
|
Thanks |
As of WordPress 5.8, having the
lineHeightandfontSizeproperties in the top-levelsupportsobject trigger warnings that they have been moved tosupports.typography