patternfly / patternfly/patternfly-react

Tooltip- rename trigger prop

Offen
#8,740 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Breaking change :boom: PF Team
Vorherrschende Sprache
TypeScript
Sterne
862
Forks
392
Ø Merge
4 T. 8 Std.
Gemergte PRs (30 T.)
9

Beschreibung

Currently the trigger prop is very similar to Popper's trigger prop, can it's purpose can be confused with triggerRef that will exist for Popper. Since it's an action trigger for the tooltip, maybe it can be renamed to triggerEvent instead.


Jira Issue: PF-139

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die trigger-Prop der Tooltip-Komponente zu lokalisieren und im Repository nach ihren Verwendungen zu suchen, einschließlich Tests und Dokumentation. Benenne die Action-Prop konsistent in triggerEvent um und aktualisiere alle Verweise, damit die Tooltip-API und ihre Tests den neuen Namen verwenden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
react, typescript
Bereich
frontend
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.