tdesign-mobile-react icon indicating copy to clipboard operation
tdesign-mobile-react copied to clipboard

feat(navbar): 更新navbar组件属性参数和实现

Open 04756 opened this issue 1 year ago • 10 comments

🤔 这个 PR 的性质是?

  • [ ] 日常 bug 修复
  • [x] 新特性提交
  • [x] 文档改进
  • [x] 演示代码改进
  • [x] 组件样式/交互改进
  • [ ] CI/CD 改进
  • [x] 重构
  • [ ] 代码风格优化
  • [ ] 测试用例
  • [ ] 分支合并
  • [ ] 其他

🔗 相关 Issue

fix #417

💡 需求背景和解决方案

  1. 重构Navbar样式, 对齐mobile-vue的Navbar组件效果。
  2. 参照mobile-vue的Navbar组件效果和API,重构react端代码。 image image image

📝 更新日志

  • breaking(Navbar): 组件重构,移除 homeIconleftIconrightIcon 等属性、移除 onHomeClick 事件,新增 capsuleleftleftArrowright 等属性、新增 onRightClick

  • [ ] 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项。⚠️

  • [x] 文档已补充或无须补充
  • [x] 代码演示已提供或无须提供
  • [x] TypeScript 定义已补充或无须补充
  • [x] Changelog 已提供或无须提供

04756 avatar Aug 11 '24 11:08 04756

完成

github-actions[bot] avatar Aug 12 '24 15:08 github-actions[bot]

navbar.md英文的是不是少修改了,然后例子都需要把jsx改成tsx,type是不是少了一下字段,没拉取最新的tdesign-api的文件吗?

之前对api平台中心理解有误。新提交本地运行了tdesign-api进行了文件生成覆盖。 同时已经拉取最新develop分支,重新rebase啦。TNode类型属性用parseTNode进行了包裹。

04756 avatar Aug 13 '24 15:08 04756

之前对api平台中心理解有误。新提交本地运行了tdesign-api进行了文件生成覆盖。 同时已经拉取最新develop分支,重新rebase啦。TNode类型属性用parseTNode进行了包裹。

ok

HaixingOoO avatar Aug 13 '24 15:08 HaixingOoO

@HaixingOoO emm,我本地运行test:snap-update,报错 12c72896b32423da8643041d90632ff 请问该如何处理呀?

04756 avatar Aug 16 '24 11:08 04756

/update-snapshot

HaixingOoO avatar Aug 17 '24 06:08 HaixingOoO

@anlyyao @uyarn 你们看看什么原因,是不是更新tdesign-icon package的问题?

HaixingOoO avatar Aug 17 '24 07:08 HaixingOoO

@anlyyao @uyarn 你们看看什么原因,是不是更新tdesign-icon package的问题?

没有更新图标,看看本地的node 版本 @04756

uyarn avatar Aug 18 '24 07:08 uyarn

@anlyyao @uyarn 你们看看什么原因,是不是更新tdesign-icon package的问题?

没有更新图标,看看本地的node 版本 @04756

node是16.14版本。拉取最新develop分支合并,重新安装依赖后test:snap-update命令正常执行了~

04756 avatar Aug 18 '24 17:08 04756

/update-snapshot

anlyyao avatar Aug 21 '24 08:08 anlyyao

/update-snapshot

done

04756 avatar Aug 21 '24 09:08 04756

@anlyyao review done

HaixingOoO avatar Aug 22 '24 03:08 HaixingOoO