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 SVG image compatible with Firefox #3466

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

webknjaz
Copy link
Contributor

It appears that Firefox has bugs around interpreting <feBlend> filters in SVG documents. This makes it render the logo as a transparent image in the docs, both in the mobile and the desktop versions.

This patch applies a workaround [1] found on the internet to make it work without waiting for Firefox to fix their bug.

Resolves #3465.

  • ran the linter to address style issues (tox -e fix)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • updated/extended the documentation

Sorry, something went wrong.

It appears that Firefox has bugs around interpreting `<feBlend>`
filters in SVG documents. This makes it render the logo as a
transparent image in the docs, both in the mobile and the desktop
versions.

This patch applies a workaround [[1]] found on the internet to make it
work without waiting for Firefox to fix their bug.

Resolves tox-dev#3465.

[1]: svg/svgo#732
@webknjaz
Copy link
Contributor Author

@gaborbernat the failure is unrelated (setup-uv's broken?). Plz also apply the change note skip label since I don't think it's needed.

@gaborbernat gaborbernat enabled auto-merge (squash) January 16, 2025 16:32
@gaborbernat gaborbernat merged commit fccbe2a into tox-dev:main Jan 16, 2025
28 checks passed
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.

SVG logo in the docs does not render in Firefox
2 participants