From 4c2264ee42eceeedc4787b9f3736b0bd19880d7a Mon Sep 17 00:00:00 2001
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sat, 28 Nov 2009 11:59:45 +0000
Subject: [PATCH] Adds 2 buttons to easily reorder selected columns (#4272).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3106 e93f8b46-1217-0410-a6f0-8f06a7374b81
---
 app/views/queries/_columns.rhtml       |  8 ++++++--
 public/javascripts/select_list_move.js | 27 ++++++++++++++++++++++++++
 2 files changed, 33 insertions(+), 2 deletions(-)

diff --git a/app/views/queries/_columns.rhtml b/app/views/queries/_columns.rhtml
index e11b286b6..91ee627b2 100644
--- a/app/views/queries/_columns.rhtml
+++ b/app/views/queries/_columns.rhtml
@@ -5,15 +5,19 @@
 		          :multiple => true, :size => 10, :style => "width:150px" %>
 		</td>
 		<td align="center" valign="middle">
-			<input type="button" value="--&gt;"
+			<input type="button" value="&#8594;"
 			 onclick="moveOptions(this.form.available_columns, this.form.selected_columns);" /><br />
-			<input type="button" value="&lt;--"
+			<input type="button" value="&#8592;"
 			 onclick="moveOptions(this.form.selected_columns, this.form.available_columns);" />
 		</td>
 		<td><%= select_tag 'query[column_names][]',
 		          options_for_select(query.columns.collect {|column| [column.caption, column.name]}),
 		          :id => 'selected_columns', :multiple => true, :size => 10, :style => "width:150px" %>
 		</td>
+		<td align="center" valign="middle">
+			<input type="button" value="&#8593;" onclick="moveOptionUp(this.form.selected_columns);" /><br />
+			<input type="button" value="&#8595;" onclick="moveOptionDown(this.form.selected_columns);" />
+		</td>
 	</tr>
 </table>
 
diff --git a/public/javascripts/select_list_move.js b/public/javascripts/select_list_move.js
index 1ced88232..a9c5bfd36 100644
--- a/public/javascripts/select_list_move.js
+++ b/public/javascripts/select_list_move.js
@@ -7,6 +7,17 @@ function addOption(theSel, theText, theValue)
   theSel.options[selLength] = newOpt;
 }
 
+function swapOptions(theSel, index1, index2)
+{
+	var text, value;
+  text = theSel.options[index1].text;
+  value = theSel.options[index1].value;
+  theSel.options[index1].text = theSel.options[index2].text;
+  theSel.options[index1].value = theSel.options[index2].value;
+  theSel.options[index2].text = text;
+  theSel.options[index2].value = value;
+}
+
 function deleteOption(theSel, theIndex)
 { 
   var selLength = theSel.length;
@@ -45,6 +56,22 @@ function moveOptions(theSelFrom, theSelTo)
   if(NS4) history.go(0);
 }
 
+function moveOptionUp(theSel) {
+	var index = theSel.selectedIndex;
+	if (index > 0) {
+		swapOptions(theSel, index-1, index);
+  	theSel.selectedIndex = index-1;
+	}
+}
+
+function moveOptionDown(theSel) {
+	var index = theSel.selectedIndex;
+	if (index < theSel.length - 1) {
+		swapOptions(theSel, index, index+1);
+  	theSel.selectedIndex = index+1;
+	}
+}
+
 function selectAllOptions(id)
 {
   var select = $(id);