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

Explicitly state extension icon size #528

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

filiptronicek
Copy link
Member

We just add width and height to icons in order to reduce CLS.

@filiptronicek filiptronicek self-assigned this Sep 25, 2022
@filiptronicek filiptronicek added the webui Component: webui label Sep 25, 2022
@amvanbaren
Copy link
Contributor

Hi @filiptronicek, can you specify some testing steps to replicate the before and after effect of this change?

@filiptronicek
Copy link
Member Author

@amvanbaren the only testing step one can try is to open the web ui with this change and check the images are still the same size. This should only help browsers with knowing the size of the images before loading them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low webui Component: webui
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants