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: 修复当页面宽度恰为640px时菜单栏和菜单按钮都不显示的问题 #257

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

asterson
Copy link

两个commits用了两种写法,第一个的commit a1e75b6 中使用的 @media screen and (width > 640px) 的浏览器支持情况如下:
✅ Chrome 104+ (2022年8月)
✅ Firefox 103+ (2022年7月)
✅ Safari 15.4+ (2022年3月)
✅ Edge 104+ (基于Chromium)
❌ IE 不支持

第二个commit c36326a 改为了传统写法,但在页面宽度由于DPR产生的子像素计算误差卡在 639.9px到640px之间时,侧边菜单栏和菜单按钮会同时显示

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.

1 participant