From d1ce12301d5e382ed5e0b179356685282b7dd92b Mon Sep 17 00:00:00 2001 From: Tom Morris Date: Wed, 10 Apr 2013 18:54:38 -0400 Subject: [PATCH 1/3] Quote problematic characters in help text - fixes #463 --- refine.bat | 16 ++++++++-------- refine.ini | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/refine.bat b/refine.bat index f9f468077..629205c18 100644 --- a/refine.bat +++ b/refine.bat @@ -23,25 +23,25 @@ goto endUtils echo Usage %0 [options] ^ echo where [options] include: echo. -echo /? print this message and exit +echo "/?" print this message and exit echo. -echo /p the port that OpenRefine will listen to +echo "/p " the port that OpenRefine will listen to echo default: 3333 echo. -echo /i the host interface OpenRefine should bind to +echo "/i " the host interface OpenRefine should bind to echo default: 127.0.0.1 echo. -echo /w path to the webapp +echo "/w " path to the webapp echo default src\main\webapp echo. -echo /d enable JVM debugging (on port 8000) +echo "/d" enable JVM debugging (on port 8000) echo. -echo /m max memory heap size to use +echo "/m " max memory heap size to use echo default: 1024M echo. -echo /x enable JMX monitoring (for jconsole and friends) +echo "/x" enable JMX monitoring (for jconsole and friends) echo. -echo and is one of +echo "and is one of echo. echo build ..................... Build OpenRefine echo run ....................... Run OpenRefine diff --git a/refine.ini b/refine.ini index 3e4c29566..1ecd3e606 100644 --- a/refine.ini +++ b/refine.ini @@ -5,7 +5,7 @@ no_proxy="localhost,127.0.0.1" #REFINE_PORT=3334 #REFINE_HOST=127.0.0.1 #REFINE_WEBAPP=main\webapp -#REFINE_MEMORY=1024M +REFINE_MEMORY=3000M # Some sample configurations. These have no defaults. #ANT_HOME=C:\grefine\tools\apache-ant-1.8.1 From 0d2881dac6fbcae082cb4a0d22974869fb66a536 Mon Sep 17 00:00:00 2001 From: Tom Morris Date: Sat, 13 Apr 2013 16:50:56 -0400 Subject: [PATCH 2/3] Update link to help Switch from Google Code to Github --- .../modules/core/scripts/dialogs/clustering-dialog.html | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/main/webapp/modules/core/scripts/dialogs/clustering-dialog.html b/main/webapp/modules/core/scripts/dialogs/clustering-dialog.html index 29eb2107d..151201ce8 100644 --- a/main/webapp/modules/core/scripts/dialogs/clustering-dialog.html +++ b/main/webapp/modules/core/scripts/dialogs/clustering-dialog.html @@ -9,7 +9,7 @@ "New York" and "new york" are very likely to refer to the same concept and just have capitalization differences, and "Gödel" and "Godel" probably refer to the same person. - Find out more ... + Find out more ... @@ -64,4 +64,4 @@ - \ No newline at end of file + From d1b2dc3fe952d3a58b9eec520e71bb85fd0cfa8f Mon Sep 17 00:00:00 2001 From: Tom Morris Date: Sat, 13 Apr 2013 17:04:03 -0400 Subject: [PATCH 3/3] Add type to text input field - fixes #706 Even though type="text" is the default, the find won't work without it explicitly included --- main/webapp/modules/core/scripts/dialogs/clustering-dialog.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/main/webapp/modules/core/scripts/dialogs/clustering-dialog.js b/main/webapp/modules/core/scripts/dialogs/clustering-dialog.js index c85f4249e..997af721a 100644 --- a/main/webapp/modules/core/scripts/dialogs/clustering-dialog.js +++ b/main/webapp/modules/core/scripts/dialogs/clustering-dialog.js @@ -153,7 +153,7 @@ ClusteringDialog.prototype._renderTable = function(clusters) { cluster.value = value; parent.find("input[type='text']").val(value); - parent.find("input:not(:checked)").attr('checked', true).change(); + parent.find("input[type='checkbox']").attr('checked', true).change(); return false; }; for (var c = 0; c < choices.length; c++) { @@ -202,7 +202,7 @@ ClusteringDialog.prototype._renderTable = function(clusters) { editCheck.attr("checked", "true"); } - var input = $('') + var input = $('') .attr("value", cluster.value) .keyup(function() { cluster.value = this.value;