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

fix(UI): Remove not loaded font #7242

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Conversation

avelad
Copy link
Member

@avelad avelad commented Sep 2, 2024

font-family settings always started with "Roboto-Regular" before "Roboto", but we don't load "Roboto-Regular". It may exist already on some systems, but we don't explicitly load it. We do explicitly load "Roboto", so that should be the only Roboto-family font in the list. This issue has been present since the very first introduction of our UI.

@avelad avelad marked this pull request as draft September 2, 2024 08:10
@shaka-bot
Copy link
Collaborator

Incremental code coverage: No instrumented code was changed.

font-family settings always started with "Roboto-Regular" before "Roboto", but we don't load "Roboto-Regular". It may exist already on some systems, but we don't explicitly load it. We do explicitly load "Roboto", so that should be the only Roboto-family font in the list.  This issue has been present since the very first introduction of our UI.
@avelad avelad marked this pull request as ready for review September 2, 2024 10:00
@avelad avelad added type: bug Something isn't working correctly component: UI The issue involves the Shaka Player UI priority: P2 Smaller impact or easy workaround labels Sep 2, 2024
@avelad avelad merged commit 47f686f into shaka-project:main Sep 3, 2024
27 of 35 checks passed
@avelad avelad deleted the css-issue branch September 3, 2024 06:57
avelad added a commit that referenced this pull request Sep 4, 2024
font-family settings always started with "Roboto-Regular" before
"Roboto", but we don't load "Roboto-Regular". It may exist already on
some systems, but we don't explicitly load it. We do explicitly load
"Roboto", so that should be the only Roboto-family font in the list.
This issue has been present since the very first introduction of our UI.
avelad added a commit that referenced this pull request Sep 4, 2024
font-family settings always started with "Roboto-Regular" before
"Roboto", but we don't load "Roboto-Regular". It may exist already on
some systems, but we don't explicitly load it. We do explicitly load
"Roboto", so that should be the only Roboto-family font in the list.
This issue has been present since the very first introduction of our UI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: UI The issue involves the Shaka Player UI priority: P2 Smaller impact or easy workaround type: bug Something isn't working correctly
3 participants