Skip to content

Correct some typos/out-of-sync in docs#669

Merged
sotrh merged 4 commits into
sotrh:masterfrom
mud2monarch:m2m-typos
May 8, 2026
Merged

Correct some typos/out-of-sync in docs#669
sotrh merged 4 commits into
sotrh:masterfrom
mud2monarch:m2m-typos

Conversation

@mud2monarch
Copy link
Copy Markdown
Contributor

Going through this tutorial and greatly appreciate it!!

Most of it is up-to-date, but some parts are out-of-sync or caused bugs.

Includes a bug fix for a bug I ran into. This is probably common on Macs with Retina displays, which have a default resolution of 3024 x 1964 for the 14"

This is where we configure the `surface`. We need the surface to be configured before we can do anything with it. We set the `is_surface_configured` flag to true here and we'll check it in the `render()` function.

Note that the max supported dimension in WebGL is 2048 px. If you have a larger display, you must cap the height and width, e.g.:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Relevant for people with big monitors, also relevant for people with high-res displays, e.g., this is a bug I encountered on my M4 MBP

queue: wgpu::Queue,
config: wgpu::SurfaceConfiguration,
is_surface_configured: bool,
window: Arc<Window>,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We added the window field in Tutorial2

@mud2monarch
Copy link
Copy Markdown
Contributor Author

@sotrh

@sotrh sotrh merged commit 008df3c into sotrh:master May 8, 2026
@mud2monarch mud2monarch deleted the m2m-typos branch May 8, 2026 15:23
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