-
Notifications
You must be signed in to change notification settings - Fork 127
Inconsistencies between parent, color and type style variations #423
Comments
TypographyNoted that the typography style variations are quite different from the typography values in the full style variations. The same applies for any type style in a full style variation. Those should be available in the typography variations. If font sizes change, then the font size presets should be included in the typography variation (just like how color palettes are included in color variations). |
If section styles are adapted, like they are in some theme style variations, those same color declarations should be made in the /styles/colors/* style variations. If colors are applied to specific blocks, then those should also exist in these color variations. |
It may be best to start just with the individual typography and color variations. |
The final variation is merged so this work can continue 👍 |
What do you mean? |
When this issue was created, all full / combined / theme style variations were not in trunk. Now they are. |
The inconsistencies should all be solved (according to the diffs between the files). |
Now that we've been tightening up type (#422) and colors (#415) there are many inconsistencies between the color and typography variations, alongside the full style variations.
I propose we tighten up the color and typography variations individually (removing any values that don't need overriding, fixed typography values—they should use presets like any other style variation—etc), then compose a few for the full style variations.
Each style variation should be a combination of typography presets and color presets. Otherwise it's very difficult to keep a high bar for these. If we can't do that, then we should only move forward with individual typography and color variations.
The text was updated successfully, but these errors were encountered: