Skip to content

Move available_monitors and primary_monitor to EventLoopWindowTarget#1616

Merged
Osspial merged 6 commits intorust-windowing:masterfrom
AdminXVII:move-available_monitors
Jul 4, 2020
Merged

Move available_monitors and primary_monitor to EventLoopWindowTarget#1616
Osspial merged 6 commits intorust-windowing:masterfrom
AdminXVII:move-available_monitors

Conversation

@AdminXVII
Copy link
Contributor

@AdminXVII AdminXVII commented Jul 3, 2020

  • Tested on all platforms changed
  • Compilation warnings were addressed
  • cargo fmt has been run on this branch
  • cargo doc builds successfully
  • 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 or updated an example program if it would help users understand this functionality
  • Updated feature matrix, if new features were added or implemented

I only have Linux machines, so the other platforms where not tested. This should not change behaviour though.

Fix #1566

@AdminXVII AdminXVII changed the title First stab at moving available_monitors and primary_monitor to EventLoopWindowTarget Move available_monitors and primary_monitor to EventLoopWindowTarget Jul 3, 2020
@Osspial
Copy link
Contributor

Osspial commented Jul 4, 2020

Looks good to me. Thanks for pulling this together!

@Osspial Osspial merged commit 3d5d05e into rust-windowing:master Jul 4, 2020
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.

available_monitors on EventLoopWindowTarget

2 participants