Changeset 887 in subversion

Timestamp:

Oct 19, 2007 8:24:04 AM (6 years ago)

Author:

robin

Message:

• Enable single-selection toggling in JS list object
• Use row id's instead of folder names in folder renaming/moving
• Fix some smaller errors in folder renaming/moving

Location:

trunk/roundcubemail/program

Files:

• js/app.js (modified) (14 diffs)
• js/list.js (modified) (2 diffs)
• steps/settings/manage_folders.inc (modified) (3 diffs)

trunk/roundcubemail/program/js/app.js

@@ -942 +942 @@
 
       case 'delete-folder':
-        if (confirm(this.get_label('deletefolderconfirm')))
-          this.delete_folder(props);
+        this.delete_folder(props);
         break;
 
@@ -2410 +2409 @@
     {
     var p = this;
-    this.subscription_list = new rcube_list_widget(this.gui_objects.subscriptionlist, {multiselect:false, draggable:true, keyboard:false});
+    this.subscription_list = new rcube_list_widget(this.gui_objects.subscriptionlist, {multiselect:false, draggable:true, keyboard:false, toggleselect:true});
     this.subscription_list.addEventListener('select', function(o){ p.subscription_select(o); });
     this.subscription_list.addEventListener('dragstart', function(o){ p.drag_active = true; });
     this.subscription_list.addEventListener('dragend', function(o){ p.subscription_move_folder(o); });
+    this.subscription_list.row_init = function (row)
+      {
+      var anchors = row.obj.getElementsByTagName('A');
+      if (anchors[0])
+        anchors[0].onclick = function() { p.rename_folder(row.id); };
+      if (anchors[1])
+        anchors[1].onclick = function() { p.delete_folder(row.id); };
+      row.obj.onmouseover = function() { p.focus_subscription(row.id); };
+      row.obj.onmouseout = function() { p.unfocus_subscription(row.id); };
+      }
     this.subscription_list.init();
     }
@@ -2466 +2475 @@
   this.focus_subscription = function(id)
     {
-    var row;
+    var row, folder;
     var reg = RegExp('['+RegExp.escape(this.env.delimiter)+']?[^'+RegExp.escape(this.env.delimiter)+']+$');
-    if (this.drag_active && this.check_droptarget(id) &&
-        (id != this.env.folder.replace(reg, '')) &&
-        (!id.match(new RegExp('^'+RegExp.escape(this.env.folder+this.env.delimiter)))) &&
-        (row = document.getElementById(this.get_folder_row_id(id))))
-      if (find_in_array(this.env.defaultfolders, id)>=0)
+    if (this.env.subscriptionrows[id] &&
+        (folder = this.env.subscriptionrows[id][0]) &&
+        this.drag_active && this.check_droptarget(folder) &&
+        (folder != this.env.folder.replace(reg, '')) &&
+        (!folder.match(new RegExp('^'+RegExp.escape(this.env.folder+this.env.delimiter)))) &&
+        (row = document.getElementById(id)))
+      if (find_in_array(this.env.defaultfolders, folder)>=0)
         {
         if (this.env.folder.replace(reg, '')!='')
@@ -2482 +2493 @@
       else
         {
-        this.set_env('dstfolder', id);
+        this.set_env('dstfolder', folder);
         this.set_classname(row, 'droptarget', true);
         }
@@ -2490 +2501 @@
   this.unfocus_subscription = function(id)
     {
-    var row;
-    if (row = document.getElementById(this.get_folder_row_id(id)))
+    var row, folder;
+    if (this.env.subscriptionrows[id] &&
+        (folder = this.env.subscriptionrows[id][0]) &&
+        (row = document.getElementById(id)))
       {
       this.set_env('dstfolder', null);
-      if (find_in_array(this.env.defaultfolders, id)>=0)
+      if (find_in_array(this.env.defaultfolders, folder)>=0)
         this.set_classname(this.subscription_list.frame, 'droptarget', false);
       else
@@ -2504 +2517 @@
   this.subscription_select = function(list)
     {
-    var id;
-    if (id = list.get_single_selection())
-      {
-      if (this.env.subscriptionrows['rcmrow'+id])
-        {
-        var folder = this.env.subscriptionrows['rcmrow'+id][0];
-        if (find_in_array(this.env.defaultfolders, folder)!=0)
-          this.set_env('folder', folder);
-        }
-      else
-        {
-        list.clear_selection();
-        this.set_env('folder', null);
-        }
-      }
+    var id, folder;
+    if ((id = list.get_single_selection()) &&
+        this.env.subscriptionrows['rcmrow'+id] &&
+        (folder = this.env.subscriptionrows['rcmrow'+id][0]) &&
+        (find_in_array(this.env.defaultfolders, folder)!=0))
+      this.set_env('folder', folder);
+    else
+      this.set_env('folder', null);
     };
 
@@ -2534 +2540 @@
       }
     this.drag_active = false;
-    this.unfocus_subscription(this.env.dstfolder);
+    this.unfocus_subscription(this.get_folder_row_id(this.env.dstfolder));
     };
 
@@ -2559 +2565 @@
   // start renaming the mailbox name.
   // this will replace the name string with an input field
-  this.rename_folder = function(folder)
+  this.rename_folder = function(id)
     {
     var temp, row, form;
-    var id = this.get_folder_row_id(folder);
+    var folder = this.env.subscriptionrows[id][0];
 
     // reset current renaming
@@ -2629 +2635 @@
 
   // delete a specific mailbox with all its messages
-  this.delete_folder = function(folder)
-    {
+  this.delete_folder = function(id)
+    {
+    var folder = this.env.subscriptionrows[id][0];
+
     if (this.edit_folder)
       this.reset_folder_rename();
-    
-    if (folder)
+
+    if (folder && confirm(this.get_label('deletefolderconfirm')))
+      {
       this.http_post('delete-folder', '_mboxes='+urlencode(folder));
-
-    this.set_env('folder', null);
+      this.set_env('folder', null);
+      }
     };
 
@@ -2654 +2663 @@
     var refrow, form;
     var tbody = this.gui_objects.subscriptionlist.tBodies[0];
-    var id = replace && replace.id ? replace.id : tbody.childNodes.length+1;
+    var id = replace && replace.id ? replace.id : 'rcmrow'+(tbody.childNodes.length+1);
     var selection = this.subscription_list.get_single_selection();
 
@@ -2666 +2675 @@
       // clone a table row if there are existing rows
       var row = this.clone_table_row(refrow);
-      row.id = 'rcmrow'+id;
+      row.id = id;
       if (replace)
         tbody.replaceChild(row, replace);
@@ -2684 +2693 @@
       }
        
-    if (row.cells[2] && row.cells[2].firstChild.tagName=='A')
-      row.cells[2].firstChild.onclick = new Function(this.ref+".command('rename-folder','"+name.replace('\'','\\\'')+"')");
-    if (row.cells[3] && row.cells[3].firstChild.tagName=='A')
-      row.cells[3].firstChild.onclick = new Function(this.ref+".command('delete-folder','"+name.replace('\'','\\\'')+"')");
-
     // add new folder to rename-folder list and clear input field
     if (!replace && (form = this.gui_objects.editform))
@@ -2703 +2707 @@
       this.subscription_list.select_row(selection);
 
-    if (document.getElementById('rcmrow'+id).scrollIntoView)
-      document.getElementById('rcmrow'+id).scrollIntoView();
+    if (document.getElementById(id).scrollIntoView)
+      document.getElementById(id).scrollIntoView();
     };
 
@@ -2716 +2720 @@
     // replace an existing table row (if found)
     this.add_folder_row(newfolder, display_name, row);
-    this.env.subscriptionrows[id] = null;
     
     // rename folder in rename-folder dropdown

trunk/roundcubemail/program/js/list.js

@@ -38 +38 @@
   this.draggable = false;
   this.keyboard = false;
+  this.toggleselect = false;
   
   this.dont_select = false;
@@ -348 +349 @@
     this.shift_start = null;
 
+  if (this.toggleselect && this.last_selected==id)
+    this.highlight_row(id, true);
+
   this.last_selected = id;
-  this.set_classname(this.rows[id].obj, 'focused', true);        
+  this.set_classname(this.rows[id].obj, 'focused', true);
 },
 

trunk/roundcubemail/program/steps/settings/manage_folders.inc

@@ -72 +72 @@
 else if ($_action=='rename-folder')
   {
-  $a_mboxes = array_merge($IMAP->list_mailboxes(), $IMAP->list_unsubscribed());
+  $a_mboxes = array_unique(array_merge($IMAP->list_mailboxes(), $IMAP->list_unsubscribed()));
   $delimiter = $IMAP->get_hierarchy_delimiter();
 
@@ -196 +196 @@
       $a_js_folders['rcmrow'.($i+1)] = array($folder, rcube_charset_convert($folder, 'UTF-7'));
 
-    $out .= sprintf('<tr id="rcmrow%d" class="%s"' .
-                    ' onmouseover="return %s.focus_subscription(\'%s\')"' .
-                    ' onmouseout="return %s.unfocus_subscription(\'%s\')"><td>%s</td>',
+    $out .= sprintf('<tr id="rcmrow%d" class="%s"><td>%s</td>',
                     $i+1,
                     $zebra_class,
-                    JS_OBJECT_NAME,
-                    $folder_js,
-                    JS_OBJECT_NAME,
-                    $folder_js,
                     Q($folder_html));
                     
@@ -214 +208 @@
     // add rename and delete buttons
     if (!$protected)
-      $out .= sprintf('<td><a href="#rename" onclick="%s.command(\'rename-folder\',\'%s\')" title="%s">%s</a>'.
-                      '<td><a href="#delete" onclick="%s.command(\'delete-folder\',\'%s\')" title="%s">%s</a></td>',
-                      JS_OBJECT_NAME,
-                      $folder_js,
+      $out .= sprintf('<td><a href="#rename" title="%s">%s</a>'.
+                      '<td><a href="#delete" title="%s">%s</a></td>',
                       rcube_label('renamefolder'),
                       $edit_button,
-                      JS_OBJECT_NAME,
-                      $folder_js,
                       rcube_label('deletefolder'),
                       $del_button);