patternfly / patternfly/patternfly-react
Drawer - Refine Drawer Usage for Better Integration with External Components
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 862
- Forks
- 392
- Ø Merge
- 4 T. 8 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
Describe the problem
This arose from fixing a layout bug with the drawer component in PR #4142.
Using DrawerPanelContent within an externally provided DrawerPanelContent (in this case from PF Topology) breaks the layout of the sidebar panel.
- Addressed by removing the nested
DrawerPanelContentand directly applyingDrawerHeadandDrawerPanelBodywithin theDrawerPanelContentthat is externally provided from TopologyView.
In our use case, the utility class pf-v6-u-p-md was applied to get the same padding, but this workaround highlights a gap in the current implementation.
How do you reproduce the problem?
See https://github.com/opendatahub-io/odh-dashboard/pull/4142.
Expected behavior
PatternFly needs an improved way to structure and compose standard drawer elements like DrawerHead and DrawerPanelBody within an external DrawerPanelContent wrapper from the TopologyView.
Is this issue blocking you? No.
What is your product and what release date are you targeting?
RHOAI
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 PR #4142 und der Verwendung von DrawerPanelContent durch TopologyView, das extern bereitgestellt wird. Lies nach, wie DrawerHead und DrawerPanelBody innerhalb der Drawer-Komponenten zusammengesetzt werden, und reproduziere dann das Szenario mit dem Sidebar-Layout. Die Aufgabe ist abgeschlossen, wenn sich standardmäßige Drawer-Elemente ohne den nested-content-Workaround oder eine manuelle Padding-Klasse in den externen Wrapper integrieren lassen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- react, typescript
- Bereich
- frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100