[DRAFT] Move error and dpi types into a separate crate + split out x11 symbol loading code + xinerama & multihead support #1298
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
goddessfreya
changed the title
goddessfreya
added
C - in progress
|
Hey @simlay, can you implement this for iOS? I couldn't get |
goddessfreya
added
DS - ios
DS - macos
DS - wayland
DS - web
DS - windows
DS - x11
C - waiting on maintainer
|
Sure! I’ll give it a whack! |
This was referenced Dec 1, 2019
|
Weren't we going to merge |
|
@vbogaevsky I don't think it particularly matters which order they land, does it? These changes don't really touch anything dpi related. |
|
@goddessfreya The DPI overhaul branch adds generics to the size and position types, and it'll be easier to manage this change if we delay merging this until after the generics have landed on master. |
goddessfreya
force-pushed
the
winit-glutin-next
branch
from
3db5c37 to
707b04a
Compare
goddessfreya
changed the title
8 tasks
Signed-off-by: Freya Gentz <zegentzy@protonmail.com>
Signed-off-by: Freya Gentz <zegentzy@protonmail.com>
Signed-off-by: Freya Gentz <zegentzy@protonmail.com>
Signed-off-by: Freya Gentz <zegentzy@protonmail.com>
Signed-off-by: Freya Gentz <zegentzy@protonmail.com>
Signed-off-by: Freya Gentz <zegentzy@protonmail.com>
goddessfreya
force-pushed
the
winit-glutin-next
branch
from
54924ef to
a56ee31
Compare
Fixes ci builds so that I can run ci on glutin pr
|
This is very old and needs to be remade completely. Continues to be tracked in #2759. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See https://github.com/rust-windowing/winit_types and https://github.com/rust-windowing/glutin_x11_sym.
@murarth @Osspial Are you folks familiar with how to setup CI for glutin_x11_sym, glutin_sys and winit_types?
Platforms:
Other:
cargo fmthas been run on this branchcargo docbuilds successfullyCHANGELOG.mdif knowledge of this change could be valuable to users