[PR #1621] [MERGED] feat: only show YouTube sidebar item on demand #1818

New issue

Closed

opened 2026-02-26 03:32:25 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 03:32:25 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/1621
Author: @phanan
Created: 12/7/2022
Status: ✅ Merged
Merged: 12/7/2022
Merged by: @phanan

Base: master ← Head: feat/youtube-on-demand

📝 Commits (1)

a21ca56 feat: only show YouTube sidebar item on demand

📊 Changes

18 files changed (+377 additions, -268 deletions)

View changed files

➖ resources/assets/js/components/layout/main-wrapper/Sidebar.spec.ts (+0 -8)
➖ resources/assets/js/components/layout/main-wrapper/Sidebar.vue (+0 -252)
📝 resources/assets/js/components/layout/main-wrapper/index.vue (+1 -1)
📝 resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistFolderSidebarItem.vue (+2 -2)
📝 resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarItem.spec.ts (+0 -0)
📝 resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarItem.vue (+1 -1)
📝 resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarList.spec.ts (+0 -0)
📝 resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarList.vue (+2 -3)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/QueueSidebarItem.vue (+39 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/Sidebar.spec.ts (+55 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/Sidebar.vue (+180 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/SidebarItem.spec.ts (+35 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/SidebarItem.vue (+23 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/YouTubeSidebarItem.spec.ts (+17 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/YouTubeSidebarItem.vue (+15 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/__snapshots__/SidebarItem.spec.ts.snap (+3 -0)
➕ resources/assets/js/components/layout/main-wrapper/sidebar/__snapshots__/YouTubeSidebarItem.spec.ts.snap (+3 -0)
📝 resources/assets/js/composables/useDragAndDrop.ts (+1 -1)

📄 Description

Also refactoring the sidebar item into a reusable component. Too bad the Playlist/PlaylistFolder sidebar items are still too flexible to be adapted.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/koel/koel/pull/1621 **Author:** [@phanan](https://github.com/phanan) **Created:** 12/7/2022 **Status:** ✅ Merged **Merged:** 12/7/2022 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `feat/youtube-on-demand` --- ### 📝 Commits (1) - [`a21ca56`](https://github.com/koel/koel/commit/a21ca56a29785aa503cbb161f23bc7e1fcbe0c55) feat: only show YouTube sidebar item on demand ### 📊 Changes **18 files changed** (+377 additions, -268 deletions) <details> <summary>View changed files</summary> ➖ `resources/assets/js/components/layout/main-wrapper/Sidebar.spec.ts` (+0 -8) ➖ `resources/assets/js/components/layout/main-wrapper/Sidebar.vue` (+0 -252) 📝 `resources/assets/js/components/layout/main-wrapper/index.vue` (+1 -1) 📝 `resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistFolderSidebarItem.vue` (+2 -2) 📝 `resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarItem.spec.ts` (+0 -0) 📝 `resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarItem.vue` (+1 -1) 📝 `resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarList.spec.ts` (+0 -0) 📝 `resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistSidebarList.vue` (+2 -3) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/QueueSidebarItem.vue` (+39 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/Sidebar.spec.ts` (+55 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/Sidebar.vue` (+180 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/SidebarItem.spec.ts` (+35 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/SidebarItem.vue` (+23 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/YouTubeSidebarItem.spec.ts` (+17 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/YouTubeSidebarItem.vue` (+15 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/__snapshots__/SidebarItem.spec.ts.snap` (+3 -0) ➕ `resources/assets/js/components/layout/main-wrapper/sidebar/__snapshots__/YouTubeSidebarItem.spec.ts.snap` (+3 -0) 📝 `resources/assets/js/composables/useDragAndDrop.ts` (+1 -1) </details> ### 📄 Description Also refactoring the sidebar item into a reusable component. Too bad the Playlist/PlaylistFolder sidebar items are still too flexible to be adapted. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-02-26 03:32:25 +03:00

closed this issue
added the
pull-request
label

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

starred/koel-koel#1818

No description provided.

Rows
Columns