[Icons] Docs: Adding info on enduser performance #2787
Conversation
| When including the same icon multiple times on one page, the same string (SVG content) | ||
| is inserted repeatedly in the HTML source. However, since most icons are small (just a few | ||
| hundred bytes), this overhead is negligible in most cases (especiall wthen proper | ||
| transfer encoding like e.g. gzip is enabled). |
There was a problem hiding this comment.
The UX Icons component is a PHP one, that does generates HTML.
Do you think on twig documentation, you'd write "every time you include a partial, it is included" ?
:)
|
I'm personally voting against this change (for the reasons explained in the original issue and here)
Let's see if other have a different opinion |
|
I'm against it also, I'm concerned this could scare users off. With modern browsers/compression/http2, I'm not convinced this is a real issue. We originally (pre-release) had a "deferred" feature to fix this "issue" but I saw numbers that showed it was nothing but a micro-optimization. |
|
I see that you have strong opinions. So just take what you think is useful, and close/delete the rest. |
|
We're always open to improvements Recent suggestions felt a bit mixed --some very valid points, others less aligned with concrete user needs or going against points we've already covered. Thanks again for your time and input! |
Page: https://symfony.com/bundles/ux-icons/current/index.html#performance
Info is taken from #2782 (comment)