From 50e152f3d4727fffe89d2926f45e81e1f18d129c Mon Sep 17 00:00:00 2001
From: Krisjanis Rijnieks <krisjanis.rijnieks@gmail.com>
Date: Sun, 30 Oct 2016 19:01:07 +0200
Subject: [PATCH] Make TextureMappingMode::moveSelection more elegant

---
 src/Application/Modes/TextureMappingMode.cpp | 24 ++++++--------------
 1 file changed, 7 insertions(+), 17 deletions(-)

diff --git a/src/Application/Modes/TextureMappingMode.cpp b/src/Application/Modes/TextureMappingMode.cpp
index d62a0c7..b422cd4 100644
--- a/src/Application/Modes/TextureMappingMode.cpp
+++ b/src/Application/Modes/TextureMappingMode.cpp
@@ -268,24 +268,14 @@ void TextureMappingMode::drawTexture(Application * app){
 }
 
 void TextureMappingMode::moveSelection(Application * app, ofVec2f by){
-	// TODO: Make the command mechanism more elegant than this
-	bool foundSelected = false;
+	int selectedTexCoord = Gui::instance()->getTextureEditorWidget().getSelectedTexCoord();
 	
-	for(
-		unsigned int i = 0;
-		i < Gui::instance()->getTextureEditorWidget().getJoints().size();
-		++i){
-		
-		if(Gui::instance()->getTextureEditorWidget().getJoints()[i]->isSelected()){
-			app->getCmdManager()->exec(
-				new MvTexCoordCmd(
-					i, &Gui::instance()->getTextureEditorWidget()));
-			foundSelected = true;
-			break;
-		}
-	}
-	
-	if(!foundSelected){
+	if(selectedTexCoord >= 0){
+		app->getCmdManager()->exec(
+			new MvTexCoordCmd(
+				selectedTexCoord,
+				&Gui::instance()->getTextureEditorWidget()));
+	}else{
 		app->getCmdManager()->exec(new MvAllTexCoordsCmd(
 			app->getSurfaceManager()->getSelectedSurface(),
 			&Gui::instance()->getTextureEditorWidget()));