knossos icon indicating copy to clipboard operation
knossos copied to clipboard

Disabling svg breaks mods page

Open SFort opened this issue 2 years ago • 6 comments

Environment

Website

Describe the bug

The mods search bar, page and sort by buttons, depend on svg being enabled / the browser supporting svg in order to function

the rest of the website is perfectly functionaly without svg support

Steps To Reproduce

  • open firefox
  • goto about:config
  • set svg.disabled to true

SFort avatar Mar 05 '22 19:03 SFort

Looking into this, I cannot reproduce it in dev, but I can on both https://modrinth.com/mods and https://staging.modrinth.com/mods

In addition, no errors seem to be put in the console or anything, so I am at a loss as to what the problem could be. It's worth noting the sveltekit rewrite seems to work flawlessly with SVG disabled.

Prospector avatar Mar 05 '22 20:03 Prospector

is there any reason why you want to disable svgs?

lucsoft avatar Mar 06 '22 12:03 lucsoft

is there any reason why you want to disable svgs?

personaly i could turn it on, just so happens that a anti-fingerprinting file i imported has them disabled.

having a website stop functioning because an image didn't load is silly and if you care about comatability / accessability it's bad practice to depend on what are advanced browser features for functionality which dosn't even use those features.

SFort avatar Mar 06 '22 15:03 SFort

As far as I can tell, this is entirely due to the LogoAnimated svg which is there while the search data loads

triphora avatar Mar 06 '22 16:03 triphora

image

  1. go to modrinth.com
  2. then navigate to mods

Then you see this error Code TypeError: can't access property "fill", t.style is undefined

This happens while the hydration is going

comatability advanced browser features

we are talking about svg, that's even before chrome existed.

lucsoft avatar Mar 06 '22 16:03 lucsoft

fair "advanced browser features" isn't accurate.

(after now using my browser with svgs enabled i've realised that github especially uses way too many, looks awfull)

SFort avatar Mar 06 '22 16:03 SFort

Going to mark this as not planned. It simply isn't worth the amount of effort it would take to make the site work with SVG disabled, and this is the only report we've ever received of anyone browsing without SVG.

triphora avatar Dec 28 '22 06:12 triphora