Add optional track row cover thumbnails#966
Open
ihor-developer wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi! This PR adds an optional setting to show album cover thumbnails directly in regular track rows.
The feature is disabled by default, so the current compact list experience stays unchanged unless the user explicitly enables it. When enabled, library and playlist rows get a small cover thumbnail at the start of the row. For the currently playing track, the existing animated playing indicator is reused in that same thumbnail slot instead of showing the cover, so the playing state remains clear without duplicating visuals.
I tried to keep the implementation small and aligned with the existing Museeks architecture:
Covercomponent and cover loading pipeline.PlayingIndicator, with a larger variant for thumbnail rows.I like the direction Museeks is going in: clean, simple, and focused. This feature is meant to fit that direction by making track lists a little easier to visually scan without turning the UI into something noisy or overly configurable.
Screenshots
Setting
Dark theme
Light theme
Notes on translations
The English and Russian translations have been checked. The Russian copy is:
Показывать обложки трековПоказывать миниатюры обложек альбомов в строках трековThe other translations were added as best-effort translations and should ideally be validated by native speakers or regular contributors for those languages.
Validation
I ran:
Results:
vp check --no-fmtpassed with existing warnings only.I did not run a broad
vp check --fixbecause the repository currently has unrelated formatting differences, and I wanted to avoid formatting churn outside this feature.Implementation details
track_view_covers, defaulting tofalse.TrackList.Coverso row thumbnails do not add redundant screen reader text.Compatibility and performance
The feature is cross-platform and uses the existing cover-art pipeline, so it does not add OS-specific behavior.
In default mode there should be no meaningful performance or memory impact because row covers are not mounted.
When enabled, thumbnails are only used in the virtualized default list, so cover loading stays bounded to visible rows plus overscan. Grouped lists are intentionally left unchanged until they are virtualized or cover loading is optimized further.