Enable mixer color-coding

include/ColorChooser.h

@@ -21,15 +21,26 @@
  *
  */
 
-#include <QColorDialog>
 #include <QApplication>
+#include <QColor>
+#include <QColorDialog>
 #include <QKeyEvent>
+#include <QVector>
 
 class ColorChooser: public QColorDialog
 {
 public:
 	ColorChooser(const QColor &initial, QWidget *parent): QColorDialog(initial, parent) {};
 	ColorChooser(QWidget *parent): QColorDialog(parent) {};
+
+	enum class CCPalette {Default, Track, Mixer};
+
+	//! Set global palette via array, checking bounds
+	void setPalette (const QVector<QColor>);
+	//! Set global paletter via enum
+	void setPalette (const CCPalette);
+	//! Set palette via enum, return self pointer for chaining
+	ColorChooser* setPaletteAndPoint (const CCPalette);
 
 protected:
 	// Forward key events to the parent to prevent stuck notes when the dialog gets focus
@@ -38,4 +49,10 @@ class ColorChooser: public QColorDialog
 		QKeyEvent ke(*event);
 		QApplication::sendEvent(parentWidget(), &ke);
 	}
+
+private:
+	//! Copy the current QColorDialog palette into an array
+	QVector<QColor> defaultPalette();
+	//! Generate a nice palette, with adjustable value
+	QVector<QColor> nicePalette (const int);
 };

include/FxLine.h

@@ -26,10 +26,12 @@
 #ifndef FX_LINE_H
 #define FX_LINE_H
 
+#include <QColorDialog>
 #include <QGraphicsView>
 #include <QLineEdit>
 #include <QWidget>
 
+#include "ColorChooser.h"
 #include "Knob.h"
 #include "LcdWidget.h"
 #include "SendButtonIndicator.h"
@@ -84,6 +86,8 @@ class FxLine : public QWidget
 private:
 	void drawFxLine( QPainter* p, const FxLine *fxLine, bool isActive, bool sendToThis, bool receiveFromThis );
 	QString elideName( const QString & name );
+	QColor mutedColor( QPainter* p, const FxLine *fxLine, bool isActive );
+	QColor unmutedColor( QPainter* p, const FxLine *fxLine, bool isActive );
 
 	FxMixerView * m_mv;
 	LcdWidget* m_lcd;
@@ -101,6 +105,9 @@ class FxLine : public QWidget
 
 public slots:
 	void renameChannel();
+	void resetColor();
+	void changeColor();
+	void randomColor();
 
 private slots:
 	void renameFinished();

include/FxMixer.h

@@ -32,6 +32,8 @@
 
 #include <atomic>
 
+#include <QColor>
+
 class FxRoute;
 typedef QVector<FxRoute *> FxRouteVector;
 
@@ -69,6 +71,9 @@ class FxChannel : public ThreadableJob
 
 		bool requiresProcessing() const override { return true; }
 		void unmuteForSolo();
+
+		QColor m_color;
+		bool m_hasColor;
 
 
 		std::atomic_int m_dependenciesMet;

src/core/FxMixer.cpp

@@ -73,6 +73,7 @@ FxChannel::FxChannel( int idx, Model * _parent ) :
 	m_lock(),
 	m_channelIndex( idx ),
 	m_queued( false ),
+	m_hasColor( false ),
 	m_dependenciesMet(0)
 {
 	BufferManager::clear( m_buffer, Engine::mixer()->framesPerPeriod() );
@@ -741,6 +742,8 @@ void FxMixer::saveSettings( QDomDocument & _doc, QDomElement & _this )
 		ch->m_soloModel.saveSettings( _doc, fxch, "soloed" );
 		fxch.setAttribute( "num", i );
 		fxch.setAttribute( "name", ch->m_name );
+		fxch.setAttribute( "hascolor", ch->m_hasColor );
+		fxch.setAttribute( "mixercolor", ch->m_color.rgb() );
 
 		// add the channel sends
 		for( int si = 0; si < ch->m_sends.size(); ++si )
@@ -786,6 +789,11 @@ void FxMixer::loadSettings( const QDomElement & _this )
 		m_fxChannels[num]->m_muteModel.loadSettings( fxch, "muted" );
 		m_fxChannels[num]->m_soloModel.loadSettings( fxch, "soloed" );
 		m_fxChannels[num]->m_name = fxch.attribute( "name" );
+		if( fxch.hasAttribute( "hascolor" ) )
+		{
+			m_fxChannels[num]->m_hasColor = fxch.attribute( "hascolor" ).toInt();
+			m_fxChannels[num]->m_color.setRgb( fxch.attribute( "mixercolor" ).toUInt() );
+		}
 
 		m_fxChannels[num]->m_fxChain.restoreState( fxch.firstChildElement(
 			m_fxChannels[num]->m_fxChain.nodeName() ) );

src/gui/CMakeLists.txt

@@ -37,6 +37,7 @@ SET(LMMS_SRCS
 
 	gui/dialogs/FileDialog.cpp
 	gui/dialogs/VersionedSaveDialog.cpp
+	gui/dialogs/ColorChooser.cpp
 
 	gui/editors/AutomationEditor.cpp
 	gui/editors/BBEditor.cpp

src/gui/dialogs/ColorChooser.cpp

@@ -0,0 +1,81 @@
+/* ColorChooser.cpp - definition of ColorChooser class.
+ *
+ * Copyright (c) 2020 russiankumar <adityakumar4644/at/gmail/dot/com>
+ *
+ * This file is part of LMMS - https://lmms.io
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public
+ * License along with this program (see COPYING); if not, write to the
+ * Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+ * Boston, MA 02110-1301 USA.
+ *
+ */
+
+#include <ColorChooser.h>
+
+
+
+void ColorChooser::setPalette (const QVector<QColor> colors)
+{
+	const int max = qMin (colors.size(), 48);
+	for (int i = 0; i < max; i++)
+	{
+		ColorChooser::setStandardColor (i, colors[i]);
+	}
+}
+
+
+void ColorChooser::setPalette (const CCPalette palette)
+{
+	switch (palette)
+	{
+		case CCPalette::Default: setPalette (defaultPalette()); break;
+		case CCPalette::Mixer: setPalette (nicePalette(140)); break;
+		case CCPalette::Track: setPalette (nicePalette(150)); break;
+	}
+}
+
+
+ColorChooser* ColorChooser::setPaletteAndPoint (const CCPalette palette)
+{
+	setPalette (palette);
+	return this;
+}
+
+
+
+
+QVector<QColor> ColorChooser::defaultPalette()
+{
+	QVector <QColor> result;
+	for (int i = 0; i < 48; i++)
+	{
+		result.push_back (QColorDialog::standardColor(i));
+	}
+	return result;
+}
+
+
+QVector<QColor> ColorChooser::nicePalette (const int base)
+{
+	QVector <QColor> result;
+	result.resize(48);
+	for (int x = 0; x < 8; x++)
+	{
+		for (int y = 0; y < 6; y++)
+		{
+			result[6 * x + y].setHsl (qMax(0, 44 * x - 1), 150 - 20 * y, base - 10 * y);
+		}
+	}
+	return result;
+}

src/gui/widgets/FxLine.cpp

@@ -25,6 +25,9 @@
 
 #include "FxLine.h"
 
+#include <cstdlib>
+#include <ctime>
+
 #include <QGraphicsProxyWidget>
 
 #include "CaptionMenu.h"
@@ -120,6 +123,11 @@ FxLine::FxLine( QWidget * _parent, FxMixerView * _mv, int _channelIndex ) :
 	proxyWidget->setPos( 8, 145 );
 
 	connect( m_renameLineEdit, SIGNAL( editingFinished() ), this, SLOT( renameFinished() ) );
+
+	connect( &Engine::fxMixer()->effectChannel( m_channelIndex )->m_muteModel, SIGNAL( dataChanged() ), this, SLOT( update() ) );
+
+	srand( time( 0 ) );
+
 }
 
 
@@ -145,10 +153,37 @@ void FxLine::setChannelIndex( int index )
 
 
 
+QColor FxLine::mutedColor( QPainter* p, const FxLine *fxLine, bool isActive )
+{
+	return isActive ? fxLine->backgroundActive().color() : p->background().color();
+}
+
+
+
+
+QColor FxLine::unmutedColor( QPainter* p, const FxLine *fxLine, bool isActive )
+{
+	auto channel = Engine::fxMixer()->effectChannel( m_channelIndex );
+
+	if( channel->m_hasColor )
+	{
+		return isActive ? channel->m_color.darker( 120 ) : channel->m_color.darker( 150 );
+	}
+	else
+	{
+		return isActive ? fxLine->backgroundActive().color() : p->background().color();
+	}
+}
+
+
+
 void FxLine::drawFxLine( QPainter* p, const FxLine *fxLine, bool isActive, bool sendToThis, bool receiveFromThis )
 {
-	QString name = Engine::fxMixer()->effectChannel( m_channelIndex )->m_name;
+	auto channel = Engine::fxMixer()->effectChannel( m_channelIndex );
+	bool muted = channel->m_muteModel.value();
+	QString name = channel->m_name;
 	QString elidedName = elideName( name );
+
 	if( !m_inRename && m_renameLineEdit->text() != elidedName )
 	{
 		m_renameLineEdit->setText( elidedName );
@@ -157,7 +192,9 @@ void FxLine::drawFxLine( QPainter* p, const FxLine *fxLine, bool isActive, bool
 	int width = fxLine->rect().width();
 	int height = fxLine->rect().height();
 
-	p->fillRect( fxLine->rect(), isActive ? fxLine->backgroundActive() : p->background() );
+	QColor color = muted ? mutedColor( p, fxLine, isActive ) : unmutedColor( p, fxLine, isActive );
+
+	p->fillRect( fxLine->rect(), color );
 
 	// inner border
 	p->setPen( isActive ? fxLine->strokeInnerActive() : fxLine->strokeInnerInactive() );
@@ -238,6 +275,11 @@ void FxLine::contextMenuEvent( QContextMenuEvent * )
 		contextMenu->addSeparator();
 	}
 	contextMenu->addAction( embed::getIconPixmap( "cancel" ), tr( "Remove &unused channels" ), this, SLOT( removeUnusedChannels() ) );
+
+	contextMenu->addSeparator();
+	contextMenu->addAction( embed::getIconPixmap( "colorize" ), tr( "Set channel color" ), this, SLOT( changeColor() ) );
+	contextMenu->addAction( embed::getIconPixmap( "colorize" ), tr( "Remove channel color" ), this, SLOT( resetColor() ) );
+	contextMenu->addAction( embed::getIconPixmap( "colorize" ), tr( "Pick random channel color" ), this, SLOT( randomColor() ) );
 	contextMenu->exec( QCursor::pos() );
 	delete contextMenu;
 }
@@ -395,3 +437,33 @@ void FxLine::setStrokeInnerInactive( const QColor & c )
 {
 	m_strokeInnerInactive = c;
 }
+
+void FxLine::changeColor()
+{
+	auto channel = Engine::fxMixer()->effectChannel( m_channelIndex );
+
+	auto new_color = ColorChooser( this ).setPaletteAndPoint( ColorChooser::CCPalette::Mixer )->getColor( channel->m_color );
+	if( ! new_color.isValid() )
+	{ return; }
+
+	channel->m_color = new_color;
+	channel->m_hasColor = true;
+
+	update();
+}
+
+void FxLine::resetColor()
+{
+	Engine::fxMixer()->effectChannel( m_channelIndex )->m_hasColor = false;
+	update();
+}
+
+void FxLine::randomColor()
+{
+	auto channel = Engine::fxMixer()->effectChannel( m_channelIndex );
+
+	channel->m_color = ColorChooser( this ).setPaletteAndPoint( ColorChooser::CCPalette::Mixer )->standardColor( rand() % 48 );
+	channel->m_hasColor = true;
+
+	update();
+}