mirror of
https://github.com/jpochyla/psst.git
synced 2026-04-27 07:25:52 +03:00
[PR #176] [MERGED] gui: Display album art in playback bar #447
Labels
No labels
api
bug
build
documentation
duplicate
enhancement
good first issue
help wanted
idea
invalid
linux
lowprio
macos
pull-request
upstream
windows
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/psst#447
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/jpochyla/psst/pull/176
Author: @brandly
Created: 9/14/2021
Status: ✅ Merged
Merged: 9/15/2021
Merged by: @jpochyla
Base:
master← Head:patch-1📝 Commits (3)
627a704gui: Display album art in playback bar24e2713gui: Fix NP cover image for album tracks3b7d3e6gui: Fix NP item overflow📊 Changes
2 files changed (+31 additions, -12 deletions)
View changed files
📝
psst-gui/src/data/playback.rs(+8 -3)📝
psst-gui/src/ui/playback.rs(+23 -9)📄 Description
Hey, I'm Matt! This app is great. I want to learn more about how it all fits together. I saw this comment from @pythonhacker and figured I'd take a stab at it.
My changes here are fairly basic. I saw
cover_widgetfunctions defined in files likeui/artist.rsand followed suit. I thought filling the available space would look nice and somewhat similar to the Albums tab, but an alternative design would be with rounded corners and margins around the image similar to the Home tab. Let me know if you'd like me to change this!The current code works in most situations. I can play from a playlist:
From my saved tracks:
From a Daily Mix:
But when I play from an album view, no album art is displayed:
Thanks for taking a look!
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.