Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(catalogue): add catalogue #519

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

feat(catalogue): add catalogue #519

wants to merge 9 commits into from

Conversation

LuckyFBB
Copy link
Collaborator

@LuckyFBB LuckyFBB commented Nov 17, 2024 •
edited
Loading

变更类型

请选择以下选项以描述 PR 的类型：

  • Bug 修复（修复现有问题）
  • 新功能（添加了一个功能）
  • 代码优化（性能改进、代码重构）
  • 文档更新
  • 单测新增或修改
  • 其他（请说明）：

相关问题

修改 catalogue

变更内容

  1. useTreeData hook 种主要操作管理 data 数据，和组件节藕。通过 onChange 方法设置数据
  2. CatalogueTree 组件主要是做树的渲染，对传入的 treeData 数据做一次 icon 的变更
  3. Catalogue 组件主要是实现目录树的渲染，配置头部以及搜索新增编辑等操作

TODO

  • 拖拽的时候样式更改
  • 支持 Tab
  • 懒加载
  • 报错的时候 formItem 下方的提示
  • 操作 Menu 支持自定义

详细描述

对应 Previewer

Previewer

@LuckyFBB LuckyFBB marked this pull request as draft November 17, 2024 12:27
@LuckyFBB LuckyFBB marked this pull request as ready for review December 17, 2024 11:59
mortalYoung
mortalYoung reviewed Dec 30, 2024
View reviewed changes
src/catalogue/index.scss
width: 100%;
display: flex;
align-items: center;
&:hover {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

建议是 &:not(.ant-tree-treenode-selected):hover 会导致 hover 的样式优先级更高覆盖掉选中的样式

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mortalYoung mortalYoung mortalYoung left review comments

@liuxy0551 liuxy0551 Awaiting requested review from liuxy0551

@mumiao mumiao Awaiting requested review from mumiao

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LuckyFBB @mortalYoung