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

Dev #324

Open
wants to merge 73 commits into
base: dev
Choose a base branch
from
Open

Dev #324

wants to merge 73 commits into from

Conversation

ZHDLFFY
Copy link

@ZHDLFFY ZHDLFFY commented Jun 14, 2024

该PR的类型是? (至少选择一个)

  • Bug修复
  • 新特性
  • 新组件

该PR是否向下兼容? (选择任一)

如果为否,请描述冲突情况:

涉及到的ISSUE:

  • 该PR如果涉及到某个ISSUE, 请在PR标题中描述出来 (例如. fix #xxx[,#xxx], "xxx"为ISSUE序号)

是否在Chrome浏览器下进行过测试?

如果这是一个新特性新组件相关的PR,请提供如下信息

  • 添加该特性或组件的原因
  • 文档应该修改哪些信息
  • 测试相关

提交新特性新组件前请先发起一个相关的ISSUE请求

其他信息:
原本的setAppScale方法 只对width做对比,本次修改是获取宽的比例和高的比例 ,拿到最小的值然后做transform,比如正常情况下我们的浏览器的高度有导航 栏,所以高度并不是1080,但是宽度却是1920的,所以此时 做对比 宽比是1920/1920=1 高是 小于1080的值/1080 结果小于1,所以scale 应该取高度比,如果打开了F12 则宽高比都小于1,此时需要获取最小的一个比值

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.

7 participants