material-components / material-components/material-web
md-input-chip: leading padding is incorrect
Nobody has claimed this yet.
- Dominant language
- SCSS
- Stars
- 11.3k
- Forks
- 1.1k
- Avg merge
- 20h 28m
- Merged PRs (30d)
- 13
Description
### What is affected?
Component
### Description
According to the design doc[1], `leading-space` should be 12px, whereas it currently is 16px in the code[2].
[1] https://m3.material.io/components/chips/specs#1dcff1cb-22f5-41da-809e-6c85b467cff1
[2] https://github.com/material-components/material-web/blob/898e9c18753343d3816565e751cc87f8337fe44e/tokens/_md-comp-input-chip.scss#L133
### Reproduction
N/A
### Workaround
N/A
### Is this a regression?
No or unsure. This never worked, or I haven't tried before.
### Affected versions
Failing in 2.2.0
### Browser/OS/Node environment
N/A
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Open tokens/_md-comp-input-chip.scss at the referenced declaration around line 133 and compare it with the linked Material Design chip specification. Update the md-input-chip leading-space value to match the documented 12px spacing, then verify the token no longer uses 16px.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scss
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 52/100