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

started x11 #2

Merged
merged 2 commits into from
Oct 13, 2021
Merged

started x11 #2

merged 2 commits into from
Oct 13, 2021

Conversation

levovix0
Copy link
Contributor

No description provided.

@guzba
Copy link
Collaborator

guzba commented Oct 12, 2021

Hello and thanks for the PR! This looks great. I expect we'll get this merged asap.

For some context, we are just getting started on Windy so things are in flux. We have not figured out how we want using Windy to work (the API), how events, callbacks, ets will work, how handling multiple windows will work. Lots of open questions at this point. This means there will probably be a lot of API churn as we sort that stuff out.

The core things though like "how does one correctly open a window" and "how do we create an OpeGL context of the correct version" etc won't really change though of course. I hope having this bit of context will help if you see us changing things and moving things and wonder why. If you want to help contribute with more PRs, we welcome your help and will figure out how to work together well!

@levovix0
Copy link
Contributor Author

I would glad to help!

I have already written a lot of code dealing with window events, I will not consider it difficult to adapt it for windy.

@treeform
Copy link
Owner

This looks great! Thank you. Merging.

@treeform treeform merged commit 7cd9e61 into treeform:master Oct 13, 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.

3 participants