patternfly / patternfly/patternfly-react
Drawer - Refine Drawer Usage for Better Integration with External Components
还没有人认领这个 Issue。
- 主要语言
- TypeScript
- 星标
- 862
- 派生
- 392
- 平均合并
- 4 天 8 小时
- 30 天内合并 PR
- 9
描述
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
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
从 PR #4142 以及 TopologyView 对外部提供的 DrawerPanelContent 的使用开始。了解 DrawerHead 和 DrawerPanelBody 如何在 drawer 组件中组合,然后复现 sidebar 布局场景。当标准 drawer 元素无需 nested-content workaround 或手动 padding class 即可与外部 wrapper 集成时,即表示完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- react, typescript
- 领域
- frontend
- Issue 类型
- 功能
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100