Skip to content

Commit

Permalink
[Win] Disallow autoscale mode "integer" for monitor-specific scaling
Browse files Browse the repository at this point in the history
The default auto-scale value is "integer"/"integer200", which makes
everything in the UI except fonts not scale according to the actual
native zoom value but according to a value rounded to 100 or 200. While
most code performing adaptations to zoom considers this for a static
zoom value applied to the whole application, it leads to hard-to-resolve
issues when scaling every shell according to the current monitor's zoom.

Since that autoscale mode is complex and should be replaced by uniform
scaling of everything (in particular when images are based on vector
graphics that can sharply be rendered for every zoom factor) anyway,
this is not to be supported by the monitor-specific scaling feature
currently being introduced for Windows. With this change, that mode will
thus be limited to reasonable autoscale modes like "quarter" and "exact"
or ones fixing the zoom value like specifying an explicit value or
"false".
  • Loading branch information
HeikoKlare committed Jan 14, 2025
1 parent d7be959 commit d3123ef
Show file tree
Hide file tree
Showing 2 changed files with 41 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -650,11 +650,49 @@ public static int getZoomForAutoscaleProperty (int nativeDeviceZoom) {
return zoom;
}

public static boolean isAutoScaleOnRuntimeActive() {
public static boolean isMonitorSpecificScalingActive() {
boolean updateOnRuntimeValue = Boolean.getBoolean (SWT_AUTOSCALE_UPDATE_ON_RUNTIME);
return updateOnRuntimeValue;
}

public static void setAutoScaleForMonitorSpecificScaling() {
boolean isDefaultAutoScale = autoScaleValue == null;
if (isDefaultAutoScale) {
autoScaleValue = "quarter";
} else if (!isSupportedAutoScaleForMonitorSpecificScaling()) {
throw new SWTError(SWT.ERROR_NOT_IMPLEMENTED,
"monitor-specific scaling is only implemented for auto-scale values \"quarter\", \"exact\", \"false\" or a concrete zoom value, but \""
+ autoScaleValue + "\" has been specified");
}
}

/**
* Monitor-specific scaling on Windows only supports auto-scale modes in which
* all elements (font, images, control bounds etc.) are scaled equally or almost
* equally. The previously default mode "integer"/"integer200", which rounded
* the scale factor for everything but fonts to multiples of 100, is complex and
* difficult to realize with monitor-specific rescaling of UI elements. Since a
* uniform scale factor for everything should perspectively be used anyway,
* there will be support for complex auto-scale modes for monitor-specific
* scaling.
*
* The supported modes are "quarter" and "exact" or explicit zoom values given
* by the value itself or "false". Every other value will be treated as
* "integer"/"integer200" and is thus not supported.
*/
private static boolean isSupportedAutoScaleForMonitorSpecificScaling() {
switch (autoScaleValue) {
case "false", "quarter", "exact": return true;
}
try {
Integer.parseInt(autoScaleValue);
return true;
} catch (NumberFormatException e) {
// unsupported value, use default
}
return false;
}

/**
* AutoScale ImageDataProvider.
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -948,8 +948,9 @@ public void close () {
protected void create (DeviceData data) {
checkSubclass ();
checkDisplay (thread = Thread.currentThread (), true);
if (DPIUtil.isAutoScaleOnRuntimeActive()) {
if (DPIUtil.isMonitorSpecificScalingActive()) {
setRescalingAtRuntime(true);
DPIUtil.setAutoScaleForMonitorSpecificScaling();
}
createDisplay (data);
register (this);
Expand Down

0 comments on commit d3123ef

Please sign in to comment.