fix(unobtrusive-player): handle shuffle play #3247

Merged
franz-dc merged 1 commit from fix/unobtrusive-player-shuffle-play into master 2025-04-16 14:18:14 +00:00
franz-dc commented 2025-04-16 11:47:17 +00:00 (Migrated from github.com)

Fixes #3235

Fixes #3235
copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed 2025-04-16 14:09:27 +00:00
copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/plugins/unobtrusive-player/index.ts:29

  • If the SVG element or its 'd' attribute is not found, shuffleIconPathData becomes undefined, resulting in an invalid query selector. Consider adding an explicit check to ensure that shuffleIconPathData is defined before using it.
document.querySelector('iron-iconset-svg[name="yt-sys-icons"] #shuffle')
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments. <details> <summary>Comments suppressed due to low confidence (1)</summary> **src/plugins/unobtrusive-player/index.ts:29** * If the SVG element or its 'd' attribute is not found, shuffleIconPathData becomes undefined, resulting in an invalid query selector. Consider adding an explicit check to ensure that shuffleIconPathData is defined before using it. ``` document.querySelector('iron-iconset-svg[name="yt-sys-icons"] #shuffle') ``` </details>
JellyBrick (Migrated from github.com) approved these changes 2025-04-16 14:18:00 +00:00
JellyBrick commented 2025-04-16 14:18:10 +00:00 (Migrated from github.com)

Thanks!

Thanks!
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: YTMD/youtube-music#3247
No description provided.