tdesign-common icon indicating copy to clipboard operation
tdesign-common copied to clipboard

fix(TimePicker): 修复使用px to rem插件时大分辨率下下mask不能居中的问题

Open SadWood opened this issue 2 years ago • 7 comments

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

使用px to rem插件时,在大分辨率下mask会出现无法居中的情况,此PR优化了mask居中相关的样式代码,无其他影响。

📝 更新日志

fix(TimePicker): 修复使用px to rem插件时大分辨率下下mask不能居中的问题

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

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

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

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

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

SadWood avatar Jul 10 '23 11:07 SadWood

为什么这样子就不会有问题呢?

uyarn avatar Jul 12 '23 02:07 uyarn

tdesign-vue preview 地址:(如果你有 DOM 变动请自行在对应技术栈仓库中发起 PR) 完成

github-actions[bot] avatar Jul 12 '23 02:07 github-actions[bot]

为什么这样子就不会有问题呢?

确实还会存在问题,抱歉,这条pr不太严谨,麻烦先关闭,谢谢

SadWood avatar Jul 12 '23 08:07 SadWood

为什么这样子就不会有问题呢?

确实还会存在问题,抱歉,这条pr不太严谨,麻烦先关闭,谢谢

可以再讨论的 我不太清楚问题的具体表现是什么 能不能看看截个图转成rem之后是什么CSS样式?

uyarn avatar Jul 12 '23 11:07 uyarn

为什么这样子就不会有问题呢?

确实还会存在问题,抱歉,这条pr不太严谨,麻烦先关闭,谢谢

可以再讨论的 我不太清楚问题的具体表现是什么 能不能看看截个图转成rem之后是什么CSS样式?

会有略微错位,样式和效果看下图

CSS样式 image image image

在4800*1620分辨率下显示的效果,此时1rem等于21.6px image

SadWood avatar Jul 13 '23 06:07 SadWood

为什么这样子就不会有问题呢?

确实还会存在问题,抱歉,这条pr不太严谨,麻烦先关闭,谢谢

可以再讨论的 我不太清楚问题的具体表现是什么 能不能看看截个图转成rem之后是什么CSS样式?

会有略微错位,样式和效果看下图

CSS样式 image image image

在4800*1620分辨率下显示的效果,此时1rem等于21.6px image

我猜应该是因为取整的关系?

uyarn avatar Jul 19 '23 12:07 uyarn

我也觉得可能是,之前提交过一次 pr 就是修复取整问题的,在那之前错位更严重

SadWood avatar Jul 19 '23 14:07 SadWood