patternfly / patternfly/patternfly-react
Button - secondary color
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Is this a new component or an extension?
Enhancement to a current feature
Existing Component
Button
Describe the feature
Right now secondary color exists but is always mapped to the primary color for example button has a secondary type but the color is the same.
Also it seems that secondary is not the same as primary for dark theme.
There is a variable called --pf-v5-global--secondary-color why not use it for things that have a secondary type?
@patternfly-v5/patternfly/components/Button/button.css: --pf-v5-c-button--m-secondary--Color: var(--pf-v5-global--primary-color--100);
User Story
As a user I would like to have secondary buttons to have a different color as primary a one... (which is only possible in dark theme, but not in light)
Visuals & Mockups
No response
Interaction States & Variations
idk
Accessibility (A11y)
No response
Product & Target Release
No response
Contribution
-
I am interested in contributing this feature.
-
I have searched for similar existing requests.
Jira Issue: PF-3513
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit @patternfly-v5/patternfly/components/Button/button.css und prüfe die Farbvariable des sekundären Buttons neben den Definitionen der primären und globalen sekundären Farbe. Bestätige, dass der sekundäre Button die vorgesehene sekundäre Farbe verwendet und sich in hellen und dunklen Themes weiterhin vom primären Button unterscheidet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- css, react
- Bereich
- design, frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 62/100