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

docs: update Tutorial 2 #472

Merged
merged 8 commits into from
Dec 1, 2023
Merged

docs: update Tutorial 2 #472

merged 8 commits into from
Dec 1, 2023

Conversation

seakrueger
Copy link
Contributor

A couple of mild tweaks I felt would be beneficial after going through Tutorial 2: The Surface.

Copy link
Owner

@sotrh sotrh left a comment

Choose a reason for hiding this comment

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

Just a couple notes

docs/beginner/tutorial2-surface/README.md Show resolved Hide resolved
docs/beginner/tutorial2-surface/README.md Outdated Show resolved Hide resolved
@sotrh
Copy link
Owner

sotrh commented Jul 29, 2023

I've updated to 0.17 so you'll need to resolve some merge conflicts.

seakrueger and others added 6 commits July 30, 2023 12:49
Adds an additional explanation that the `event_loop` needs to be updated after the window has been moved. While this is fixed in the input() section, including this mention in new() allows the program to compile during the new() and resize() section if the reader did not catch this themselves. 

Adds clarity following "changed State to own window" (09207e9) commit
With `wasm-bindgen-futures`  being added as a dependency in tutorial 1 (bbe3d24), it is no longer necessary to add it again in tutorial 2. This section has been reworked as a note stating that WASM would fail to build without it.
@seakrueger
Copy link
Contributor Author

I've rebased and fixed the conflict

docs/beginner/tutorial2-surface/README.md Outdated Show resolved Hide resolved
docs/beginner/tutorial2-surface/README.md Outdated Show resolved Hide resolved
@seakrueger
Copy link
Contributor Author

Updated

@sotrh sotrh merged commit 1f66769 into sotrh:master Dec 1, 2023
1 check passed
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