Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove rounded corners on emulation render window #12743

Merged

Conversation

Filoppi
Copy link
Contributor

@Filoppi Filoppi commented Apr 28, 2024

On Windows 11, when playing windowed in a separate window/widget from the main emulator window, we don't want the window to have rounded corners, as it prevents the corner pixels from being visible

Here's how it would look, there's not much to it:
image
image

@Filoppi Filoppi force-pushed the remove-round-corners-on-game-window branch from 089cc6a to a816915 Compare April 28, 2024 01:32
@iwubcode
Copy link
Contributor

Can you post a before/after screenshot?

On Windows 11, when playing windowed in a separate window/widget from the main emulator window, we don't want the window to have rounded corners, as it prevents the corner pixels from being visible
@Filoppi Filoppi force-pushed the remove-round-corners-on-game-window branch from a816915 to 9106704 Compare April 28, 2024 09:41
@BhaaLseN
Copy link
Member

Does this cause any issues on Windows versions before 11? Or do they just silently ignore the flag?

@Filoppi
Copy link
Contributor Author

Filoppi commented Apr 28, 2024

The flag is just ignored

@MayImilae
Copy link
Contributor

I am extremely happy with this. I wish we could do the same on macOS!

Copy link
Contributor

@iwubcode iwubcode left a comment

Choose a reason for hiding this comment

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

LGTM. Tested on Win11 as confirmed working.

@MayImilae
Copy link
Contributor

MayImilae commented Apr 28, 2024

For progress report reasons, please hold off merging this until the report goes up in roughly 36 hours from this comment. I absolutely want that in the next report and launching now would make versioning the beta messier.

Progress Report is done and we're in the clear!

Copy link
Contributor

@MayImilae MayImilae left a comment

Choose a reason for hiding this comment

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

Untested, but everything LGTM.

@JMC47 JMC47 merged commit 2b261b9 into dolphin-emu:master Apr 30, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants