patternfly / patternfly/patternfly-react
Popper - possibly add as a dependency
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 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).
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
Überprüfe die README in helpers/Popper/thirdparty und die package.json-Referenzen auf react-popper und @popperjs/core. Vergleiche diesen Abhängigkeitsansatz mit der in patternfly-react issue 9089 beschriebenen Alternative und kläre, welche Richtung für eine zukünftige Veröffentlichung gewünscht ist und was das Ersetzen des third-party-Ordners umfassen würde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100