Attempt to fix bug #3787; works for table creation but not for editing a

row

js/keyhandler.js

@@ -4,6 +4,21 @@
   *
   * @param object   event data
   */
+
+AJAX.registerTeardown('keyhandler.js', function() {
+    $('#table_columns').unbind('keydown');
+    $('#insertRowTable').unbind('keydown');
+});
+
+AJAX.registerOnload('keyhandler.js', function() {
+    $('#table_columns').bind('keydown', function(event) {
+        onKeyDownArrowsHandler(event);
+    });
+    $('#insertRowTable').bind('keydown', function(event) {
+        onKeyDownArrowsHandler(event);
+    });
+});
+
 function onKeyDownArrowsHandler(e)
 {
     e = e||window.event;

libraries/tbl_columns_definition_form.inc.php

@@ -723,13 +723,6 @@ for ($i = 0; $i < $num_fields; $i++) {
     }
 } // end for
 
-$html .= '<script src="js/keyhandler.js" type="text/javascript"></script>'
-    . '<script type="text/javascript">'
-    . '// <![CDATA['
-    . 'document.onkeydown = onKeyDownArrowsHandler;'
-    . '// ]]>'
-    . '</script>';
-
 $html .= '<form method="post" action="' . $action  . '" class="'
     . ($action == 'tbl_create.php' ? 'create_table' : 'append_fields')
     . '_form ajax">';

tbl_change.php

@@ -114,6 +114,7 @@ $header   = $response->getHeader();
 $scripts  = $header->getScripts();
 $scripts->addFile('functions.js');
 $scripts->addFile('tbl_change.js');
+$scripts->addFile('keyhandler.js');
 $scripts->addFile('jquery/jquery-ui-timepicker-addon.js');
 $scripts->addFile('gis_data_editor.js');
 
@@ -196,11 +197,6 @@ $chg_evt_handler = (PMA_USR_BROWSER_AGENT == 'IE'
 // some browsers send wrongly this form to the http server.
 
 $html_output = '';
-//Set on key handler for moving using by Ctrl+arrows
-$html_output .= '<script src="js/keyhandler.js" type="text/javascript"></script>'
-    . '<script type="text/javascript">'
-    . 'document.onkeydown = onKeyDownArrowsHandler;'
-    . '</script>';
 // Set if we passed the first timestamp field
 $timestamp_seen = false;
 $columns_cnt     = count($table_fields);

tbl_create.php

@@ -13,6 +13,15 @@ $action = 'tbl_create.php';
 
 $titles = PMA_Util::buildActionTitles();
 
+/**
+ * Load JavaScript files
+ */
+$response = PMA_Response::getInstance();
+$header   = $response->getHeader();
+$scripts  = $header->getScripts();
+$scripts->addFile('keyhandler.js');
+
+
 // Check parameters
 PMA_Util::checkParameters(array('db'));