Changeset 51 for branches

Timestamp:

02/15/09 20:13:19 (3 years ago)

Author:

daris

Message:

quick search: fixed bug when enter hides form instead of submitting it + release 1.0.1

Location:

branches/quick_search

Files:

• manifest.xml (modified) (2 diffs)
• scripts.js (modified) (4 diffs)

branches/quick_search/manifest.xml

@@ -4 +4 @@
         <id>quick_search</id>
         <title>Quick Search</title>
-        <version>1.0</version>
+        <version>1.0.1</version>
         <description>Allows quickly search for topics and posts</description>
         <author>daris</author>
@@ -12 +12 @@
 
                 <hook id="hd_head"><![CDATA[
-
+global $lang_search;
 require_once FORUM_ROOT.'lang/'.$forum_user['language'].'/search.php';
 $language_js = 'var lang_search = {\'Display results\' : \''.$lang_search['Display results'].'\', \'Show as topics\' : \''.$lang_search['Show as topics'].'\', \'Show as posts\' : \''.$lang_search['Show as posts'].'\', \'Submit search\' : \''.$lang_search['Submit search'].'\', \'Perform new search\' : \''.$lang_search['Perform new search'].'\'};';

branches/quick_search/scripts.js

@@ -10 +10 @@
 
 /*
-        Shows popup popup
+        Shows popup
 */
 function quick_search()
@@ -28 +28 @@
         popup.onmouseout = function () {quick_search_popup_hovered = false}
 
-        popup.innerHTML = '<form action="' + url_search + '" method="get"><div>' +
+        popup.innerHTML = '<form id="quick_search_form" action="' + url_search + '" method="get"><div>' +
                         '<input type="hidden" name="action" value="search" />' + 
-                        '<input type="text" size="20" id="quick_search" name="keywords" /><input type="submit" value="' + lang_search['Submit search'] + '" name="search" />' +
+                        '<input type="text" size="20" id="quick_search" name="keywords" onkeypress="quick_search_onchange(event)" /><input type="submit" value="' + lang_search['Submit search'] + '" name="search" />' +
                         '</div><div>' + lang_search['Display results'] + ': ' + 
                         '<input type="radio" id="show_as_topics" name="show_as" value="topics" checked="checked" /><label for="show_as_topics">' + lang_search['Show as topics'] + '</label> ' + 
@@ -54 +54 @@
 
 /*
-        Hides popup popup
+        Hides popup
 */
 function hide_quick_search_popup()
@@ -66 +66 @@
 
 
+
+function quick_search_onchange(e)
+{
+        if (e.keyCode == 13)
+                document.getElementById('quick_search_form').submit();
+}
 
 /*