Skip to content

Conversation

sandhose
Copy link
Member

@sandhose sandhose commented Sep 3, 2025

I had the issue in another project with this exact same background. The SVG is really expensive to render, as it's a gaussian blur of 200px. This is noticeable on loading, but especially when resizing the window. This is quite bad on safari, but especially on firefox.

Here's a recording before/after on safari:

ec-bg.mp4

@toger5
Copy link
Contributor

toger5 commented Sep 9, 2025

Thanks @sandhose. This looks way better indeed. It does look very different to the svg version so this needs a design review to see if the png version is desired.

@sandhose
Copy link
Member Author

sandhose commented Sep 9, 2025

Is it really that different? It's closer to the Figma designs, and on safari the rendering differences are negligible. I think the biggest difference is in Firefox, but I would say that the SVG version in Firefox is the one currently most far off the designs?

@toger5
Copy link
Contributor

toger5 commented Sep 9, 2025

Yes I was testing on FF.
The PNG also seems a bit grainy. Is that intentional?

@sandhose
Copy link
Member Author

sandhose commented Sep 9, 2025

The PNG also seems a bit grainy. Is that intentional?

It is very low res; I can probably double the resolution if it's too noticeable

@bmarty
Copy link
Member

bmarty commented Sep 9, 2025

My 2 cents: I prefer SVG as it's a vector format. Would it be possible, to mitigate the white stripes when increasing the browser window size to apply another plain dark color background? Or maybe render the background to 120% (for instance) of the window size and centered so that we have room to increase the browser size without viewing the white stripes?

@toger5 toger5 added the PR-Improvement Release note category. A PR that improves EC's performance or stability. label Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR-Improvement Release note category. A PR that improves EC's performance or stability.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants