put cover art on the spinny widget

Bug #890421 reported by Daniel Schürmann on 2011-11-14
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Wishlist
Unassigned

Bug Description

This was original proposed in bug #680806 by RJ Ryan. Since it has the Status "Fix Commited" we need a new one. Here it is.

RJ Skerry-Ryan (rryan) on 2011-11-14
Changed in mixxx:
status: New → Confirmed
importance: Undecided → Wishlist
tags: added: wspinny
jus (jus) wrote :

While i agree that we should have an option to display the cover art and it might look interesting there, i do not think the vinyl widget is the right place for it.

* Depending on the skin the spinning vinyl can be hidden
* We already display the signal quality for vinyl control there
* On the long? run it might be distracting to look at rotating pictures, especially if you trying to use the spinning vinyl widget for orientation

Imho the cover art belongs to the library.

Daniel Schürmann (daschuer) wrote :

The main cover art bug is this: Bug #661459.

I would like to have an option to display cover art instead of the spinny widget. Maybe we can extend the spinny switch to a multi state button like:

off -> spinny -> cover > spinny cover (for crazy) -> off

jus (jus) wrote :

In Deere we display the cover of the track loaded in a deck right next to the track title.
https://github.com/mixxxdj/mixxx/commit/359dade9f53052ce8c7d6f77958d16993774e830
Good enough to mark this bug fixed?

Daniel Schürmann (daschuer) wrote :

Yes.

Changed in mixxx:
status: Confirmed → Fix Committed
assignee: nobody → Max Linke (max-linke)
RJ Skerry-Ryan (rryan) wrote :

I don't think the bug is fixed (since cover art is not on the spinny widget yet) -- but we can mark it WontFix due to Jus's points above.

Changed in mixxx:
status: Fix Committed → Won't Fix
assignee: Max Linke (max-linke) → nobody
Owen Williams (ywwg) wrote :

But actually I want to fix this -- there's not a lot of room in LateNight for cover art.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers