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

Improve styling of the example/s graphics API callout #1918

Merged
merged 11 commits into from
Jan 29, 2025

Conversation

doup
Copy link
Contributor

@doup doup commented Dec 11, 2024

Examples

  • Use callout component to style the web graphics API warning.
image image

Example

  • Use callout component to style the web graphics API warning.
  • Add callout--plain modifier that hides the callout top border.
  • Change link to it goes directly to the example and not the listing.
image image

@mockersf
Copy link
Member

if you're touching this callout, something that I would like is a way to switch between webgl2/webgpu on the same example without going through the index page

@doup
Copy link
Contributor Author

doup commented Dec 11, 2024

Makes sense, I've updated the example page link.

@doup doup changed the title Improve styling of the example web graphics API callout Improve styling of the example graphics API callout Dec 13, 2024
@doup doup changed the title Improve styling of the example graphics API callout Improve styling of the example/s graphics API callout Dec 14, 2024
@doup doup marked this pull request as ready for review December 14, 2024 00:20
Copy link
Contributor

@rparrett rparrett left a comment

Choose a reason for hiding this comment

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

Looks good. Love the new links.

Comment on lines +10 to +11
WebGPU is currently only supported on Chrome starting with version 113, and only on desktop.
If they don't work on your configuration, you can check the WebGL2 examples <a href="/examples">here</a>.
Copy link
Member

@mockersf mockersf Jan 28, 2025

Choose a reason for hiding this comment

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

for a followup PR: this is no longer true, WebGPU also works on some mobile browsers, and on Safari TP, and Firefox Nightly
We should probably link to something like https://caniuse.com/webgpu for a list or https://browserleaks.com/webgpu so that user can test their browser instead of keeping a list ourselves

@mockersf mockersf added this pull request to the merge queue Jan 28, 2025
Merged via the queue into bevyengine:main with commit 79d53da Jan 29, 2025
10 checks passed
@doup doup deleted the example-callout branch January 30, 2025 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants