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

fix: compatibility with chrome 51 #454

Merged

Conversation

pecoram
Copy link
Contributor

@pecoram pecoram commented Nov 22, 2024

  • added alternative to roundRect it not supported
  • added WebGlContext error message

@pecoram pecoram changed the title Fix/compatibility-with-chrome-51 fix: compatibility with chrome 51 Nov 22, 2024
@wouterlucas wouterlucas mentioned this pull request Nov 25, 2024
github-merge-queue bot pushed a commit that referenced this pull request Nov 27, 2024
Fixes for handling base64 src strings on Chrome v38, rework Image
texture fallback & detection.

This PR includes:
* Using a 1x1 PNG pixel to determine which `createImageBitmap` signature
is supported on the browser
* Fallback the image worker and main thread `ImageTexture` handling
according to the detected signature
* Support for base64 `src` on Chrome v38
* Support for `createImageBitmap(blob)` without a config object for
Chrome v50 (linked to #454)
* Support for ES5 legacy output of the `examples` directory (will be
published on https://lightning-js.github.io/renderer/ automatically)
* Upgrade `vite` and `vitest` to 2.x devDependencies, pin to TS 5.6.x
for now (as 5.7.x has issues with other dependencies)
* Upgrade Playwright 1.39 to 1.49 for automated tests
* Refactor Visual Regression test pixel matching and snapshot handling
by using [pixelmatch](https://www.npmjs.com/package/pixelmatch) and
[pngjs](https://www.npmjs.com/package/pngjs) over a homebrew detection +
upng (which is stale).
* SDF font initialization waits for image support detection before
uploading the SDF texture atlas.
* Added documentation on the fallback process and references to chrome
versions in `BROWSERS.md`
@pecoram pecoram marked this pull request as ready for review November 28, 2024 10:43
Copy link
Contributor

@wouterlucas wouterlucas left a comment

Choose a reason for hiding this comment

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

LGTM

@wouterlucas wouterlucas merged commit 18de7ad into lightning-js:main Nov 28, 2024
2 checks 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