Fix: Treat mouse moving in selecting by mouse as one move

Fix #284
LiMinggang · Apr 30, 2019 · 2359fed · 2359fed
1 parent d472a45
commit 2359fed
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/README.txt b/README.txt
@@ -124,6 +124,7 @@ New: Improve MadPython API to support readable APIs
 New: Japanese Translation(Thanks Tilt <tiltstr@gmail.com>)
 New: Package for FreeBSD 12.0
 Fix: wxWidgets 3.0.x backward compatibility
+Fix: Treat mouse moving in selecting by mouse as one move #284
 Change *nix build to use shared libs released by the OS. You have to install libwxgtk by yourself
 Add wxWidgets version number in About dialog
 Update boostpython to 1.70

diff --git a/src/MadEdit/MadEdit.cpp b/src/MadEdit/MadEdit.cpp
@@ -10237,6 +10237,10 @@ void MadEdit::OnMouseLeftUp( wxMouseEvent &evt )
 				if( !m_DragDrop )
 				{
 					EndUpdateSelection( true );
+					if( m_RecordCaretMovements && m_Selection )
+					{
+						m_CaretTracker->Add( m_SelectionPos1.pos, m_SelectionPos2.pos );
+					}
 				}
 			}
 
@@ -10479,11 +10483,6 @@ void MadEdit::OnMouseMotion( wxMouseEvent &evt )
 		}
 
 		DoSelectionChanged();
-
-		if( m_RecordCaretMovements && oldCaretPos != m_CaretPos.pos )
-		{
-			m_CaretTracker->Add( oldCaretPos, m_CaretPos.pos );
-		}
 	}
 
 	evt.Skip();