Summary: | cover art is not entirely visible in "Now playing" view | ||
---|---|---|---|
Product: | [Applications] Elisa | Reporter: | Patrick Silva <bugseforuns> |
Component: | general | Assignee: | Matthieu Gallien <matthieu_gallien> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | nate |
Priority: | NOR | ||
Version: | 19.12.2 | ||
Target Milestone: | --- | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/kde/elisa/commit/003f0ba3c09b5e6b2035649c9f2edb2847f44b18 | Version Fixed In: | 20.04.0 |
Attachments: | screenshot |
Description
Patrick Silva
2020-02-07 13:33:05 UTC
Created attachment 125737 [details]
screenshot
This is intentional, but I admit it's not my favorite thing either. As you can see, the album art is entirely visible in the headerbar on top (albeit in smaller form) so I kine of wonder what we even need this skinny strip in the Now Playing view for anyway. Nice taste in music, BTW. :) (In reply to Nate Graham from comment #2) > This is intentional, but I admit it's not my favorite thing either. > > As you can see, the album art is entirely visible in the headerbar on top > (albeit in smaller form) so I kine of wonder what we even need this skinny > strip in the Now Playing view for anyway. > > Nice taste in music, BTW. :) When I implemented this view, I worked on the basis of a design where it looked nice on itself. Integrated in the whole UI, it is somewhat odd. I agree. We could maybe have it in full size but behind the text with the appropriate visual effect such that the text is still easy to read ? I think that's a great idea. I've submitted a merge request: https://invent.kde.org/kde/elisa/merge_requests/79 Git commit 003f0ba3c09b5e6b2035649c9f2edb2847f44b18 by Nate Graham. Committed on 12/02/2020 at 17:01. Pushed by ngraham into branch 'master'. Change style in Context view to use blurred album art as the background FIXED-IN: 20.04.0 M +104 -96 src/qml/ContextView.qml https://invent.kde.org/kde/elisa/commit/003f0ba3c09b5e6b2035649c9f2edb2847f44b18 |