From 9d2a3a819a588f0fad2aaac0b330bafe594954ec Mon Sep 17 00:00:00 2001
From: thomascube <thomas@roundcube.net>
Date: Wed, 29 Sep 2010 04:11:51 -0400
Subject: [PATCH] Focus search box or first edit field on page load

---
 program/js/app.js |  151 ++++++++++++++++++++++++++++++++------------------
 1 files changed, 97 insertions(+), 54 deletions(-)

diff --git a/program/js/app.js b/program/js/app.js
index 71daec5..64cc7c6 100644
--- a/program/js/app.js
+++ b/program/js/app.js
@@ -1,9 +1,9 @@
 /*
  +-----------------------------------------------------------------------+
- | RoundCube Webmail Client Script                                       |
+ | Roundcube Webmail Client Script                                       |
  |                                                                       |
- | This file is part of the RoundCube Webmail client                     |
- | Copyright (C) 2005-2010, RoundCube Dev, - Switzerland                 |
+ | This file is part of the Roundcube Webmail client                     |
+ | Copyright (C) 2005-2010, Roundcube Dev, - Switzerland                 |
  | Licensed under the GNU GPL                                            |
  |                                                                       |
  +-----------------------------------------------------------------------+
@@ -56,7 +56,7 @@
   $.ajaxSetup({
     cache:false,
     error:function(request, status, err){ ref.http_error(request, status, err); },
-    beforeSend:function(xmlhttp){ xmlhttp.setRequestHeader('X-RoundCube-Request', ref.env.request_token); }
+    beforeSend:function(xmlhttp){ xmlhttp.setRequestHeader('X-Roundcube-Request', ref.env.request_token); }
   });
 
   // set environment variable(s)
@@ -301,10 +301,14 @@
         if (this.env.cid)
           this.enable_command('show', 'edit', true);
 
-        if ((this.env.action=='add' || this.env.action=='edit') && this.gui_objects.editform)
+        if ((this.env.action=='add' || this.env.action=='edit') && this.gui_objects.editform) {
           this.enable_command('save', true);
-        else
+          $("input[type='text']").first().select();
+        }
+        else if (this.gui_objects.qsearchbox) {
           this.enable_command('search', 'reset-search', 'moveto', true);
+          $(this.gui_objects.qsearchbox).select();
+        }
 
         if (this.contact_list && this.contact_list.rowcount > 0)
           this.enable_command('export', true);
@@ -1288,7 +1292,7 @@
       }
     }
 
-    this.http_post('save-pref', '_name=collapsed_folders&_value='+urlencode(this.env.collapsed_folders));
+    this.http_post('utils/save-pref', '_name=collapsed_folders&_value='+urlencode(this.env.collapsed_folders));
     this.set_unread_count_display(id, false);
   };
 
@@ -1459,7 +1463,7 @@
     if ((found = $.inArray('subject', this.env.coltypes)) >= 0)
       this.set_env('subject_col', found);
 
-    this.http_post('save-pref', { '_name':'list_cols', '_value':this.env.coltypes });
+    this.http_post('utils/save-pref', { '_name':'list_cols', '_value':this.env.coltypes, '_session':'list_attrib/columns' });
   };
 
   this.check_droptarget = function(id)
@@ -1782,11 +1786,11 @@
   {
     if (page == 'next')
       page = this.env.current_page+1;
-    if (page == 'last')
+    else if (page == 'last')
       page = this.env.pagecount;
-    if (page == 'prev' && this.env.current_page > 1)
+    else if (page == 'prev' && this.env.current_page > 1)
       page = this.env.current_page-1;
-    if (page == 'first' && this.env.current_page > 1)
+    else if (page == 'first' && this.env.current_page > 1)
       page = 1;
 
     if (page > 0 && page <= this.env.pagecount) {
@@ -2847,7 +2851,7 @@
   {
     var ed;
     if (window.tinyMCE && (ed = tinyMCE.get(this.env.composebody))) {
-      if (ed.plugins.spellchecker.active)
+      if (ed.plugins.spellchecker && ed.plugins.spellchecker.active)
         ed.execCommand('mceSpellCheck');
     }
     else if ((ed = this.env.spellcheck) && !this.spellcheck_ready) {
@@ -4148,19 +4152,18 @@
       this.goto_url('folders');
       return false;
     }
-    else {
-      // clone a table row if there are existing rows
-      var row = this.clone_table_row(refrow);
-      row.id = id;
 
-      if (before && (before = this.get_folder_row_id(before)))
-        tbody.insertBefore(row, document.getElementById(before));
-      else
-        tbody.appendChild(row);
+    // clone a table row if there are existing rows
+    var row = this.clone_table_row(refrow);
+    row.id = id;
 
-      if (replace)
-        tbody.removeChild(replace);
-    }
+    if (before && (before = this.get_folder_row_id(before)))
+      tbody.insertBefore(row, document.getElementById(before));
+    else
+      tbody.appendChild(row);
+
+    if (replace)
+      tbody.removeChild(replace);
 
     // add to folder/row-ID map
     this.env.subscriptionrows[row.id] = [name, display_name, 0];
@@ -4168,21 +4171,22 @@
     // set folder name
     row.cells[0].innerHTML = display_name;
 
-    // set messages count to zero
-    if (!replace)
+    if (!replace) {
+      // set messages count to zero
       row.cells[1].innerHTML = '*';
 
-    if (!replace && row.cells[2] && row.cells[2].firstChild.tagName.toLowerCase()=='input') {
-      row.cells[2].firstChild.value = name;
-      row.cells[2].firstChild.checked = true;
-    }
+      // update subscription/threading checkboxes
+      $('input[name="_subscribed[]"]', row).val(name).attr('checked', true);
+      $('input[name="_threaded[]"]', row).val(name).attr('checked', false);
 
-    // add new folder to rename-folder list and clear input field
-    if (!replace && (form = this.gui_objects.editform)) {
-      if (form.elements['_folder_oldname'])
-        form.elements['_folder_oldname'].options[form.elements['_folder_oldname'].options.length] = new Option(name,name);
-      if (form.elements['_folder_name'])
-        form.elements['_folder_name'].value = ''; 
+      var elem;
+      // add new folder to rename-folder list and clear input field
+      if (form = this.gui_objects.editform) {
+        if (elem = form.elements['_folder_oldname'])
+          elem.options[elem.options.length] = new Option(name, name);
+        if (elem = form.elements['_folder_name'])
+          elem.value = ''; 
+      }
     }
 
     this.init_subscription_list();
@@ -4236,8 +4240,8 @@
       }
     }
 
-    if (form && form.elements['_folder_newname'])
-      form.elements['_folder_newname'].value = '';
+    if (form && (elm = form.elements['_folder_newname']))
+      elm.value = '';
   };
 
   this.subscribe = function(folder)
@@ -4516,9 +4520,15 @@
   // make a message row disapear
   this.hide_message = function(fade)
   {
-    if (this.gui_objects.message) {
-      $(this.gui_objects.message).unbind()[(fade?'fadeOut':'hide')]();
-      this.gui_objects.message.__type = null;
+    var msg;
+    if (this.gui_objects.message)
+      msg = this.gui_objects.message;
+    else if (this.env.framed && parent.rcmail)
+      msg = parent.rcmail.gui_objects.message;
+
+    if (msg) {
+      $(msg).unbind()[(fade?'fadeOut':'hide')]();
+      msg.__type = null;
     }
   };
 
@@ -4881,25 +4891,41 @@
 
   this.goto_url = function(action, query, lock)
   {
-    var querystring = query ? '&'+query : '';
-    this.redirect(this.env.comm_path+'&_action='+action+querystring, lock);
+    var url = this.env.comm_path,
+     querystring = query ? '&'+query : '';
+
+    // overwrite task name
+    if (action.match(/([a-z]+)\/([a-z-_]+)/)) {
+      action = RegExp.$2;
+      url = url.replace(/\_task=[a-z]+/, '_task='+RegExp.$1);
+    }
+
+    this.redirect(url+'&_action='+action+querystring, lock);
   };
 
   // send a http request to the server
-  this.http_request = function(action, querystring, lock)
+  this.http_request = function(action, query, lock)
   {
+    var url = this.env.comm_path;
+
+    // overwrite task name
+    if (action.match(/([a-z]+)\/([a-z-_]+)/)) {
+      action = RegExp.$2;
+      url = url.replace(/\_task=[a-z]+/, '_task='+RegExp.$1);
+    }
+
     // trigger plugin hook
-    var result = this.triggerEvent('request'+action, querystring);
+    var result = this.triggerEvent('request'+action, query);
+
     if (typeof result != 'undefined') {
       // abort if one the handlers returned false
       if (result === false)
         return false;
       else
-        querystring = result;
+        query = result;
     }
 
-    querystring += (querystring ? '&' : '') + '_remote=1';
-    var url = this.env.comm_path + '&_action=' + action + '&' + querystring;
+    url += '&_remote=1&_action=' + action + (query ? '&' : '') + query;
 
     // send request
     console.log('HTTP GET: ' + url);
@@ -4909,7 +4935,15 @@
   // send a http POST request to the server
   this.http_post = function(action, postdata, lock)
   {
-    var url = this.env.comm_path+'&_action=' + action;
+    var url = this.env.comm_path;
+
+    // overwrite task name
+    if (action.match(/([a-z]+)\/([a-z-_]+)/)) {
+      action = RegExp.$2;
+      url = url.replace(/\_task=[a-z]+/, '_task='+RegExp.$1);
+    }
+
+    url += '&_action=' + action;
 
     if (postdata && typeof(postdata) == 'object') {
       postdata._remote = 1;
@@ -5145,19 +5179,28 @@
     }
   };
 
-  // set all fields of a form disabled
+  // disable/enable all fields of a form
   this.lock_form = function(form, lock)
   {
     if (!form || !form.elements)
       return;
 
-    var type;
-    for (var n=0, len=form.elements.length; n<len; n++) {
-      type = form.elements[n];
-      if (type == 'hidden')
+    var n, len, elm;
+
+    if (lock)
+      this.disabled_form_elements = [];
+
+    for (n=0, len=form.elements.length; n<len; n++) {
+      elm = form.elements[n];
+
+      if (elm.type == 'hidden')
         continue;
 
-      form.elements[n].disabled = lock;
+      // remember which elem was disabled before lock
+      if (lock && elm.disabled)
+        this.disabled_form_elements.push(elm);
+      else if (lock || $.inArray(elm, this.disabled_form_elements)<0)
+        elm.disabled = lock;
     }
   };
 

--
Gitblit v1.9.1