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

Make `canvas_minimal` example rerender on refresh #404

Merged
merged 2 commits into from Jul 23, 2020

Conversation

@leroycep
Copy link
Contributor

leroycep commented Jul 17, 2020

I'm not sure if this actually works, because the canvas_minimal example doesn't run on my computer.

@pcwalton
Copy link
Collaborator

pcwalton commented Jul 22, 2020

This breaks on macOS because winit doesn't send an initial Refresh event there.

@leroycep
Copy link
Contributor Author

leroycep commented Jul 22, 2020

Hmm, I can't find anything that looks like a "window shown" event. I think I'll move the rendering code again, and then use boolean to indicate that the screen needs to be redrawn.

@pcwalton
Copy link
Collaborator

pcwalton commented Jul 23, 2020

@bors-servo
Copy link
Contributor

bors-servo commented Jul 23, 2020

📌 Commit 263dc0a has been approved by pcwalton

@bors-servo
Copy link
Contributor

bors-servo commented Jul 23, 2020

Testing commit 263dc0a with merge 953a1da...

@bors-servo
Copy link
Contributor

bors-servo commented Jul 23, 2020

☀️ Test successful - checks-travis
Approved by: pcwalton
Pushing 953a1da to master...

@bors-servo bors-servo merged commit 953a1da into servo:master Jul 23, 2020
3 checks passed
3 checks passed
Travis CI - Pull Request Build Passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.