patternfly / patternfly/patternfly-react

Drawer - Refine Drawer Usage for Better Integration with External Components

Offen
#11,804 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Stale
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 DrawerPanelContent and directly applying DrawerHead and DrawerPanelBody within the DrawerPanelContent that 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

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 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

Neue Issues direkt in Ihr Postfach

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