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

Read initial window size and position from game.project #124

Merged
merged 3 commits into from
Oct 18, 2021

Conversation

AGulev
Copy link
Collaborator

@AGulev AGulev commented Sep 19, 2021

Pls don't merge before releasing Defold with this fix (PR)

@subsoap
Copy link
Owner

subsoap commented Sep 19, 2021

Looks good, you can merge whenever the Defold fix happens.

@dapetcu21
Copy link
Collaborator

I would make this set the view size, not the window size, since the window size also includes window chrome (titlebar, window borders, etc.)

@AGulev
Copy link
Collaborator Author

AGulev commented Sep 20, 2021

@dapetcu21 I didn't know about this method, I'll take a look. Thank you!

@AGulev
Copy link
Collaborator Author

AGulev commented Sep 21, 2021

Could somebody test in on windows? (on mac it works perfect) http://d.defold.com/alpha/

@AGulev
Copy link
Collaborator Author

AGulev commented Sep 21, 2021

@dapetcu21 I use set_view* instead of set_window*

@AGulev
Copy link
Collaborator Author

AGulev commented Sep 21, 2021

Sep-21-2021 18-32-42

@dapetcu21
Copy link
Collaborator

I think you forgot to change the name of the props. You only changed them in the README

@AGulev
Copy link
Collaborator Author

AGulev commented Sep 22, 2021

🙉 fixed!

@subsoap
Copy link
Owner

subsoap commented Oct 18, 2021

@AGulev AGulev merged commit 64f9000 into subsoap:master Oct 18, 2021
@AGulev
Copy link
Collaborator Author

AGulev commented Oct 18, 2021

@subsoap yes, thank you

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