patternfly / patternfly/patternfly-react
Popper - possibly add as a dependency
未关闭
还没有人认领这个 Issue。
Breaking change :boom:
- 主要语言
- TypeScript
- 星标
- 862
- 派生
- 392
- 平均合并
- 4 天 8 小时
- 30 天内合并 PR
- 9
描述
Is this a new component or an extension of an existing one?
Popper
Describe the feature
A README in our helpers/Popper/thirdparty directory mentions:
- Delete the thirdparty folder and add them as dependencies to the package.json:
"react-popper": "2.2.3"
"@popperjs/core": "2.4.2"
Most likely not as part of this breaking change, but is this something we'd want to look into for a future major release? An alternative would be https://github.com/patternfly/patternfly-react/issues/9089 which would involve just updating our thirdparty code (possibly).
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
调研方向
查看 helpers/Popper/thirdparty 中的 README，以及 package.json 对 react-popper 和 @popperjs/core 的引用。将这种依赖方式与 patternfly-react issue 9089 中描述的替代方案进行比较，然后明确未来版本所期望的方向，以及替换 third-party 文件夹将包括哪些内容。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- react, typescript
- 领域
- frontend
- Issue 类型
- 功能
- 难度
- 5/5
- 预计耗时
- 一周以上
- 活跃度
- 停滞
- 描述清晰度
- 需要澄清
- 新手友好度
- 25/100