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

Some svg files can't be recognized for previewing #29432

Closed
xuhdev opened this issue Mar 6, 2025 · 6 comments
Closed

Some svg files can't be recognized for previewing #29432

xuhdev opened this issue Mar 6, 2025 · 6 comments
Labels
A-Media 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 X-Cannot-Reproduce

Comments

@xuhdev
Copy link

xuhdev commented Mar 6, 2025

Steps to reproduce

  1. Log into matrix.io. Create a new room.
  2. Upload this svg file: https://raw.githubusercontent.com/element-hq/logos/382dadaf56fbcc12cf1cd950781ed8edef85fb3a/element/Element%20Logomark.svg
  3. The svg file isn't recognized. It is treated as a normal file and the file is shown as an attachment.

A different svg file, such as https://raw.githubusercontent.com/element-hq/logos/382dadaf56fbcc12cf1cd950781ed8edef85fb3a/matrix/matrix%20favicon.svg doesn't suffer from this problem.

Outcome

What did you expect?

The svg file should display.

What happened instead?

The svg file isn't recognized.

Operating system

No response

Browser information

Chrome Version 133.0.6943.141 (Official Build) (64-bit)

URL for webapp

app.element.io

Application version

Element version: aa99601-js-3e512711d7ac Crypto version: Rust SDK 0.10.0 (3cc301d), Vodozemac 0.9.0

Homeserver

No response

Will you send logs?

Yes

@xuhdev xuhdev added the T-Defect label Mar 6, 2025
@dosubot dosubot bot added A-Media O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist labels Mar 6, 2025
@t3chguy
Copy link
Member

t3chguy commented Mar 6, 2025

Will you send logs?
Yes

Not seeing any logs from you

@t3chguy t3chguy added the X-Needs-Info This issue is blocked awaiting information from the reporter label Mar 6, 2025
@xuhdev
Copy link
Author

xuhdev commented Mar 6, 2025

@t3chguy Log sent, sorry for the confusion

@t3chguy
Copy link
Member

t3chguy commented Mar 6, 2025

Image

Cannot reproduce - they both look fine for me

@t3chguy t3chguy added X-Cannot-Reproduce and removed X-Needs-Info This issue is blocked awaiting information from the reporter labels Mar 6, 2025
@xuhdev
Copy link
Author

xuhdev commented Mar 6, 2025

@t3chguy Did you enable end-to-end encryption for the room?

@t3chguy
Copy link
Member

t3chguy commented Mar 7, 2025

No, where in your repro steps does it say that?

Image

@xuhdev
Copy link
Author

xuhdev commented Mar 7, 2025

Sorry for the noise here, I mixed up bug reports.

As I dug through this, it turned out this is not an element bug. The issue on my system was caused by a Firefox bug that changed the svg file in an unexpected way when downloading the svg file: https://bugzilla.mozilla.org/show_bug.cgi?id=1952411

@xuhdev xuhdev closed this as not planned Won't fix, can't repro, duplicate, stale Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Media 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 X-Cannot-Reproduce
Projects
None yet
Development

No branches or pull requests

2 participants