Skip to content

Conversation

@RSS1102
Copy link
Collaborator

@RSS1102 RSS1102 commented Jan 18, 2026

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

📝 更新日志

  • fix(组件名称): 处理问题或特性描述 ...

  • 本条 PR 不需要纳入 Changelog

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

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

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 18, 2026

Open in StackBlitz

npm i https://pkg.pr.new/tdesign-mobile-react@868

commit: 1500de5

@github-actions
Copy link
Contributor

github-actions bot commented Jan 18, 2026

失败

@RSS1102 RSS1102 force-pushed the rss1102/test/swiper branch from 3252cbb to f30b3b8 Compare January 18, 2026 08:09
@RSS1102 RSS1102 marked this pull request as draft January 18, 2026 09:00
@RSS1102
Copy link
Collaborator Author

RSS1102 commented Jan 18, 2026

image

快照测试错误总结

组件 失败的示例文件 错误类型 原因说明
Swiper animation.tsx 新增 t-swiper--inside 类名
新增 t-swiper-nav--inside 类名
根据 Vue 版本实现,当 navigation.placement'inside'(默认值)且 paginationPosition'bottom' 时,应添加这些类名标识导航器位于内部
Swiper base.tsx 同上 同上
Swiper card.tsx 同上 同上
Swiper current.tsx 同上 同上
Swiper custom.tsx 同上 同上
Swiper direction.tsx 同上 同上
Swiper fraction.tsx 同上 同上
Swiper index.tsx 同上 同上
Swiper outside.tsx 同上 同上
Swiper pagination-type.tsx 同上 同上
Swiper test.tsx 同上 同上
Swiper vertical.tsx 同上 同上
ActionSheet grid.tsx 1. 新增 t-swiper--inside 类名
2. flex-direction: column → row
1. 内部使用了 Swiper 组件,继承了 Swiper 的类名变更
ActionSheet grid-multiple.tsx 同上 同上
ActionSheet index.tsx 同上 同上(index 包含 grid 示例)

错误统计

  • 总失败数: 43 个快照测试
  • 涉及组件: 2 个 (Swiper, ActionSheet)

提取常量到组件外部避免重复创建,将纯函数和事件处理函数改为 useCallback 优化重渲染,优化 useImperativeHandle 暴露 ref,修复定时器清理
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants