-
-
Notifications
You must be signed in to change notification settings - Fork 55
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 supports #122
base: main
Are you sure you want to change the base?
Egui supports #122
Conversation
Signed-off-by: Amirhossein Akhlaghpour <m9.akhlaghpoor@gmail.com>
Signed-off-by: Amirhossein Akhlaghpour <m9.akhlaghpoor@gmail.com>
Hi! 👋 I'm not sure if I understand why we'd need to support egui directly in glyphon. It seems like matching the wgpu version should be enough to use glyphon with egui already? Please let me know if I'm missing something! |
Hi @grovesNL , I'll explain my situation as an example : |
Ok thanks for the context. I think you should be able to use glyphon without any changes by making sure the wgpu version matches between glyohob and egui. Does that not work for you? |
As I mentioned egui_wgpu and wgpu are similar crates , but can't use wgpu directly in wgpu not compatible and mismatch types error will occur. we have to use egui_wgpu instead . |
That seems ok though - are you able to use glyphon with egui_wgpu? I don't think we'll be able to depend on any UI crates directly in glyphon. You could look at iced to see how glyphon support works there as an example. |
@grovesNL |
I mean you should be able to use them together like you're doing without the changes in this PR, if the wgpu versions are the same. |
@grovesNL |
@Mehrbod2002 have you checked some of the other approaches like https://github.com/StratusFearMe21/egui-glyphon ? This doesn't seem to require changes to glyphon. |
Support egui as text renderer based on egui-wgpu
https://github.com/emilk/egui/tree/master/crates/egui-wgpu