patternfly / patternfly/patternfly-react
Bug - Table - Refactor Column Management with Draggable demo
还没有人认领这个 Issue。
- 主要语言
- TypeScript
- 星标
- 862
- 派生
- 392
- 平均合并
- 4 天 8 小时
- 30 天内合并 PR
- 9
描述
Describe the problem
A clear and concise description of the problem. Which components are affected?
@jenny-s51 This looks great, but there seems to be a bug with column reordering via drag & drop. If I reorder columns and then click Save, upon returning to the table, it's empty.
Originally posted by @mcarrano in https://github.com/patternfly/patternfly-react/pull/7446#pullrequestreview-986229584
@mcarrano noticed this bug upon review of #7446 ... this bug already existed in the column management draggable demo. Looks like we're using DataList's deprecated onDrag API here, along with the deprecated itemOrder prop.
In order to fix this bug, we need to refactor this demo to use DragDrop with DataList here.
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
调研方向
从列管理的可拖动演示开始，检查其对 DataList 的使用，尤其是已弃用的 onDrag API 和 itemOrder prop。将其重构为使用 DragDrop 和 DataList，然后验证重新排列列、保存并返回表格后仍能保留这些列，而不是显示一个空表格。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- react, typescript
- 领域
- frontend
- Issue 类型
- 重构
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100