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.

Sign up for GitHub

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

Jump to bottom

Animated SVG pfp replaced with generic pfp. #19557

Closed
nathanielcwm opened this issue Oct 31, 2021 · 1 comment
Closed

Animated SVG pfp replaced with generic pfp. #19557

nathanielcwm opened this issue Oct 31, 2021 · 1 comment
Labels
A-Avatar A-User-Settings O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@nathanielcwm
Copy link

nathanielcwm commented Oct 31, 2021 •
edited
Loading

Steps to reproduce

  1. Download a test image
  2. Open settings and change your pfp to that image
  3. Observe that it displays properly and is animated in settings
    image
  4. Exit out of settings
  5. Observe that your pfp is now replaced by the generic pfp
    image
  6. Observe that CPU usage while having element open is quite high as if it were trying to render the animated svg.
  7. Open settings and observe that the animated svg is no longer displayed as the pfp and only the alt text is visible.
    image

Outcome

What did you expect?

It actually works as a profile picture or it is blocked from being set as the profile picture on save.

What happened instead?

It allows it to be set as the profile picture however in other parts of element it either doesn't display or it switches to the generic blank pfp.

Operating system

Windows 10

Browser information

Firefox 93

URL for webapp

app.element.io

Application version

Element version: 1.9.3 Olm version: 3.2.3

Homeserver

matrix.org

Will you send logs?

Yes
@SimonBrandner SimonBrandner added A-Avatar A-User-Settings O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist labels Oct 31, 2021
@t3chguy
Copy link
Member

t3chguy commented Nov 1, 2021

Duplicate of #2581

Unrelated to them being animated

@t3chguy t3chguy closed this as completed Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Avatar A-User-Settings O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@t3chguy @nathanielcwm @SimonBrandner