Skip to content

Conversation

@sout233
Copy link

@sout233 sout233 commented Mar 23, 2025

After enabling undecorated shadow on Windows, the borders, except for the top, become very thin, making dragging the window difficult.

I have modified the border size to make it easier to grab and drag. However, I've realized that this might not be the best solution.

  • Tested on all platforms changed
  • Added an entry to the changelog module if knowledge of this change could be valuable to users
  • Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • Created or updated an example program if it would help users understand this functionality

sout233 added 2 commits March 23, 2025 15:15
Increases the border size to make it easier to grip the window.
@sout233 sout233 requested a review from notgull as a code owner March 23, 2025 07:46
Comment on lines 1112 to 1114
params.rgrc[0].bottom -= 2;
params.rgrc[0].left += 2;
params.rgrc[0].right -= 2;
Copy link
Member

Choose a reason for hiding this comment

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

in which coordinates it is? logical or physical? I'd assume you generally want to apply scaling for it.

Copy link
Author

Choose a reason for hiding this comment

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

The adjustments might be using logical coordinates. Should I adapt them for DPI scaling?

Copy link
Member

Choose a reason for hiding this comment

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

The point is that if you have DPR 2, the area where you can grip should match logically if you had DPR 1, otherwise on high DPI monitor you won't be able to grab anything.

Copy link
Author

Choose a reason for hiding this comment

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

Got it. I'll try adding DPI scaling tomorrow and test it on a high-DPI monitor (if I can find one).

Copy link
Member

Choose a reason for hiding this comment

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

Just change settings in windows to scale.

Copy link
Author

Choose a reason for hiding this comment

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

I see...thanks!

@madsmtm madsmtm added the DS - win32 Affects the Win32/Windows backend label Apr 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DS - win32 Affects the Win32/Windows backend

Development

Successfully merging this pull request may close these issues.

3 participants