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

Feat/image viewer

Open Kryst4ljy opened this issue 3 years ago • 4 comments

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

#259

💡 需求背景和解决方案

📝 更新日志

  • fix(imageViewer): 新增组件 imageViewer

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

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

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

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

Kryst4ljy avatar Aug 02 '22 11:08 Kryst4ljy

抱歉啊。最近有点忙,一直没合并

LeeJim avatar Aug 12 '22 07:08 LeeJim

失败

github-actions[bot] avatar Aug 12 '22 07:08 github-actions[bot]

多图切换我后续调整下。

Kryst4ljy avatar Aug 12 '22 08:08 Kryst4ljy

需要调整两个地方,

  • 多图切换应该底层依赖 swiper 来实现
  • 样式有点问题,不知道是不是因为 MobileVue 最近重构了一下

样式问题应该是我的样式没更新,在 tdesign-common 那边提交了个 pr。https://github.com/Tencent/tdesign-common/pull/692

Kryst4ljy avatar Aug 12 '22 08:08 Kryst4ljy