jpmorganchase / salt-ds

React UI components built with a focus on accessibility, customization and ease-of-use
https://www.saltdesignsystem.com
Apache License 2.0
123 stars 90 forks source link

--size-modifier updates #2935

Open bhoppers2008 opened 8 months ago

bhoppers2008 commented 8 months ago

Whilst reviewing data grid updates inconsistencies were found between the size of the cell corner flag in figma and what we have as tokens in code. Mainly… we don’t have a token to match what we have in code.

Use of different tokens was explored explored using existing tokens to size the flag but --size-icon didn’t feel semantically correct and --size-adornment gives limited affordance, especially in hd-compact and hd.

That led to considering where we are using --size-adornment.

This issue will address three actions:

  1. Rename --size-adornment to --size-modifier.
  2. Update --size-modifier values from 6,8,10,12 to 8,10,12,14 (to offer improved affordance and standardisation).
  3. Update components using --size-modifier.

Specific component features affected for #3 are:

The result of updates are documented here.

Notes:

First column of grids is hd-compact. Left grouping is the proposal, right is whats currently available. I’ve illustrated how these would apply to Calendar, but this will be picked up in Calendar work which is currently WIP.

mark-tate commented 5 months ago

If anyone knows they need this, let us know and we will reconsider it's priority. Parking in Q3 for re-prioritisation at start of Q3