Enable track-wide color coding

include/AutomationPatternView.h

@@ -27,10 +27,11 @@
 
 #include <QStaticText>
 
+#include "AutomationPattern.h"
+#include "Song.h"
+#include "SongEditor.h"
 #include "Track.h"
 
-class AutomationPattern;
-
 
 class AutomationPatternView : public TrackContentObjectView
 {

include/BBTrack.h

@@ -50,33 +50,11 @@ class BBTCO : public TrackContentObject
 		return( "bbtco" );
 	}
 
-	unsigned int color() const
-	{
-		return( m_color.rgb() );
-	}
-
-	QColor colorObj() const
-	{
-		return m_color;
-	}
-
-	void setColor( const QColor & c )
-	{
-		m_color = QColor( c );
-	}
-
-	void setUseStyleColor( bool b )
-	{
-		m_useStyleColor = b;
-	}
-
 	int bbTrackIndex();
 
 	TrackContentObjectView * createView( TrackView * _tv ) override;
 
 private:
-	QColor m_color;
-	bool m_useStyleColor;
 
 
 	friend class BBTCOView;
@@ -92,11 +70,6 @@ class BBTCOView : public TrackContentObjectView
 	BBTCOView( TrackContentObject * _tco, TrackView * _tv );
 	virtual ~BBTCOView() = default;
 
-	QColor color() const
-	{
-		return( m_bbTCO->m_color );
-	}
-	void setColor( QColor _new_color );
 
 public slots:
 	void update() override;
@@ -105,8 +78,6 @@ protected slots:
 	void openInBBEditor();
 	void resetName();
 	void changeName();
-	void changeColor();
-	void resetColor();
 
 
 protected:
@@ -162,27 +133,6 @@ class LMMS_EXPORT BBTrack : public Track
 		m_disabledTracks.removeAll( _track );
 	}
 
-	static void setLastTCOColor( const QColor & c )
-	{
-		if( ! s_lastTCOColor )
-		{
-			s_lastTCOColor = new QColor( c );
-		}
-		else
-		{
-			*s_lastTCOColor = QColor( c );
-		}
-	}
-
-	static void clearLastTCOColor()
-	{
-		if( s_lastTCOColor )
-		{
-			delete s_lastTCOColor;
-		}
-		s_lastTCOColor = NULL;
-	}
-
 protected:
 	inline QString nodeName() const override
 	{
@@ -196,8 +146,6 @@ class LMMS_EXPORT BBTrack : public Track
 	typedef QMap<BBTrack *, int> infoMap;
 	static infoMap s_infoMap;
 
-	static QColor * s_lastTCOColor;
-
 	friend class BBTrackView;
 
 } ;

include/Track.h

@@ -133,6 +133,25 @@ class LMMS_EXPORT TrackContentObject : public Model, public JournallingObject
 	{
 		return m_autoResize;
 	}
+
+	QColor color() const
+	{
+		return m_color;
+	}
+
+	void setColor( const QColor & c )
+	{
+		m_color = c;
+	}
+
+	bool hasColor();
+
+	void useCustomClipColor( bool b );
+
+	bool usesCustomClipColor()
+	{
+		return m_useCustomClipColor;
+	}
 
 	virtual void movePosition( const MidiTime & pos );
 	virtual void changeLength( const MidiTime & length );
@@ -154,6 +173,8 @@ class LMMS_EXPORT TrackContentObject : public Model, public JournallingObject
 
 	MidiTime startTimeOffset() const;
 	void setStartTimeOffset( const MidiTime &startTimeOffset );
+
+	void updateColor();
 
 public slots:
 	void copy();
@@ -165,6 +186,7 @@ public slots:
 	void lengthChanged();
 	void positionChanged();
 	void destroyedTCO();
+	void trackColorChanged();
 
 
 private:
@@ -188,6 +210,9 @@ public slots:
 
 	bool m_selectViewOnCreate;
 
+	QColor m_color;
+	bool m_useCustomClipColor;
+
 	friend class TrackContentObjectView;
 
 } ;
@@ -263,12 +288,17 @@ class TrackContentObjectView : public selectableObject, public ModelView
 	// some metadata to be written to the clipboard.
 	static void remove( QVector<TrackContentObjectView *> tcovs );
 	static void toggleMute( QVector<TrackContentObjectView *> tcovs );
+
+	QColor getColorForDisplay( QColor );
 
 public slots:
 	virtual bool close();
 	void cut();
 	void remove();
 	void update() override;
+
+	void changeClipColor();
+	void useTrackColor();
 
 protected:
 	enum ContextMenuAction
@@ -486,6 +516,10 @@ private slots:
 	void cloneTrack();
 	void removeTrack();
 	void updateMenu();
+	void changeTrackColor();
+	void randomTrackColor();
+	void resetTrackColor();
+	void useTrackColor();
 	void toggleRecording(bool on);
 	void recordingOn();
 	void recordingOff();
@@ -503,6 +537,9 @@ private slots:
 
 signals:
 	void trackRemovalScheduled( TrackView * t );
+	void colorChanged( QColor & c );
+	void colorParented();
+	void colorReset();
 
 } ;
 
@@ -635,7 +672,16 @@ class LMMS_EXPORT Track : public Model, public JournallingObject
 	{
 		return m_processingLock.tryLock();
 	}
-
+
+	QColor color()
+	{
+		return m_color;
+	}
+	bool useColor()
+	{
+		return m_hasColor;
+	}
+
 	BoolModel* getMutedModel();
 
 public slots:
@@ -647,6 +693,8 @@ public slots:
 
 	void toggleSolo();
 
+	void trackColorChanged( QColor & c );
+	void trackColorReset();
 
 private:
 	TrackContainer* m_trackContainer;
@@ -665,6 +713,9 @@ public slots:
 	tcoVector m_trackContentObjects;
 
 	QMutex m_processingLock;
+
+	QColor m_color;
+	bool m_hasColor;
 
 	friend class TrackView;
 

src/core/AutomationPattern.cpp

@@ -538,6 +538,11 @@ void AutomationPattern::saveSettings( QDomDocument & _doc, QDomElement & _this )
 	_this.setAttribute( "prog", QString::number( progressionType() ) );
 	_this.setAttribute( "tens", QString::number( getTension() ) );
 	_this.setAttribute( "mute", QString::number( isMuted() ) );
+
+	if( usesCustomClipColor() )
+	{
+		_this.setAttribute( "color", color().name() );
+	}
 
 	for( timeMap::const_iterator it = m_timeMap.begin();
 						it != m_timeMap.end(); ++it )
@@ -593,6 +598,12 @@ void AutomationPattern::loadSettings( const QDomElement & _this )
 			m_idsToResolve << element.attribute( "id" ).toInt();
 		}
 	}
+
+	if( _this.hasAttribute( "color" ) )
+	{
+		useCustomClipColor( true );
+		setColor( _this.attribute( "color" ) );
+	}
 
 	int len = _this.attribute( "len" ).toInt();
 	if( len <= 0 )