Skip to content
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

[suggestion] bevy_egui support #7

Closed
robbiemu opened this issue Jun 14, 2023 · 5 comments
Closed

[suggestion] bevy_egui support #7

robbiemu opened this issue Jun 14, 2023 · 5 comments

Comments

@robbiemu
Copy link

I was hoping I could drop this into a bevy_egui app, which I am building to learn bevy (perhaps I'm the wrong person to get very detailed into what people would want!). But this indicates:

error[E0308]: mismatched types
--> src/menu.rs:100:14
|
100 | Modal::new(contexts.ctx_mut(), LOAD_MAP_MODAL_TITLE).show(|ui| {
| ---------- ^^^^^^^^^^^^^^^^^^ expected &Context, found &mut Context
| |
| arguments to this function are incorrect
|
= note: bevy_egui::egui::Context and egui::context::Context have similar names, but are actually distinct types

@n00kii
Copy link
Owner

n00kii commented Jun 14, 2023

what version of bevy_egui are you using? its likely that it is reexporting a different version of egui

@n00kii
Copy link
Owner

n00kii commented Jun 14, 2023

it seems the latest version of bevy_egui uses egui = "0.21.0", but the latest version of this crate uses egui = "0.22.0"

there seems to be an open pull request for updating the dep here though im not sure when/how the bevy_egui team will get to that

in the meantime, perhaps you could specify the updated branch as a git dependancy

@n00kii
Copy link
Owner

n00kii commented Jun 20, 2023

closing to inactivity

@n00kii n00kii closed this as completed Jun 20, 2023
@calteran
Copy link

PR #184 hasn't been merged yet. I'm having this same error with updated bevy_notify and bevy_inspector_egui. Perhaps this issue should be reopened for visibility?

@n00kii n00kii reopened this Jun 27, 2023
@n00kii
Copy link
Owner

n00kii commented Jul 22, 2023

seems like its been merged, closing again

@n00kii n00kii closed this as completed Jul 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants