[PR #2137] [CLOSED] feat: add album cover to full screen #2098

New issue

Closed

opened 2026-02-26 03:33:16 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 03:33:16 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/2137
Author: @Doppelkeks12
Created: 10/19/2025
Status: ❌ Closed

Base: master ← Head: feat/album-cover-viewer

📝 Commits (1)

9cb5327 feat: add album cover to full screen

📊 Changes

7 files changed (+160 additions, -15 deletions)

View changed files

📝 app/Http/Resources/SongResource.php (+2 -0)
📝 app/Models/Album.php (+43 -9)
📝 app/Observers/AlbumObserver.php (+7 -2)
📝 app/Services/ImageStorage.php (+8 -1)
📝 resources/assets/js/components/layout/app-footer/index.vue (+25 -1)
➕ tests/Feature/SongFullScreenCoverTest.php (+48 -0)
📝 tests/Unit/Services/ImageStorageTest.php (+27 -2)

📄 Description

Description

This adds a full album cover image on top of the existing backdrop when in fullscreen mode.
To ensure good image quality, a new 1920 px version has been added, with a fallback to the existing 500 px image.

Motivation

Personally, I felt the album cover was previously underrepresented — when an artist image was available, it wasn’t shown at all, and otherwise only appeared as a cropped black-and-white version.

Screenshots (if applicable)

Screenshot 2025-10-19 at 18-17-59 21st Century ♫ Koel

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

Unfortunately, the original image was no longer available, so I couldn't generate new thumbnails with higher resolution.
As a result, the existing albums will all have a 500 px resolution.
I tried to refactor the code so that the original image would always be persisted (for future changes), but this would affect many other parts of the system (artists, playlists, etc.).
Therefore, I think it’s out of scope for this PR, and I don’t know the codebase well enough to predict all potential side effects.

I don’t have access to any third-party integrations (like spotify), so I wasn’t able to test this with them.

_{🔄 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/2137 **Author:** [@Doppelkeks12](https://github.com/Doppelkeks12) **Created:** 10/19/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `feat/album-cover-viewer` --- ### 📝 Commits (1) - [`9cb5327`](https://github.com/koel/koel/commit/9cb53277fe324c89e42eb29bab1c6b8f24b6d405) feat: add album cover to full screen ### 📊 Changes **7 files changed** (+160 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `app/Http/Resources/SongResource.php` (+2 -0) 📝 `app/Models/Album.php` (+43 -9) 📝 `app/Observers/AlbumObserver.php` (+7 -2) 📝 `app/Services/ImageStorage.php` (+8 -1) 📝 `resources/assets/js/components/layout/app-footer/index.vue` (+25 -1) ➕ `tests/Feature/SongFullScreenCoverTest.php` (+48 -0) 📝 `tests/Unit/Services/ImageStorageTest.php` (+27 -2) </details> ### 📄 Description ## Description This adds a full album cover image on top of the existing backdrop when in fullscreen mode. To ensure good image quality, a new 1920 px version has been added, with a fallback to the existing 500 px image. ## Motivation Personally, I felt the album cover was previously underrepresented — when an artist image was available, it wasn’t shown at all, and otherwise only appeared as a cropped black-and-white version. ## Screenshots (if applicable) <img width="1920" height="1080" alt="Screenshot 2025-10-19 at 18-17-59 21st Century ♫ Koel" src="https://github.com/user-attachments/assets/78e85f01-1ffd-470c-b00a-b5892d2d4fb1" /> ## 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 Unfortunately, the original image was no longer available, so I couldn't generate new thumbnails with higher resolution. As a result, the existing albums will all have a 500 px resolution. I tried to refactor the code so that the original image would always be persisted (for future changes), but this would affect many other parts of the system (artists, playlists, etc.). Therefore, I think it’s out of scope for this PR, and I don’t know the codebase well enough to predict all potential side effects. I don’t have access to any third-party integrations (like spotify), so I wasn’t able to test this with them. --- <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:16 +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#2098

No description provided.

Rows
Columns