patternfly / patternfly/patternfly
Bug - Breadcrumb - Missing `font-weight` CSS variable
Nobody has claimed this yet.
- Dominant language
- SCSS
- Stars
- 817
- Forks
- 116
- Avg merge
- 4d 11h
- Merged PRs (30d)
- 14
Description
Describe the problem
This is an issue with
- Patternfly 5
- Patternfly 6
- other
Within Breadcrumb component I'm noticing that --pf-v6-c-breadcrumb__item--FontWeight is being used but is undefined. Checking out PF5 in Cockpit I'm noticing it was also missing back there.
How do you reproduce the problem?
- Inspect a breadcrumb item
- See font-weight CSS for
.pf-v6-c-breadcrumb__itemclass - See it undefined
https://www.patternfly.org/components/breadcrumb/react/basic
Expected behavior
Either the font-weight is defined through a variable or removed altogether.
Is this issue blocking you?
No
Screenshots
What is your environment?
- OS: [e.g. iOS]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]
What is your product and what release date are you targeting?
Any other information?
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
Start with the Breadcrumb component styles and the PatternFly Breadcrumb React basic example linked in the issue. Inspect where --pf-v6-c-breadcrumb__item--FontWeight is used and whether it is defined; done when the breadcrumb item no longer produces an undefined font-weight, either through a valid variable or by removing the use.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scss
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100