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

Add HasRawWindowHandle for WindowHandle in druid #1828

Merged
merged 1 commit into from
Jul 23, 2021

Conversation

djeedai
Copy link
Contributor

@djeedai djeedai commented Jun 20, 2021

Expose the HasRawWindowHandle trait impl of druid-shell for
WindowHandle into druid itself, so druid users can access it. This
is also under a raw-win-handle feature, like the druid-shell one.

Bug: #1505

@djeedai
Copy link
Contributor Author

djeedai commented Jun 20, 2021

Note: CI failing due to clippy and existing changes in master, not from this PR. Probably a new warning from 1.53 since #1822 had green CI 13 days ago on its last commit, but went red after it was merged to master.

@maan2003
Copy link
Collaborator

rebasing should fix CI

@playmer
Copy link

playmer commented Jul 18, 2021

Any way to nudge this along? I've kicked the tires on this a little bit and it's very useful! I'd love to be able to switch back to linebender/druid.

@cmyr
Copy link
Member

cmyr commented Jul 19, 2021

@playmer Feel free to cherry-pick this commit into a new PR that is based off the current master? I will be happy to merge that.

@playmer
Copy link

playmer commented Jul 20, 2021

@playmer Feel free to cherry-pick this commit into a new PR that is based off the current master? I will be happy to merge that.

Already had done so for my own testing :) here's a PR #1878

Appreciate it!

@djeedai
Copy link
Contributor Author

djeedai commented Jul 20, 2021

I'll have a look at rebasing.

@cmyr
Copy link
Member

cmyr commented Jul 20, 2021

@djeedai sure, I think it should be pretty straight-forward. I'll give you a bit of time to do that, and otherwise I'll merge #1878.

Expose the `HasRawWindowHandle` trait impl of druid-shell for
`WindowHandle` into druid itself, so druid users can access it. This
is also under a `raw-win-handle` feature, like the druid-shell one.

Bug: linebender#1505
@djeedai
Copy link
Contributor Author

djeedai commented Jul 22, 2021

What the heck happened? I just pushed to my branch and that closed the PR?!

@djeedai djeedai reopened this Jul 22, 2021
@djeedai
Copy link
Contributor Author

djeedai commented Jul 22, 2021

All green @cmyr, if someone can review/merge. Thanks!

Copy link
Member

@cmyr cmyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@cmyr cmyr merged commit a51e19e into linebender:master Jul 23, 2021
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.

4 participants