fix(macos): prevent window contents from being stretched when resizing the window #22179
Conversation
There was a problem hiding this comment.
Pull request overview
This PR addresses issue #22159 by fixing window content stretching during resize on macOS. The fix applies two MTKView layer configuration properties that control how the view's content behaves during window resizing.
Key changes:
- Sets
layerContentsPlacementto pin content to the top-left corner - Configures
layerContentsRedrawPolicyto trigger redraws during resize instead of stretching existing content
src/Uno.UI.Runtime.Skia.MacOS/UnoNativeMac/UnoNativeMac/UNOMetalViewDelegate.m
Outdated
Show resolved
Hide resolved
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22179/wasm-skia-net9/index.html |
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-22179/docs/index.html |
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22179/wasm-skia-net9/index.html |
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-22179/docs/index.html |
spouliot
left a comment
There was a problem hiding this comment.
After testing this change breaks when moving the window from a retina display to a normal display - the content is scaled into the top/left quadrant (1/4 sized).
src/Uno.UI.Runtime.Skia.MacOS/UnoNativeMac/UnoNativeMac/UNOMetalViewDelegate.m
Outdated
Show resolved
Hide resolved
- set `layerContentsPlacement` only while resizing - explicitly set scale while resizing (required for lowres) - don't set `layerContentsPlacement` to its default value
spouliot
left a comment
There was a problem hiding this comment.
Fixed switching screen (hi to low res)
|
🤖 Your WebAssembly Skia Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-22179/wasm-skia-net9/index.html |
|
🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-22179/docs/index.html |
GitHub Issue: closes #22159
PR Type:
What is the current behavior? 🤔
What is the new behavior? 🚀
PR Checklist ✅
Please check if your PR fulfills the following requirements:
Screenshots Compare Test Runresults.Other information ℹ️