Re: [PATCH] ui/cocoa: Do not automatically zoom for HiDPI

2024-03-22 Thread Peter Maydell
On Mon, 18 Mar 2024 at 09:02, Akihiko Odaki  wrote:
>
> Cocoa automatically zooms for a HiDPI display like Retina and makes
> the display blurry. Revert the automatic zooming.
>
> Signed-off-by: Akihiko Odaki 
> ---
>  ui/cocoa.m | 9 ++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/ui/cocoa.m b/ui/cocoa.m
> index fa879d7dcd4b..c5b3c28000ff 100644
> --- a/ui/cocoa.m
> +++ b/ui/cocoa.m
> @@ -522,7 +522,10 @@ - (void) resizeWindow
>  [[self window] setContentAspectRatio:NSMakeSize(screen.width, 
> screen.height)];
>
>  if (!([[self window] styleMask] & NSWindowStyleMaskResizable)) {
> -[[self window] setContentSize:NSMakeSize(screen.width, 
> screen.height)];
> +CGFloat width = screen.width / [[self window] backingScaleFactor];
> +CGFloat height = screen.height / [[self window] backingScaleFactor];
> +
> +[[self window] setContentSize:NSMakeSize(width, height)];
>  [[self window] center];
>  } else if ([[self window] styleMask] & NSWindowStyleMaskFullScreen) {
>  [[self window] setContentSize:[self screenSafeAreaSize]];
> @@ -575,8 +578,8 @@ - (void) updateUIInfoLocked
>
>  info.xoff = 0;
>  info.yoff = 0;
> -info.width = frameSize.width;
> -info.height = frameSize.height;
> +info.width = frameSize.width * [[self window] backingScaleFactor];
> +info.height = frameSize.height * [[self window] backingScaleFactor];
>
>  dpy_set_ui_info(dcl.con, , TRUE);
>  }

Could we / should we use convertRectToBacking and convertRectFromBacking
rather than doing the multiply/divide ourselves? The docs seem to
recommend against directly using backingScaleFactor when possible.

thanks
-- PMM



[PATCH] ui/cocoa: Do not automatically zoom for HiDPI

2024-03-18 Thread Akihiko Odaki
Cocoa automatically zooms for a HiDPI display like Retina and makes
the display blurry. Revert the automatic zooming.

Signed-off-by: Akihiko Odaki 
---
 ui/cocoa.m | 9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/ui/cocoa.m b/ui/cocoa.m
index fa879d7dcd4b..c5b3c28000ff 100644
--- a/ui/cocoa.m
+++ b/ui/cocoa.m
@@ -522,7 +522,10 @@ - (void) resizeWindow
 [[self window] setContentAspectRatio:NSMakeSize(screen.width, 
screen.height)];
 
 if (!([[self window] styleMask] & NSWindowStyleMaskResizable)) {
-[[self window] setContentSize:NSMakeSize(screen.width, screen.height)];
+CGFloat width = screen.width / [[self window] backingScaleFactor];
+CGFloat height = screen.height / [[self window] backingScaleFactor];
+
+[[self window] setContentSize:NSMakeSize(width, height)];
 [[self window] center];
 } else if ([[self window] styleMask] & NSWindowStyleMaskFullScreen) {
 [[self window] setContentSize:[self screenSafeAreaSize]];
@@ -575,8 +578,8 @@ - (void) updateUIInfoLocked
 
 info.xoff = 0;
 info.yoff = 0;
-info.width = frameSize.width;
-info.height = frameSize.height;
+info.width = frameSize.width * [[self window] backingScaleFactor];
+info.height = frameSize.height * [[self window] backingScaleFactor];
 
 dpy_set_ui_info(dcl.con, , TRUE);
 }

---
base-commit: ba49d760eb04630e7b15f423ebecf6c871b8f77b
change-id: 20240318-zoom-df4d6834e56b

Best regards,
-- 
Akihiko Odaki