Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feature #48945 [WebProfilerBundle] Use a dynamic SVG favicon in the p…
…rofiler (javiereguiluz) This PR was squashed before being merged into the 6.3 branch. Discussion ---------- [WebProfilerBundle] Use a dynamic SVG favicon in the profiler | Q | A | ------------- | --- | Branch? | 6.3 | Bug fix? | no | New feature? | yes | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - The Profiler favicon looks blurry in high definition monitors. In this PR I propose: (1) To use a SVG icon instead (2) To make it dynamic to display a red or yellow background depending on the HTTP status code of the response This is how the colored favicons look on Chrome _(on the left you can see the current blurry icon)_: <img width="377" alt="svg-favicon-chrome" src="https://user-images.githubusercontent.com/73419/211763047-98f3742e-bc94-4ced-80ac-d02eb6fa4e96.png"> And this is how it looks on Firefox: <img width="540" alt="svg-favicon-firefox" src="https://user-images.githubusercontent.com/73419/211763095-99d7d4ac-8699-46c0-a3e6-2bbd667337bd.png"> About accessibility issues for color-blind people. I think this wouldn't be an issue because we're not using color as the only way of conveying information. It's just a little addition. But if you disagree I can revert. Thanks. Commits ------- e6c56a3 [WebProfilerBundle] Use a dynamic SVG favicon in the profiler
- Loading branch information