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 focus indication on tags, artist and album #6046

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

dmitrylyzo
Copy link
Contributor

@dmitrylyzo dmitrylyzo commented Sep 9, 2024

Changes
Remove emby-button class - it should be added by EmbyButton on its own.
emby-button class is used as a flag for EmbyButton.createdCallback.

Issues
Fixes #6045

Notes
This fixes the indication, but the focus breaks on Back action because these elements are re-generated.

@dmitrylyzo dmitrylyzo requested a review from a team as a code owner September 9, 2024 23:09
@dmitrylyzo dmitrylyzo added bug Something isn't working stable backport Backport into the next stable release ui & ux This PR or issue mainly concerns UI & UX labels Sep 9, 2024
@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Sep 12, 2024
@jellyfin-bot
Copy link
Collaborator

This pull request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

@dmitrylyzo dmitrylyzo changed the base branch from release-10.9.z to master September 12, 2024 18:10
@dmitrylyzo dmitrylyzo removed the stable backport Backport into the next stable release label Sep 12, 2024
@dmitrylyzo dmitrylyzo added this to the v10.10.0 milestone Sep 12, 2024
@jellyfin-bot jellyfin-bot removed the merge conflict Conflicts prevent merging label Sep 12, 2024
`emby-button` class is used as a flag for `EmbyButton.createdCallback`.
`emby-button` class is used as a flag for `EmbyButton.createdCallback`.
Copy link

sonarcloud bot commented Sep 13, 2024

@jellyfin-bot
Copy link
Collaborator

Cloudflare Pages deployment

Latest commit edfae1f1b899a3eda8c0340184b722cf00e08f84
Status ✅ Deployed!
Preview URL https://bee1b78d.jellyfin-web.pages.dev
Type 🔀 Preview

@thornbill thornbill merged commit 9f1bccb into jellyfin:master Sep 20, 2024
12 checks passed
@dmitrylyzo dmitrylyzo deleted the fix-focus branch September 20, 2024 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui & ux This PR or issue mainly concerns UI & UX
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Tags are not selectable on Samsung TV (Tizen) for 10.9.x
3 participants