-
Notifications
You must be signed in to change notification settings - Fork 199
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: 修复浏览器窗口多次放大后表格渲染模糊 close #2884 #2897
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -104,24 +115,34 @@ export class HdAdapter { | |||
options: { width, height }, | |||
} = this.spreadsheet; | |||
const canvas = this.spreadsheet.getCanvasElement(); | |||
const currentRatio = Math.ceil(ratio); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果是窗口缩放场景, DPR 可能是 1.2 , 也可能是 1.2123123 这种, 没啥意义, 统一取整
Size Change: +30 B (+0.01%) Total Size: 329 kB
ℹ️ View Unchanged
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## next #2897 +/- ##
==========================================
+ Coverage 75.77% 80.81% +5.03%
==========================================
Files 257 180 -77
Lines 11994 10412 -1582
Branches 2464 2360 -104
==========================================
- Hits 9088 8414 -674
+ Misses 1398 554 -844
+ Partials 1508 1444 -64 ☔ View full report in Codecov by Sentry. |
🎉 This PR is included in version @antv/s2-v2.0.0-next.28 🎉 The release is available on: Your semantic-release bot 📦🚀 |
👀 PR includes
🐛 Bugfix
📝 Description
系分 VisualViewport resize 场景下 Mac 触控板和浏览器窗口的区分, 避免相互冲突设置错误的 DPR
🖼️ Screenshot
🔗 Related issue link
🔍 Self-Check before the merge