patternfly / patternfly/patternfly-react
Bug - RTL demo cleanup
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Noticed this page could use some updates https://www.patternfly.org/patterns/right-to-left/react-demos/paginated-table/
-
Remove the card wrapping the page content
-
Update probably all of the
<Icon />components to use the inline variant so they inherit the icon color from their parent. You can see the problem when switching to dark theme -
On mobile the masthead menu goes off screen as seen in Exhibit A
-
The logo shifts away from the hamburger menu in RTL as seen in Exhibit B
References
Exhibit A
Exhibit B
Jira Issue: PF-3991
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 der RTL-Demo für eine paginierte React-Tabelle auf der verlinkten PatternFly-Seite und untersuche ihren Card-Wrapper, die Icon-Komponenten, das Masthead-Menü und das Logo-Layout. Erledigt ist die Aufgabe, wenn die Card entfernt wurde, die Icons im dunklen Theme die Farbe ihres übergeordneten Elements übernehmen und Masthead und Logo auf Mobilgeräten in RTL korrekt positioniert bleiben.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- design, frontend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 58/100