Skip to content

Conversation

@cppcoffee
Copy link
Contributor

@cppcoffee cppcoffee commented Oct 12, 2025

The WM_DPICHANGED suggested RECT is calculated for non-maximized windows. When a maximized window's DPI changes, we now query the monitor's work area directly to ensure the window correctly fills the entire screen.

For non-maximized windows, the original behavior using the system-suggested RECT is preserved.

Release Notes:

  • windows: Fixed maximized window size when DPI scale changes

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Oct 12, 2025
@reflectronic reflectronic self-assigned this Oct 12, 2025
The WM_DPICHANGED suggested RECT is calculated for non-maximized windows.
When a maximized window's DPI changes, we now query the monitor's work area
directly to ensure the window correctly fills the entire screen.

For non-maximized windows, the original behavior using the system-suggested
RECT is preserved.

Signed-off-by: Xiaobo Liu <cppcoffee@gmail.com>
Copy link
Member

@reflectronic reflectronic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apologies for the delay. This patch appears to work for me. I am not quite convinced it's the right solution--I haven't seen other applications implementing a similar workaround. I think we are doing something wrong with our borderless window handling, and this patch is just working around those issues. But this seems like a fine change to have until that is fully investigated.

@reflectronic reflectronic enabled auto-merge (squash) November 20, 2025 20:30
@reflectronic reflectronic merged commit b41eb3c into zed-industries:main Nov 20, 2025
24 checks passed
mikayla-maki pushed a commit that referenced this pull request Nov 20, 2025
The WM_DPICHANGED suggested RECT is calculated for non-maximized
windows. When a maximized window's DPI changes, we now query the
monitor's work area directly to ensure the window correctly fills the
entire screen.

For non-maximized windows, the original behavior using the
system-suggested RECT is preserved.

Release Notes:

- windows: Fixed maximized window size when DPI scale changes

Signed-off-by: Xiaobo Liu <cppcoffee@gmail.com>
@cppcoffee cppcoffee deleted the rect branch November 21, 2025 01:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants