Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@emilk
Copy link
Owner

@emilk emilk commented Oct 18, 2021

Follow-up to #685

Is there a better way to do pick-exactly-one for some optional dependencies in eframe/Cargo.toml?

TODO: add instructions on how to use egui_glow to eframe/README.md

@AlexApps99
Copy link
Contributor

Is there a better way to do pick-exactly-one for some optional dependencies in eframe/Cargo.toml?

That's the main reason why I didn't address this in the main PR, I'm not really sure how to do that elegantly.
Still not sure, as a last resort we could do separate flags for egui_glow, like glow_persistence etc, but that's quite a bad compromise to make and would be very confusing for users.

@emilk
Copy link
Owner Author

emilk commented Oct 19, 2021

I think this is a good enough start

@emilk emilk merged commit cdd4dcc into master Oct 19, 2021
@emilk emilk deleted the optional-glow-eframe branch October 19, 2021 13:32
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

Successfully merging this pull request may close these issues.

2 participants