[PR #2139] [MERGED] chore: make more elements custom theme-compatible #2102

Closed
opened 2026-02-26 03:33:17 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/2139
Author: @phanan
Created: 10/20/2025
Status: Merged
Merged: 10/20/2025
Merged by: @phanan

Base: masterHead: theme-fixes


📝 Commits (1)

  • 639cc4a chore: make more elements custom theme-compatible

📊 Changes

56 files changed (+148 additions, -176 deletions)

View changed files

📝 resources/assets/css/app.pcss (+2 -2)
📝 resources/assets/css/partials/context-menu.pcss (+3 -9)
📝 resources/assets/css/partials/hack.pcss (+1 -1)
📝 resources/assets/css/partials/shared.pcss (+7 -6)
📝 resources/assets/css/partials/vars.pcss (+0 -1)
📝 resources/assets/js/components/album/AlbumTrackList.vue (+1 -1)
📝 resources/assets/js/components/album/AlbumTrackListItem.vue (+1 -1)
📝 resources/assets/js/components/album/__snapshots__/AlbumCard.spec.ts.snap (+2 -2)
📝 resources/assets/js/components/album/__snapshots__/AlbumContextMenu.spec.ts.snap (+10 -5)
📝 resources/assets/js/components/album/__snapshots__/AlbumTrackListItem.spec.ts.snap (+1 -1)
📝 resources/assets/js/components/artist/__snapshots__/ArtistCard.spec.ts.snap (+2 -2)
📝 resources/assets/js/components/artist/__snapshots__/ArtistContextMenu.spec.ts.snap (+10 -5)
📝 resources/assets/js/components/embed/widget/EmbedWidgetTrackItem.vue (+1 -1)
📝 resources/assets/js/components/genre/__snapshots__/GenreContextMenu.spec.ts.snap (+6 -3)
📝 resources/assets/js/components/koel-plus/KoelPlusModal.vue (+2 -2)
📝 resources/assets/js/components/layout/app-footer/__snapshots__/FooterPlaybackControls.spec.ts.snap (+2 -2)
📝 resources/assets/js/components/layout/main-wrapper/side-sheet/SideSheetButton.vue (+2 -2)
📝 resources/assets/js/components/layout/main-wrapper/side-sheet/__snapshots__/SideSheet.spec.ts.snap (+3 -3)
📝 resources/assets/js/components/layout/main-wrapper/sidebar/HomeButton.vue (+1 -1)
📝 resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistFolderSidebarItem.vue (+1 -1)

...and 36 more files

📄 Description

Description

Vest some remaining elements to make them theme-able.

Motivation

Screenshots (if applicable)

Checklist

  • I've tested my changes thoroughly and added tests where applicable
  • I've updated relevant documentation (if any)
  • My code follows the project's conventions

🔄 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/2139 **Author:** [@phanan](https://github.com/phanan) **Created:** 10/20/2025 **Status:** ✅ Merged **Merged:** 10/20/2025 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `theme-fixes` --- ### 📝 Commits (1) - [`639cc4a`](https://github.com/koel/koel/commit/639cc4a63ff8b8edc3c99c3740fe987514573691) chore: make more elements custom theme-compatible ### 📊 Changes **56 files changed** (+148 additions, -176 deletions) <details> <summary>View changed files</summary> 📝 `resources/assets/css/app.pcss` (+2 -2) 📝 `resources/assets/css/partials/context-menu.pcss` (+3 -9) 📝 `resources/assets/css/partials/hack.pcss` (+1 -1) 📝 `resources/assets/css/partials/shared.pcss` (+7 -6) 📝 `resources/assets/css/partials/vars.pcss` (+0 -1) 📝 `resources/assets/js/components/album/AlbumTrackList.vue` (+1 -1) 📝 `resources/assets/js/components/album/AlbumTrackListItem.vue` (+1 -1) 📝 `resources/assets/js/components/album/__snapshots__/AlbumCard.spec.ts.snap` (+2 -2) 📝 `resources/assets/js/components/album/__snapshots__/AlbumContextMenu.spec.ts.snap` (+10 -5) 📝 `resources/assets/js/components/album/__snapshots__/AlbumTrackListItem.spec.ts.snap` (+1 -1) 📝 `resources/assets/js/components/artist/__snapshots__/ArtistCard.spec.ts.snap` (+2 -2) 📝 `resources/assets/js/components/artist/__snapshots__/ArtistContextMenu.spec.ts.snap` (+10 -5) 📝 `resources/assets/js/components/embed/widget/EmbedWidgetTrackItem.vue` (+1 -1) 📝 `resources/assets/js/components/genre/__snapshots__/GenreContextMenu.spec.ts.snap` (+6 -3) 📝 `resources/assets/js/components/koel-plus/KoelPlusModal.vue` (+2 -2) 📝 `resources/assets/js/components/layout/app-footer/__snapshots__/FooterPlaybackControls.spec.ts.snap` (+2 -2) 📝 `resources/assets/js/components/layout/main-wrapper/side-sheet/SideSheetButton.vue` (+2 -2) 📝 `resources/assets/js/components/layout/main-wrapper/side-sheet/__snapshots__/SideSheet.spec.ts.snap` (+3 -3) 📝 `resources/assets/js/components/layout/main-wrapper/sidebar/HomeButton.vue` (+1 -1) 📝 `resources/assets/js/components/layout/main-wrapper/sidebar/PlaylistFolderSidebarItem.vue` (+1 -1) _...and 36 more files_ </details> ### 📄 Description <!-- Thank you for contributing to Koel! Please provide a clear description of your changes below. --> ## Description Vest some remaining elements to make them theme-able. ## Motivation <!-- Explain why this change is necessary, e.g., if targeting an existing issue, link to it. --> ## Screenshots (if applicable) ## Checklist - [x] I've tested my changes thoroughly and added tests where applicable - [x] I've updated relevant documentation (if any) - [x] My code follows the project's conventions --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 03:33:17 +03:00
Sign in to join this conversation.
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#2102
No description provided.