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

Disable antialiasing when running in Windows Firefox due to possible Firefox bug #5857

Merged
merged 4 commits into from
Nov 28, 2023

Conversation

mvaligursky
Copy link
Contributor

@mvaligursky mvaligursky commented Nov 27, 2023

a workaround for #5856 - disable AA on Windows Firefox

Only handling version 120+ based on userAgent:
Screenshot 2023-11-27 at 15 28 51

When Firefox releases a fix, we'll need to adjust the condition to avoid the workaround from the fixed version.

@mvaligursky mvaligursky self-assigned this Nov 27, 2023
Copy link
Member

@marklundin marklundin left a comment

Choose a reason for hiding this comment

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

This should work, although worth noting that users can disable the useragent in Firefox

src/core/platform.js Outdated Show resolved Hide resolved
@mvaligursky mvaligursky requested a review from slimbuck November 27, 2023 14:29
Copy link
Member

@slimbuck slimbuck left a comment

Choose a reason for hiding this comment

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

Shouldn't we limit to 120 (as opposed to >120) for now?

@mvaligursky
Copy link
Contributor Author

Shouldn't we limit to 120 (as opposed to >120) for now?

We don't know when this will be fixed. It's easier to put a stop version when we confirm the fix, vs patching the engine each time a new broken FF is released.

@tnikkel
Copy link

tnikkel commented Nov 28, 2023

Shouldn't we limit to 120 (as opposed to >120) for now?

We don't know when this will be fixed. It's easier to put a stop version when we confirm the fix, vs patching the engine each time a new broken FF is released.

Firefox 115 esr (basically longer term support) will also have this bug.

@mvaligursky mvaligursky merged commit eb85bdb into main Nov 28, 2023
7 checks passed
@mvaligursky mvaligursky deleted the mv-disable-aa-firefox-windows branch November 28, 2023 11:12
mvaligursky added a commit that referenced this pull request Nov 28, 2023
…Firefox bug (#5857)

* Disable antialiasing when running in Windows Firefox due to possible Firefox bug

* types

* only for firefox 120+

* updated to cover esr115 as well

---------

Co-authored-by: Martin Valigursky <mvaligursky@snapchat.com>
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.

4 participants