-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
egui-wgpu: Fix docs.rs build #5204
egui-wgpu: Fix docs.rs build #5204
Conversation
Preview available at https://egui-pr-preview.github.io/pr/5204-lucas/5202-egui-wgpu-docs-build-is-failing-since-029 |
@@ -37,7 +37,7 @@ default = ["fragile-send-sync-non-atomic-wasm"] | |||
puffin = ["dep:puffin"] | |||
|
|||
## Enable [`winit`](https://docs.rs/winit) integration. On Linux, requires either `wayland` or `x11` | |||
winit = ["dep:winit"] | |||
winit = ["dep:winit", "winit/rwh_06"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could do the same as eframe for consistency, although it's the same result in the end.
Line 149 in 24cd711
winit = { workspace = true, default-features = false, features = ["rwh_06"] } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking like it's already enabled for winit in eframe? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! 👍
This enables the rwh_06 feature in winit which is required to correctly build egui-wgpu - fixes emilk#5202
This enables the rwh_06 feature in winit which is required to correctly build egui-wgpu