Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 2 commits into from

Conversation

SadWood
Copy link

@SadWood SadWood commented Jul 10, 2023

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

💡 需求背景和解决方案

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

📝 更新日志

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

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

  • 本条 PR 不需要纳入 Changelog

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

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

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

@uyarn
Copy link
Collaborator

uyarn commented Jul 12, 2023

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

@github-actions
Copy link

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

@SadWood
Copy link
Author

SadWood commented Jul 12, 2023

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

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

@uyarn
Copy link
Collaborator

uyarn commented Jul 12, 2023

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

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

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

@SadWood
Copy link
Author

SadWood commented Jul 13, 2023

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

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

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

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

CSS样式
image
image
image

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

@uyarn
Copy link
Collaborator

uyarn commented Jul 19, 2023

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

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

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

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

CSS样式 image image image

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

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

@SadWood
Copy link
Author

SadWood commented Jul 19, 2023

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

@SadWood SadWood closed this Apr 24, 2024
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