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

Support winit-0.26 #583

Merged
merged 2 commits into from Dec 22, 2021
Merged

Support winit-0.26 #583

merged 2 commits into from Dec 22, 2021

Conversation

cwfitzgerald
Copy link
Contributor

This adds winit 0.26 support to imgui-winit-support as non-default. This is a non-breaking change and I've tested all versions of winit with the crate.

I have bumped the version number, as it would be greatly appreciated a patch version could be released. This is currently the last blocking piece of updating my project to winit 0.26.

@sanbox-irl
Copy link
Member

Thanks -- I need to release a patch anyway. I'll try to get that done on Sunday night

@cwfitzgerald
Copy link
Contributor Author

Great, thanks!

@thomcc
Copy link
Member

thomcc commented Dec 18, 2021

@dbr
Copy link
Contributor

dbr commented Dec 18, 2021

Along with the additions Thom mentions - if the version number is left untouched it'd make it easier to merge in, then we can cherry-pick the change into the branch referenced in #581 (which includes my attempts at backporting the text-input bugfixs)

@dbr dbr added this to the v0.8.1 milestone Dec 18, 2021
@cwfitzgerald
Copy link
Contributor Author

All three things changed!

@sanbox-irl
Copy link
Member

going to merge this in once it goes green

@sanbox-irl sanbox-irl merged commit e999579 into imgui-rs:main Dec 22, 2021
@sanbox-irl
Copy link
Member

thanks @cwfitzgerald !

@thomcc
Copy link
Member

thomcc commented Dec 24, 2021

We should remember to move from 0.25-by-default to 0.26-by-default on the next semver-major change of imgui-winit-support

@dbr dbr mentioned this pull request Dec 24, 2021
@cwfitzgerald cwfitzgerald deleted the winit-0.26 branch December 25, 2021 06:17
@parasyte parasyte mentioned this pull request Dec 30, 2021
9 tasks
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.

None yet

4 participants