patternfly / patternfly/patternfly-react

Popper - possibly add as a dependency

Aperta
#9,169 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Breaking change :boom:
Lingua principale
TypeScript
Stelle
862
Fork
392
Merge medio
4g 8h
PR unite (30g)
9

Descrizione

Is this a new component or an extension of an existing one?
Popper

Describe the feature
A README in our helpers/Popper/thirdparty directory mentions:

  • Delete the thirdparty folder and add them as dependencies to the package.json:
    "react-popper": "2.2.3"
    "@popperjs/core": "2.4.2"

Most likely not as part of this breaking change, but is this something we'd want to look into for a future major release? An alternative would be https://github.com/patternfly/patternfly-react/issues/9089 which would involve just updating our thirdparty code (possibly).

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

Esamina il README in helpers/Popper/thirdparty e i riferimenti in package.json a react-popper e @popperjs/core. Confronta questo approccio alle dipendenze con l’alternativa descritta in patternfly-react issue 9089, quindi chiarisci quale direzione si desidera per una versione futura e cosa comporterebbe la sostituzione della cartella third-party.

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

Valutazione

Stack tecnologico
react, typescript
Ambito
frontend
Tipo di issue
Funzionalità
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.