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

Embed SVG delete button icon in an IE 11 compatible way #83

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

sabieber
Copy link
Member

Description

IE11 doesn't seem to like utf-8 embedded SVGs as background images, so we switch to base64 encoding them.

Before:

image

After:

image

Additional Notes

  • This PR fixes or works on following ticket(s): OX-11077

Checklist

  • Code change has been tested and works locally
  • TSC (TypeScript Compiler) has been run and changes are reflected in the dist folder

IE11 doesn't seem to like utf-8 embedded SVGs as background images, so we switch to base64 encoding them.

Fixes: OX-11077
@sabieber sabieber added 🐛 Bugfix Contains only a small fix for an existing bug 👶🏻 Trivial Easy to review labels Jun 25, 2024
@sabieber sabieber merged commit 162ec88 into master Jun 25, 2024
@sabieber sabieber deleted the feature/sbi/OX-11077 branch June 25, 2024 13:27
sabieber added a commit to scireum/sirius-web that referenced this pull request Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bugfix Contains only a small fix for an existing bug 👶🏻 Trivial Easy to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants