Skip to content

Conversation

@liweijie0812
Copy link
Collaborator

@liweijie0812 liweijie0812 commented Dec 26, 2025

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

Tencent/tdesign-vue-next#6290

💡 需求背景和解决方案

问题原因:

原代码使用了 d: path('M3.75 10.2002L7.99274 5.7998L12.2361 10.0425');
CSS path() 函数在某些版本的 Safari 中支持不完整,特别是用于动态更改 SVG path 数据时
解决方案:

移除了不兼容的 path() 函数和 d 属性动画
改用 transform: scaleY(-1) 实现箭头翻转效果
保持了相同视觉效果,但使用更广泛支持的 CSS transform 属性
添加了 transform-origin: center 确保旋转中心正确
兼容性提升:

transform: scaleY(-1) 适配所有现代浏览器(含Safari),兼容性优异;实现垂直翻转效果更流畅稳定,且完全保留组件原有交互体验。

📝 更新日志

  • fix(Menu): 在 safari 浏览器中点击展开图标没有变化

  • 本条 PR 不需要纳入 Changelog

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

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

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

@tdesign-bot
Copy link
Collaborator

tdesign-bot commented Dec 26, 2025

TDesign Component Repositories CI Test Open

Component Lint Test Build Preview
tdesign-vue 👀
tdesign-vue-next 👀
tdesign-react 👀
tdesign-web-components 👀
tdesign-mobile-vue 👀
tdesign-mobile-react 👀

@Wesley-0808
Copy link
Collaborator

🤔 向左转还是向右转呢 设计稿没这点🤣

@HaixingOoO
Copy link
Collaborator

原来这个是使用path修改动画的,需要确定统一的动画和旋转方向

@liweijie0812
Copy link
Collaborator Author

改了,跟原来一样上下翻转

@uyarn
Copy link
Collaborator

uyarn commented Dec 30, 2025

其实还是不太一样的 这个改动方式是最早的实现方式,是设计侧没妥协才用d path 实现,不过确实safari没兼容

  • 使用 transform 放慢看在翻转的时候会有一个水平空白的角度的过程
1.mov
  • 使用 d path
2.mov

@uyarn
Copy link
Collaborator

uyarn commented Dec 30, 2025

也许可以再单独增加个transform的class 判断是UA 是safari 增加那transform的class 兼容 不要直接删掉d?

@HaixingOoO
Copy link
Collaborator

也许可以再单独增加个transform的class 判断是UA 是safari 增加那transform的class 兼容 不要直接删掉d?

那只能加个兼容性处理样式了

@uyarn uyarn merged commit 770f1a1 into develop Jan 4, 2026
4 checks passed
@uyarn uyarn deleted the fix/safari/fake-arrow branch January 4, 2026 08:22
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.

6 participants