Skip to content

Fix initial dimensions of a fullscreen window on Windows#909

Merged
Osspial merged 1 commit intomasterfrom
unknown repository
Jun 13, 2019
Merged

Fix initial dimensions of a fullscreen window on Windows#909
Osspial merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jun 11, 2019

Previously, creating a fullscreen window on Windows would result in the window spanning more than a single monitor since the DPI factor was not taken into account. A workaround was to manually specify the size for the window, but this is no longer necessary.

  • Tested on all platforms changed
  • Added an entry to CHANGELOG.md 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 an example program if it would help users understand this functionality
  • Updated feature matrix, if new features were added or implemented

@elinorbgr
Copy link
Copy Markdown
Contributor

I think this may be in conflict with #895

@ghost
Copy link
Copy Markdown
Author

ghost commented Jun 11, 2019

I think you're right. From a cursory glance at the changes of #895, it would also appear to fix the issue. However, the fullscreen example in that branch appears to be broken, so I cannot test this.

@ghost
Copy link
Copy Markdown
Author

ghost commented Jun 11, 2019

Yes, it looks like #895 now fixes this issue. I'll keep this pull request around in case we want to merge this before #895 lands.

@Osspial Osspial changed the base branch from eventloop-2.0 to master June 13, 2019 05:26
@ghost ghost mentioned this pull request Jun 13, 2019
@ghost
Copy link
Copy Markdown
Author

ghost commented Jun 13, 2019

I've rebased this against master again. It's kind of ridiculous that git is not able to resolve this one-line merge automatically..

@Osspial
Copy link
Copy Markdown
Contributor

Osspial commented Jun 13, 2019

Yeah, git merging on CHANGELOG has always been somewhat shaky, and I'm not entirely sure why...

I wonder if there's any way to change the merge algorithm git uses for that specific file.

@Osspial Osspial merged commit 91f05e9 into rust-windowing:master Jun 13, 2019
kosyak pushed a commit to kosyak/winit that referenced this pull request Jul 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants