Fixed Shell::setBounds scaling

amartya4256 · amartya4256 · commit 2decaac49820 · 2025-01-17T15:21:44.000+01:00
This commit contributes to the correct scaling of the Shell using its own zoom on Shell::setBounds call as the scaling of the shell in case of a monitor change is directly handled by the DPI_CHANGED event. contributes to #62 and #127
diff --git a/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Shell.java b/bundles/org.eclipse.swt/Eclipse SWT/win32/org/eclipse/swt/widgets/Shell.java
@@ -1596,7 +1596,7 @@ public void setBounds(Rectangle rect) {
 	if (rect == null) error (SWT.ERROR_NULL_ARGUMENT);
 	checkWidget ();
 	Rectangle boundsInPixels = getDisplay().translateToDisplayCoordinates(rect, getZoom());
-	setBoundsInPixels(boundsInPixels.x, boundsInPixels.y, boundsInPixels.width, boundsInPixels.height);
+	setBoundsInPixels(boundsInPixels.x, boundsInPixels.y, DPIUtil.scaleUp(rect.width, getZoom()), DPIUtil.scaleUp(rect.height, getZoom()));
 }
 
 @Override

Original file line number	Diff line number	Diff line change
`@@ -1596,7 +1596,7 @@ public void setBounds(Rectangle rect) {`
`1596`	`1596`	`if (rect == null) error (SWT.ERROR_NULL_ARGUMENT);`
`1597`	`1597`	`checkWidget ();`
`1598`	`1598`	`Rectangle boundsInPixels = getDisplay().translateToDisplayCoordinates(rect, getZoom());`
`1599`		`- setBoundsInPixels(boundsInPixels.x, boundsInPixels.y, boundsInPixels.width, boundsInPixels.height);`
	`1599`	`+ setBoundsInPixels(boundsInPixels.x, boundsInPixels.y, DPIUtil.scaleUp(rect.width, getZoom()), DPIUtil.scaleUp(rect.height, getZoom()));`
`1600`	`1600`	`}`
`1601`	`1601`
`1602`	`1602`	`@Override`