dequelabs / dequelabs/cauldron

Font weight epic

Open
#855 4 comments 0 reactions 0 assignees View on GitHub
breaking design sync ux debt
Dominant language
TypeScript
Stars
127
Forks
31
Avg merge
2d 12h
Merged PRs (30d)
8

Description

## Tasks

In reviewing some UI recently with @bobbyomari, he noticed some issues with our font-weight

- [ ] drop support for [`var(--font-weight-thin)`](https://github.com/dequelabs/cauldron/blob/develop/packages/styles/variables.css#L92) as it is unused (this will be a BREAKING CHANGE for any cauldron consumers who rely on this var)
- [ ] drop support for [`var(--font-weight-light)`](https://github.com/dequelabs/cauldron/blob/develop/packages/styles/variables.css#L93) as it is unused (this will be a BREAKING CHANGE for any cauldron consumers who rely on this var)
- [ ] no longer need to [import it](https://github.com/dequelabs/cauldron/blob/60782888daab263a83724252e8891c1317da2c89/docs/index.js#L26) either
- [ ] [button's](https://github.com/dequelabs/cauldron/blob/develop/packages/styles/button.css) font-weight should be set to [`var(--font-weight-medium)`](https://github.com/dequelabs/cauldron/blob/develop/packages/styles/variables.css#L95)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.