add hotcue text labels, position, and right click menu to overview waveform

build/depends.py

@@ -901,7 +901,9 @@ def sources(self, build):
                    "src/sources/soundsourceproviderregistry.cpp",
                    "src/sources/soundsourceproxy.cpp",
 
+                   "src/widget/colormenu.cpp",
                    "src/widget/controlwidgetconnection.cpp",
+                   "src/widget/cuemenu.cpp",
                    "src/widget/wbasewidget.cpp",
                    "src/widget/wwidget.cpp",
                    "src/widget/wwidgetgroup.cpp",
@@ -1106,7 +1108,6 @@ def sources(self, build):
 
                    "src/waveform/renderers/waveformrenderersignalbase.cpp",
                    "src/waveform/renderers/waveformmark.cpp",
-                   "src/waveform/renderers/waveformmarkproperties.cpp",
                    "src/waveform/renderers/waveformmarkset.cpp",
                    "src/waveform/renderers/waveformmarkrange.cpp",
                    "src/waveform/renderers/glwaveformrenderersimplesignal.cpp",
@@ -1115,6 +1116,7 @@ def sources(self, build):
                    "src/waveform/renderers/glslwaveformrenderersignal.cpp",
                    "src/waveform/renderers/glvsynctestrenderer.cpp",
 
+                   "src/waveform/waveformmarklabel.cpp",
                    "src/waveform/widgets/waveformwidgetabstract.cpp",
                    "src/waveform/widgets/emptywaveformwidget.cpp",
                    "src/waveform/widgets/softwarewaveformwidget.cpp",

res/skins/Deere/style.qss

@@ -736,10 +736,15 @@ QAbstractScrollArea::corner {
 
 WWidget, QLabel {
   font-family: "Open Sans";
-  font-size: 8px;
+  font-size: 12px;
   text-transform: uppercase;
 }
 
+WOverview {
+  /* This affects the labels for cue points. */
+  text-transform: none;
+}
+
 /* Start spacing for Deck overview row (small waveform, option grid) */
 #OptionGrid, #ButtonGrid {
   background-color: #333333;

src/skin/tooltips.cpp

@@ -36,8 +36,10 @@ void Tooltips::addStandardTooltips() {
 
     add("waveform_overview")
             << tr("Waveform Overview")
-            << tr("Shows information about the track currently loaded in this channel.")
+            << tr("Shows information about the track currently loaded in this deck.")
             << tr("Jump around in the track by clicking anywhere on the waveform.")
+            << tr("Right click hotcues to edit their labels and colors.")
+            << tr("Right click anywhere else to show the time at that point.")
             << dropTracksHere;
 
     QString scratchMouse = tr("Use the mouse to scratch, spin-back or throw tracks.");
@@ -538,7 +540,8 @@ void Tooltips::addStandardTooltips() {
             << QString("%1: %2").arg(leftClick, tr("If hotcue is set, jumps to the hotcue."))
             << tr("If hotcue is not set, sets the hotcue to the current play position.")
             << quantizeSnap
-            << QString("%1: %2").arg(rightClick, tr("If hotcue is set, clears the hotcue."));
+            << QString("%1: %2").arg(rightClick, tr("If hotcue is set, clears the hotcue."))
+            << tr("Right click hotcues on the overview waveform to edit their labels and colors.");
 
     // Status displays and toggle buttons
     add("toggle_recording")

src/track/cue.h

@@ -34,6 +34,8 @@ class Cue : public QObject {
         OUTRO   = 7,
     };
 
+    static constexpr double kPositionNotDefined = -1.0;
+
     ~Cue() override = default;
 
     bool isDirty() const;

src/waveform/renderers/waveformmark.cpp

@@ -1,39 +1,105 @@
 #include <QtDebug>
 
 #include "skin/skincontext.h"
-#include "waveform/renderers/waveformmarkproperties.h"
+#include "waveform/renderers/waveformsignalcolors.h"
+#include "widget/wskincolor.h"
 
 #include "waveformmark.h"
 
-WaveformMark::WaveformMark(const QString& group,
-                           const QDomNode& node,
-                           const SkinContext& context,
-                           const WaveformSignalColors& signalColors)
-        : m_iHotCue(kNoHotCue) {
-    QString item = context.selectString(node, "Control");
-    if (!item.isEmpty()) {
-        m_pPointCos = std::make_unique<ControlProxy>(group, item);
-        if (item.startsWith("hotcue_") && item.endsWith("_position")) {
-            m_iHotCue = item.midRef(7, item.count() - 16).toInt() - 1;
+namespace {
+Qt::Alignment decodeAlignmentFlags(QString alignString, Qt::Alignment defaultFlags) {
+    QStringList stringFlags = alignString.toLower()
+            .split("|", QString::SkipEmptyParts);
+
+    Qt::Alignment hflags = 0L;
+    Qt::Alignment vflags = 0L;
+
+    for (auto stringFlag : stringFlags) {
+        if (stringFlag == "center") {
+            hflags |= Qt::AlignHCenter;
+            vflags |= Qt::AlignVCenter;
+        } else if (stringFlag == "left") {
+            hflags |= Qt::AlignLeft;
+        } else if (stringFlag == "hcenter") {
+            hflags |= Qt::AlignHCenter;
+        } else if (stringFlag == "right") {
+            hflags |= Qt::AlignRight;
+        } else if (stringFlag == "top") {
+            vflags |= Qt::AlignTop;
+        } else if (stringFlag == "vcenter") {
+            vflags |= Qt::AlignVCenter;
+        } else if (stringFlag == "bottom") {
+            vflags |= Qt::AlignBottom;
         }
     }
-    QString visibilityControl = context.selectString(node, "VisibilityControl");
-    if (!visibilityControl.isEmpty()) {
-        ConfigKey key = ConfigKey::parseCommaSeparated(visibilityControl);
-        m_pVisibleCos = std::make_unique<ControlProxy>(key);
+
+    if (hflags != Qt::AlignLeft && hflags != Qt::AlignHCenter && hflags != Qt::AlignRight) {
+        hflags = defaultFlags & Qt::AlignHorizontal_Mask;
+    }
+
+    if (vflags != Qt::AlignTop && vflags != Qt::AlignVCenter && vflags != Qt::AlignBottom) {
+        vflags = defaultFlags & Qt::AlignVertical_Mask;
     }
-    m_properties = WaveformMarkProperties(node, context, signalColors, m_iHotCue);
+
+    return hflags | vflags;
 }
+} // anonymous namespace
 
 WaveformMark::WaveformMark(const QString& group,
                            const QDomNode& node,
                            const SkinContext& context,
                            const WaveformSignalColors& signalColors,
                            int hotCue)
         : m_iHotCue(hotCue) {
-    if (hotCue >= 0) {
-        QString item = "hotcue_" + QString::number(hotCue + 1) + "_position";
-        m_pPointCos = std::make_unique<ControlProxy>(group, item);
+    QString control;
+    if (hotCue != kNoHotCue) {
+        control = "hotcue_" + QString::number(hotCue + 1) + "_position";
+    } else {
+        control = context.selectString(node, "Control");
+    }
+    if (!control.isEmpty()) {
+        m_pPointCos = std::make_unique<ControlProxy>(group, control);
+    }
+
+    QString visibilityControl = context.selectString(node, "VisibilityControl");
+    if (!visibilityControl.isEmpty()) {
+        ConfigKey key = ConfigKey::parseCommaSeparated(visibilityControl);
+        m_pVisibleCos = std::make_unique<ControlProxy>(key);
+    }
+
+    QColor color(context.selectString(node, "Color"));
+    if (!color.isValid()) {
+        // As a fallback, grab the color from the parent's AxesColor
+        color = signalColors.getAxesColor();
+        qDebug() << "Didn't get mark <Color>, using parent's <AxesColor>:" << color;
+    } else {
+        color = WSkinColor::getCorrectColor(color);
+    }
+    setBaseColor(color);
+
+    m_textColor = context.selectString(node, "TextColor");
+    if (!m_textColor.isValid()) {
+        // Read the text color, otherwise use the parent's BgColor.
+        m_textColor = signalColors.getBgColor();
+        qDebug() << "Didn't get mark <TextColor>, using parent's <BgColor>:" << m_textColor;
+    }
+
+    QString markAlign = context.selectString(node, "Align");
+    m_align = decodeAlignmentFlags(markAlign, Qt::AlignBottom | Qt::AlignHCenter);
+
+    // Hotcue text is set by the cue's label in the database, not by the skin.
+    if (hotCue == WaveformMark::kNoHotCue) {
+        m_text = context.selectString(node, "Text");
+    }
+
+    m_pixmapPath = context.selectString(node, "Pixmap");
+    if (!m_pixmapPath.isEmpty()) {
+        m_pixmapPath = context.makeSkinPath(m_pixmapPath);
     }
-    m_properties = WaveformMarkProperties(node, context, signalColors, hotCue);
 }
+
+void WaveformMark::setBaseColor(QColor baseColor) {
+    m_fillColor = baseColor;
+    m_borderColor = Color::chooseContrastColor(baseColor);
+    m_labelColor = Color::chooseColorByBrightness(baseColor, QColor(255,255,255,255), QColor(0,0,0,255));
+};

src/waveform/renderers/waveformmark.h

@@ -4,10 +4,10 @@
 #include <QDomNode>
 #include <QImage>
 
+#include "waveform/waveformmarklabel.h"
 #include "control/controlproxy.h"
 #include "util/memory.h"
 
-#include "waveform/renderers/waveformmarkproperties.h"
 #include "control/controlobject.h"
 
 class SkinContext;
@@ -18,31 +18,17 @@ class WOverview;
 class WaveformMark {
   public:
     static const int kNoHotCue = -1;
-    WaveformMark(
-            const QString& group,
-            const QDomNode& node,
-            const SkinContext& context,
-            const WaveformSignalColors& signalColors);
-
     WaveformMark(
             const QString& group,
             const QDomNode& node,
             const SkinContext& context,
             const WaveformSignalColors& signalColors,
-            int hotCue);
-
+            int hotCue = kNoHotCue);
 
     // Disable copying
     WaveformMark(const WaveformMark&) = delete;
     WaveformMark& operator=(const WaveformMark&) = delete;
 
-    const WaveformMarkProperties& getProperties() const {
-        return m_properties;
-    };
-    void setProperties(const WaveformMarkProperties& properties) {
-        m_properties = properties;
-    };
-
     int getHotCue() const { return m_iHotCue; };
 
     //The m_pPointCos related function
@@ -57,23 +43,56 @@ class WaveformMark {
 
     // The m_pVisibleCos related function
     bool hasVisible() const { return m_pVisibleCos && m_pVisibleCos->valid(); }
-    bool isVisible() const { return m_pVisibleCos->get(); }
+    bool isVisible() const {
+        if (!hasVisible()) {
+            return true;
+        }
+        return m_pVisibleCos->get();
+    }
 
     template <typename Receiver, typename Slot>
     void connectVisibleChanged(Receiver receiver, Slot slot) const {
         m_pVisibleCos->connectValueChanged(receiver, slot, Qt::AutoConnection);
     }
 
+    // Sets the appropriate mark colors based on the base color
+    void setBaseColor(QColor baseColor);
+    QColor fillColor() const {
+        return m_fillColor;
+    }
+    QColor borderColor() const {
+        return m_borderColor;
+    }
+    QColor labelColor() const {
+        return m_labelColor;
+    }
+
+    QColor m_textColor;
+    QString m_text;
+    Qt::Alignment m_align;
+    QString m_pixmapPath;
+
+    float m_linePosition;
+
+    WaveformMarkLabel m_label;
+
   private:
     std::unique_ptr<ControlProxy> m_pPointCos;
     std::unique_ptr<ControlProxy> m_pVisibleCos;
-    WaveformMarkProperties m_properties;
     int m_iHotCue;
     QImage m_image;
 
+    QColor m_fillColor;
+    QColor m_borderColor;
+    QColor m_labelColor;
+
     friend class WaveformRenderMark;
 };
 
 typedef QSharedPointer<WaveformMark> WaveformMarkPointer;
 
+inline bool operator<(const WaveformMarkPointer& lhs, const WaveformMarkPointer& rhs) {
+    return lhs->getSamplePosition() < rhs->getSamplePosition();
+}
+
 #endif // WAVEFORMMARK_H