remove inline onchange

Signed-off-by: Manish Bisht <manish.bisht490@gmail.com>
phpmyadmin · Jun 9, 2017 · e2e2add · e2e2add
1 parent a42aa8a
commit e2e2add
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 3 deletions.
diff --git a/js/tbl_select.js b/js/tbl_select.js
@@ -81,6 +81,13 @@ AJAX.registerOnload('tbl_select.js', function () {
             return false;
         });
 
+    var tableRows = $('select');
+    $.each(tableRows, function(index, item){
+        $(item).on("change", function() {
+            changeValueFieldType(this, index);
+        });
+    });
+
     /**
      * Ajax event handler for Table search
      */

diff --git a/js/tbl_structure.js b/js/tbl_structure.js
@@ -503,6 +503,10 @@ AJAX.registerOnload('tbl_structure.js', function () {
             $.post(url, params, AJAX.responseHandler);
         });
     });
+
+    $(document).on('change', 'select[name=after_field]', function () {
+        checkFirst();
+    });
 });
 
 /** Handler for "More" dropdown in structure table rows */

diff --git a/templates/table/search/column_comparison_operators.twig b/templates/table/search/column_comparison_operators.twig
@@ -1,4 +1,3 @@
-<select name="criteriaColumnOperators[{{ search_index }}]"
-    onchange="changeValueFieldType(this, {{ search_index }})">
+<select id="ColumnOperator{{ search_index }}" name="criteriaColumnOperators[{{ search_index }}]">
     {{ type_operators|raw }}
 </select>
diff --git a/templates/table/structure/add_column.twig b/templates/table/structure/add_column.twig
@@ -9,7 +9,7 @@
     {{ 'Add %s column(s)'|trans|format(num_fields)|raw }}
     <input type="hidden" name="field_where" value="after"/>&nbsp;
     {# I tried displaying the drop-down inside the label but with Firefox the drop-down was blinking #}
-    <select name="after_field" onchange="checkFirst()">
+    <select name="after_field">
         <option value="first" data-pos="first">
             {% trans 'at beginning of table' %}
         </option>