Skip to content
This repository has been archived by the owner on Nov 18, 2024. It is now read-only.

Inconsistencies between parent, color and type style variations #423

Closed
richtabor opened this issue Sep 25, 2024 · 8 comments
Closed

Inconsistencies between parent, color and type style variations #423

richtabor opened this issue Sep 25, 2024 · 8 comments
Labels
[Component] Style Variations [Priority] Highest Used to indicate widespread, critical issues that need immediate attention Task [Type] Bug An existing feature does not function as intended.

Comments

@richtabor
Copy link
Member

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.

@richtabor
Copy link
Member Author

richtabor commented Sep 25, 2024

Typography

Noted 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).

@richtabor
Copy link
Member Author

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.

@richtabor richtabor added [Type] Bug An existing feature does not function as intended. [Component] Style Variations labels Sep 25, 2024
@richtabor
Copy link
Member Author

It may be best to start just with the individual typography and color variations.

@carolinan
Copy link
Contributor

The final variation is merged so this work can continue 👍

@richtabor
Copy link
Member Author

The final variation is merged so this work can continue 👍

What do you mean?

@richtabor
Copy link
Member Author

For instance, if a full theme style variation is defining a different set of values for section styles, those should be in the color variations as well. As-is, it's going to be very challenging fixings things with so many variables.

CleanShot 2024-09-26 at 11 51 12

@carolinan
Copy link
Contributor

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.

@carolinan
Copy link
Contributor

The inconsistencies should all be solved (according to the diffs between the files).
There may still be some work left regarding removing duplicate styles, especially as there may be need to be some changes for the search block across all variations.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
[Component] Style Variations [Priority] Highest Used to indicate widespread, critical issues that need immediate attention Task [Type] Bug An existing feature does not function as intended.
Projects
None yet
Development

No branches or pull requests

2 participants