No track end warning if waveform renderer is set to empty

Bug #1204294 reported by Ferran Pujol
14
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
Daniel Schürmann

Bug Description

If waveform renderer is set to empty, then the track end warning does not work in the waveform overview.

Not sure if it's a bug or an intended behaviour.

Tags: waveform
description: updated
Revision history for this message
Daniel Schürmann (daschuer) wrote :

Yes, This is a bug!

I would be happy if you can adopt it according to my hints from mixxx-devel.
Thank you!

Changed in mixxx:
status: New → Confirmed
importance: Undecided → Low
Changed in mixxx:
assignee: nobody → Ferran Pujol (ferranpujol)
jus (jus)
tags: added: waveform
Changed in mixxx:
assignee: Ferran Pujol (ferranpujol) → nobody
Revision history for this message
jus (jus) wrote :

I think the end of track warning does not belong to the main waveform at all, it is pretty redundant there.

Currently we display
  a) A blinking gradient on the background of the main waveform
  b) a static rectangle on the background of the waveform overview.

 The end of track warning should go as blinking rectangle exclusively to the waveform overview. Saves performance on the main waveform, while being always prominently visible on the waveform overview.

Revision history for this message
ronso0 (ronso0) wrote :

I agree, there should be a blinking rectangle in the overview. Now, that we have 3 skins where the scrolling waveforms can be hidden it's easy to miss the warning. It would also help a lot in minimal decks with max. library' https://github.com/mixxxdj/mixxx/pull/1933 which is almost finished.

To get the most attention for the end warning it should be stacked on top of the dimmed (played) part of the overview
https://bugs.launchpad.net/mixxx/+bug/1792797

Revision history for this message
Daniel Schürmann (daschuer) wrote :
Changed in mixxx:
assignee: nobody → Daniel Schürmann (daschuer)
milestone: none → 2.3.0
ronso0 (ronso0)
Changed in mixxx:
status: Confirmed → Fix Committed
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7114

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.