Commit d48943b
Fix ESM build dropping form-control color tokens
The `--brand-control-*` color tokens only reached CSS through a shared
side-effect import in the forms barrel (`src/forms/index.ts`), which the
ESM build's tree-shaking dropped. As a result, consumers importing from
`@primer/react-brand/esm` without the global `lib/css/main.css` lost all
form-control borders and boxes (checkboxes and radios rendered invisible).
Co-locate the control `colors-with-modes.css` import in each form
component (TextInput, Textarea, Select, Checkbox, Radio, FormControl),
matching the established Button pattern, so the tokens ride along in the
component modules that are unavoidably included when the components are
used. Remove the now-redundant barrel import.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: afd2ea59-8442-4214-bd4b-ecc92d8987a51 parent 3c89bdc commit d48943b
8 files changed
Lines changed: 13 additions & 5 deletions
File tree
- .changeset
- packages/react/src/forms
- Checkbox
- FormControl
- Radio
- Select
- TextInput
- Textarea
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
| 16 | + | |
16 | 17 | | |
17 | 18 | | |
18 | 19 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
8 | 9 | | |
9 | 10 | | |
10 | 11 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
7 | 8 | | |
8 | 9 | | |
9 | 10 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| 7 | + | |
7 | 8 | | |
8 | 9 | | |
9 | 10 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
2 | | - | |
3 | | - | |
4 | | - | |
5 | | - | |
6 | 1 | | |
7 | 2 | | |
8 | 3 | | |
| |||
0 commit comments