search

clementine-player / Clementine

:tangerine: Clementine Music Player
https://www.clementine-player.org/
GNU General Public License v3.0
3.74k stars 675 forks source link

Feature Request Album cover visualization #5230

Open gfcapalbo opened 8 years ago

gfcapalbo commented 8 years ago

Fantastic feature request.

The °show fullsize° feature for covers is OK but shows a 600x600 window. Most people like to use all screen estate to look at the cover when listening to music and not multitasking-

A visualization option that just shows the cover , stretched over full screen would be really really cool.

santigl commented 8 years ago

Right now that is implemented using a QDialog that is created, displayed and just waits until you close it. It doesn't have the capabilities of updating the image or responding to other events.

In order to implement that, we would need to create a new class. Since we would have a dedicated handler for that window, it would probably end up making sense to also allow to show lyrics, and maybe also display playback controls.

Is that something that could be added?

narunlifescience commented 8 years ago

this is not that hard to implement.. but stretching an image to full screen will distort it completely with respect to aspect ratio & resolution.. so i don't think this is a good idea.

santigl commented 8 years ago

That's true. If it has a relatively big size it could be displayed 1:1 and centered. The problem would be with small covers...

gfcapalbo commented 8 years ago

i was thinking about a scaling, non deforming zoom, image proportions should remain the same, so on 16:9 screens there would be two black bands on the side. Images bigger than the monitor resolution would be reduced, also without deforming.

If there is unbearable quality loss, the user could just use the current "show fullsize" , or perhaps download a higher res cover.

The whole reason for this request is emulating the old pleasure of enjoying the large LP pictures while listnening to music.

alexanderdd commented 6 years ago

Hey, there is a related issue here https://github.com/clementine-player/Clementine/issues/1284