patternfly / patternfly/patternfly-react

Bug - RTL demo cleanup

Aperta
#12,351 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

PF Team Stale
Lingua principale
TypeScript
Stelle
862
Fork
392
Merge medio
4g 8h
PR unite (30g)
9

Descrizione

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

image

Exhibit B

image


Jira Issue: PF-3991

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia con la demo React di tabella paginata RTL nella pagina PatternFly collegata e ispeziona il relativo wrapper della card, i componenti Icon, il menu del masthead e il layout del logo. Il lavoro è completato quando la card viene rimossa, le icone ereditano il colore del relativo elemento padre nel tema scuro e il masthead e il logo rimangono correttamente posizionati su dispositivi mobili in RTL.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
react, typescript
Ambito
design, frontend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
58/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.