Knife Tool for Sample Clips (Again)

data/themes/classic/style.css

@@ -596,10 +596,10 @@ FxLine {
 	color: #e0e0e0;
 	qproperty-backgroundActive: qlineargradient(spread:reflect, x1:0, y1:0, x2:1, y2:0,
 								stop:0 #7b838d, stop:1 #6b7581 );
-	qproperty-strokeOuterActive: rgb( 0, 0, 0 );	
-	qproperty-strokeOuterInactive: rgba( 0, 0, 0, 50 );	
-	qproperty-strokeInnerActive: rgba( 255, 255, 255, 100 );	
-	qproperty-strokeInnerInactive: rgba( 255, 255, 255, 50 );					
+	qproperty-strokeOuterActive: rgb( 0, 0, 0 );
+	qproperty-strokeOuterInactive: rgba( 0, 0, 0, 50 );
+	qproperty-strokeInnerActive: rgba( 255, 255, 255, 100 );
+	qproperty-strokeInnerInactive: rgba( 255, 255, 255, 50 );
 }
 
 /* persistent peak markers for fx peak meters */
@@ -656,7 +656,8 @@ TrackContentObjectView   {
 	qproperty-gradient: true; /* boolean property, set true to have a gradient */
 	/* finger tip offset of cursor */
 	qproperty-mouseHotspotHand: 3px 3px;
-
+	qproperty-mouseHotspotKnife: 0px 0px;
+
 	font-size: 11px;
 }
 

data/themes/default/style.css

@@ -702,7 +702,8 @@ TrackContentObjectView   {
 	qproperty-gradient: false; /* boolean property, set true to have a gradient */
 	/* finger tip offset of cursor */
 	qproperty-mouseHotspotHand: 7px 2px;
-
+	qproperty-mouseHotspotKnife: 0px 0px;
+
 	font-size: 11px;
 }
 

include/SampleTrack.h

@@ -93,7 +93,6 @@ public slots:
 	BoolModel m_recordModel;
 	bool m_isPlaying;
 
-
 	friend class SampleTCOView;
 
 
@@ -132,6 +131,7 @@ public slots:
 private:
 	SampleTCO * m_tco;
 	QPixmap m_paintPixmap;
+	bool splitTCO( const TimePos pos ) override;
 } ;
 
 

include/SongEditor.h

@@ -55,6 +55,7 @@ class SongEditor : public TrackContainerView
 	enum EditMode
 	{
 		DrawMode,
+		KnifeMode,
 		SelectMode
 	};
 
@@ -77,6 +78,7 @@ public slots:
 
 	void setEditMode( EditMode mode );
 	void setEditModeDraw();
+	void setEditModeKnife();
 	void setEditModeSelect();
 	void toggleProportionalSnap();
 
@@ -112,6 +114,7 @@ private slots:
 	void wheelEvent( QWheelEvent * we ) override;
 
 	bool allowRubberband() const override;
+	bool knifeMode() const override;
 
 	int trackIndexFromSelectionPoint(int yPos);
 	int indexOfTrackView(const TrackView* tv);
@@ -173,6 +176,7 @@ class SongEditorWindow : public Editor
 	QSize sizeHint() const override;
 
 	SongEditor* m_editor;
+	void syncEditMode();
 
 protected:
 	void resizeEvent( QResizeEvent * event ) override;
@@ -194,16 +198,14 @@ protected slots:
 	void resized();
 
 private:
-	void keyPressEvent( QKeyEvent * ke ) override;
-	void keyReleaseEvent( QKeyEvent * ke ) override;
-
 	QAction* m_addBBTrackAction;
 	QAction* m_addSampleTrackAction;
 	QAction* m_addAutomationTrackAction;
 	QAction* m_setProportionalSnapAction;
 
 	ActionGroup * m_editModeGroup;
 	QAction* m_drawModeAction;
+	QAction* m_knifeModeAction;
 	QAction* m_selectModeAction;
 	QAction* m_crtlAction;
 

include/Track.h

@@ -177,7 +177,7 @@ class LMMS_EXPORT Track : public Model, public JournallingObject
 	{
 		return m_processingLock.tryLock();
 	}
-	
+
 	QColor color()
 	{
 		return m_color;
@@ -186,7 +186,7 @@ class LMMS_EXPORT Track : public Model, public JournallingObject
 	{
 		return m_hasColor;
 	}
-	
+
 	BoolModel* getMutedModel();
 
 public slots:
@@ -218,7 +218,7 @@ public slots:
 	tcoVector m_trackContentObjects;
 
 	QMutex m_processingLock;
-	
+
 	QColor m_color;
 	bool m_hasColor;
 

include/TrackContainerView.h

@@ -80,6 +80,7 @@ class TrackContainerView : public QWidget, public ModelView,
 	const TrackView * trackViewAt( const int _y ) const;
 
 	virtual bool allowRubberband() const;
+	virtual bool knifeMode() const;
 
 	inline bool rubberBandActive() const
 	{

include/TrackContentObjectView.h

@@ -58,6 +58,7 @@ class TrackContentObjectView : public selectableObject, public ModelView
 	// We have to use a QSize here because using QPoint isn't supported.
 	// width -> x, height -> y
 	Q_PROPERTY( QSize mouseHotspotHand WRITE setMouseHotspotHand )
+	Q_PROPERTY( QSize mouseHotspotKnife WRITE setMouseHotspotKnife )
 
 public:
 	TrackContentObjectView( TrackContentObject * tco, TrackView * tv );
@@ -93,6 +94,7 @@ class TrackContentObjectView : public selectableObject, public ModelView
 	void setBBPatternBackground( const QColor & c );
 	void setGradient( const bool & b );
 	void setMouseHotspotHand(const QSize & s);
+	void setMouseHotspotKnife(const QSize & s);
 
 	// access needsUpdate member variable
 	bool needsUpdate();
@@ -118,6 +120,9 @@ class TrackContentObjectView : public selectableObject, public ModelView
 
 	QColor getColorForDisplay( QColor );
 
+	void inline setMarkerPos(int x) { m_markerPos = x; }
+	void inline setMarkerEnabled(bool e) { m_marker = e; }
+
 public slots:
 	virtual bool close();
 	void remove();
@@ -137,6 +142,13 @@ public slots:
 		Merge
 	};
 
+	TrackView * m_trackView;
+	TimePos m_initialTCOPos;
+	TimePos m_initialTCOEnd;
+
+	bool m_marker = false;
+	int m_markerPos = 0;
+
 	virtual void constructContextMenu( QMenu * )
 	{
 	}
@@ -145,7 +157,6 @@ public slots:
 	void contextMenuAction( ContextMenuAction action );
 	void dragEnterEvent( QDragEnterEvent * dee ) override;
 	void dropEvent( QDropEvent * de ) override;
-	void leaveEvent( QEvent * e ) override;
 	void mousePressEvent( QMouseEvent * me ) override;
 	void mouseMoveEvent( QMouseEvent * me ) override;
 	void mouseReleaseEvent( QMouseEvent * me ) override;
@@ -155,6 +166,9 @@ public slots:
 		selectableObject::resizeEvent( re );
 	}
 
+	bool unquantizedModHeld( QMouseEvent * me );
+	TimePos quantizeSplitPos( TimePos, bool shiftMode );
+
 	float pixelsPerBar();
 
 
@@ -176,19 +190,17 @@ protected slots:
 		MoveSelection,
 		Resize,
 		ResizeLeft,
+		Split,
 		CopySelection,
 		ToggleSelected
 	} ;
 
 	static TextFloat * s_textFloat;
 
 	TrackContentObject * m_tco;
-	TrackView * m_trackView;
 	Actions m_action;
 	QPoint m_initialMousePos;
 	QPoint m_initialMouseGlobalPos;
-	TimePos m_initialTCOPos;
-	TimePos m_initialTCOEnd;
 	QVector<TimePos> m_initialOffsets;
 
 	TextFloat * m_hint;
@@ -202,7 +214,10 @@ protected slots:
 	QColor m_textShadowColor;
 	QColor m_BBPatternBackground;
 	bool m_gradient;
-	QSize m_mouseHotspotHand; // QSize must be used because QPoint isn't supported by property system
+	QSize m_mouseHotspotHand; // QSize must be used because QPoint
+	QSize m_mouseHotspotKnife; // isn't supported by property system
+	QCursor m_cursorHand;
+	QCursor m_cursorKnife;
 	bool m_cursorSetYet;
 
 	bool m_needsUpdate;
@@ -217,6 +232,10 @@ protected slots:
 
 	bool mouseMovedDistance( QMouseEvent * me, int distance );
 	TimePos draggedTCOPos( QMouseEvent * me );
+	int knifeMarkerPos( QMouseEvent * me );
+	//! Return true iff TCO could be split. Currently only implemented for samples
+	virtual bool splitTCO( const TimePos pos ){ return false; };
+	void updateCursor(QMouseEvent * me);
 } ;
 
 

include/TrackContentWidget.h

@@ -99,6 +99,7 @@ public slots:
 	void dragEnterEvent( QDragEnterEvent * dee ) override;
 	void dropEvent( QDropEvent * de ) override;
 	void mousePressEvent( QMouseEvent * me ) override;
+	void mouseReleaseEvent( QMouseEvent * me ) override;
 	void paintEvent( QPaintEvent * pe ) override;
 	void resizeEvent( QResizeEvent * re ) override;
 

src/core/Track.cpp

@@ -210,12 +210,12 @@ void Track::saveSettings( QDomDocument & doc, QDomElement & element )
 	{
 		element.setAttribute( "trackheight", m_height );
 	}
-	
+
 	if( m_hasColor )
 	{
 		element.setAttribute( "color", m_color.name() );
 	}
-	
+
 	QDomElement tsDe = doc.createElement( nodeName() );
 	// let actual track (InstrumentTrack, bbTrack, sampleTrack etc.) save
 	// its settings
@@ -671,4 +671,3 @@ BoolModel *Track::getMutedModel()
 {
 	return &m_mutedModel;
 }
-

src/gui/TrackContainerView.cpp

@@ -308,6 +308,14 @@ bool TrackContainerView::allowRubberband() const
 
 
 
+bool TrackContainerView::knifeMode() const
+{
+	return false;
+}
+
+
+
+
 void TrackContainerView::setPixelsPerBar( int ppb )
 {
 	m_ppb = ppb;
@@ -374,7 +382,7 @@ void TrackContainerView::dropEvent( QDropEvent * _de )
 		//it->toggledInstrumentTrackButton( true );
 		_de->accept();
 	}
-	else if( type == "samplefile" || type == "pluginpresetfile" 
+	else if( type == "samplefile" || type == "pluginpresetfile"
 		|| type == "soundfontfile" || type == "vstpluginfile"
 		|| type == "patchfile" )
 	{