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

Device getDPI scaling fix #1646

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Conversation

amartya4256
Copy link
Contributor

@amartya4256 amartya4256 commented Dec 11, 2024

This contribution fixes the device getDPI calculation logic by usibg the device zoom to be used for scaling down the DPI values instead of the native zoom.

contributes to #62 and #127

Fixes #1671

This commit enforces device zoom to be used for scaling down the DPI
values instead of the native zoom.

contributes to eclipse-platform#62 and eclipse-platform#127
Copy link
Contributor

Test Results

   383 files  ±0     383 suites  ±0   5m 16s ⏱️ +23s
 4 286 tests ±0   4 272 ✅ ±0  14 💤 ±0  0 ❌ ±0 
12 150 runs  ±0  12 065 ✅ ±0  85 💤 ±0  0 ❌ ±0 

Results for commit a556188. ± Comparison against base commit 2a61cb2.

@jukzi
Copy link
Contributor

jukzi commented Dec 12, 2024

Until CI and I Builds are green again please hold back any submit that does not contribute to that.

@tmssngr
Copy link
Contributor

tmssngr commented Dec 19, 2024

This fixes #1671 for me. Thanks.

@fedejeanne
Copy link
Contributor

Test failures are unrelated

@fedejeanne fedejeanne merged commit 4621729 into eclipse-platform:master Dec 19, 2024
9 of 14 checks passed
@akurtakov
Copy link
Member

@fedejeanne I see more test failures and with different errors like https://ci.eclipse.org/releng/job/eclipse.platform.swt/job/PR-1646/1/testReport/junit/org.eclipse.swt.tests.junit/Test_org_eclipse_swt_browser_Browser/test_BrowserFunction_callback_with_boolean_browser_flags__0_/ . Would you please investigate whether it comes from this PR or some of the others just merged (I haven't investigated which one exactly caused it).

@fedejeanne
Copy link
Contributor

fedejeanne commented Dec 19, 2024

Not in this PR.
In any case, that is also one of the "usual suspects": #1676

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.

[Win32] Broken compatibility with swt.autoScale=false Analyse Display::getDPI contract
5 participants