Display all Hotcues in the waveform / waveform overview

Bug #994647 reported by jus on 2012-05-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Low
Thomas Vincent

Bug Description

As of lp:mixxx r3093 only the first four hotcue markers are displayed in the waveform / waveform overview.
We should allow all 36 hotcues to be displayed like we did in v1.10

Steps to reproduce:
* Load a skin with more than 4 Hotcues ( e.g. Phoney)
* Set Hotcue>4
* Hotcue works but misses marker in waveform / waveform overview

Until now we have to define every single Hotcue per deck/sample deck in the skin.xml .
It would be nice to have a way to define a group of hotcues if they share the same attributes. Basically only have them displayed the hotcue number like we do now if no other attributes are defined.

This would make the skin.xml less bloated.

Related branches

jus (jus) on 2012-05-04
tags: added: waveform
summary: - Display all Hotcues in the waveform
+ Display all Hotcues in the waveform / waveform overview
Changed in mixxx:
milestone: none → 1.11.0
status: New → Confirmed
importance: Undecided → Low
description: updated
Thomas Vincent (vrince) wrote :

I get your point. What about a 'default' cue style in skins that can be used for any non explicitly defined ones ?

jus (jus) wrote :

Having the possibility to define a 'default' cue style sounds excellent.

Thomas Vincent (vrince) on 2012-05-17
Changed in mixxx:
assignee: nobody → Thomas Vincent (vrince)
Thomas Vincent (vrince) wrote :

Implemented in pure-gl branch
A new <DefaultMark> tag shoul be present in skin to create all missing hotcues (from 1 to 36).

Changed in mixxx:
status: Confirmed → In Progress
jus (jus) wrote :

The <DefaultMark> tag which was merged to trunk does currently only works in the main Waveform Display <Visual> but not in the Waveform overview <Overview>.
It should work on both so we do not have to use mixed tags on each.

Thomas Vincent (vrince) wrote :

I thought 36 hot cues in overview was a little intense ... but you are right if it works for main display it should work for overview. Let me handle it ...

Thomas Vincent (vrince) wrote :

Jus can you give it a shot ?
I also add <DefaultMark> text field as the mark text pattern.
It takes the hot cue number as argument %1. Like " HOTCUE %1 " ...

Changed in mixxx:
status: In Progress → Fix Committed
jus (jus) wrote :

Thanks, Thomas.
Works fine - just committed the skin fixes to trunk.

BTW:
Whats up with the "no text draw triangle" in waveformrendermark.cpp around line 166 ff ? How can i use this one, could you please give an example... I tried to leave out <Text>...</Text> from within <DefaultMark>...</DefaultMark> with no success.

Thomas Vincent (vrince) wrote :

Hum ... this should work on the main waveform.
I need to check what append if the text pattern is empty like <Text></Text> but it should not be a problem.

RJ Skerry-Ryan (rryan) on 2013-05-09
Changed in mixxx:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers