From a61bbb24aafec5718ca9bc985e7c596c5821f018 Mon Sep 17 00:00:00 2001 From: thomascube Date: Fri, 26 Mar 2010 16:38:20 +0000 Subject: [PATCH] Added basic contact groups feature --- CHANGELOG | 1 + SQL/mysql.initial.sql | 25 +++ SQL/mysql.update.sql | 23 +++ SQL/sqlite.initial.sql | 22 ++- SQL/sqlite.update.sql | 18 ++ config/db.inc.php.dist | 4 + index.php | 4 + program/include/rcmail.php | 14 +- program/include/rcube_addressbook.php | 14 ++ program/include/rcube_contacts.php | 180 ++++++++++++++++- program/js/app.js | 204 +++++++++++++++----- program/localization/de_CH/labels.inc | 3 + program/localization/de_CH/messages.inc | 2 + program/localization/en_US/labels.inc | 3 + program/localization/en_US/messages.inc | 2 + program/steps/addressbook/func.inc | 32 ++- program/steps/addressbook/groups.inc | 61 ++++++ program/steps/mail/autocomplete.inc | 34 +++- skins/default/addressbook.css | 99 ++++++++-- skins/default/functions.js | 75 ++++--- skins/default/images/icons/folders.png | Bin 3392 -> 4771 bytes skins/default/images/icons/groupactions.png | Bin 0 -> 660 bytes skins/default/images/listheader.gif | Bin 538 -> 487 bytes skins/default/mail.css | 2 +- skins/default/templates/addressbook.html | 37 ++-- skins/default/templates/mail.html | 3 +- 26 files changed, 729 insertions(+), 133 deletions(-) create mode 100644 program/steps/addressbook/groups.inc create mode 100644 skins/default/images/icons/groupactions.png diff --git a/CHANGELOG b/CHANGELOG index 80dd8c6b0..548043029 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,6 +1,7 @@ CHANGELOG RoundCube Webmail =========================== +- Added contact groups in address book (not finished yet) - Added PageUp/PageDown/Home/End keys support on lists (#1486430) - Added possibility to select all messages in a folder (#1484756) - Added 'imap_force_caps' option for after-login CAPABILITY checking (#1485750) diff --git a/SQL/mysql.initial.sql b/SQL/mysql.initial.sql index 7a22a60f6..c201beae5 100644 --- a/SQL/mysql.initial.sql +++ b/SQL/mysql.initial.sql @@ -94,6 +94,31 @@ CREATE TABLE `contacts` ( INDEX `user_contacts_index` (`user_id`,`email`) ) /*!40000 ENGINE=INNODB */ /*!40101 CHARACTER SET utf8 COLLATE utf8_general_ci */; +-- Table structure for table `contactgroups` + +CREATE TABLE `contactgroups` ( + `contactgroup_id` int(10) UNSIGNED NOT NULL AUTO_INCREMENT, + `user_id` int(10) UNSIGNED NOT NULL DEFAULT '0', + `changed` datetime NOT NULL DEFAULT '0000-00-00 00:00:00', + `del` tinyint(1) NOT NULL DEFAULT '0', + `name` varchar(128) NOT NULL DEFAULT '', + PRIMARY KEY(`contactgroup_id`), + CONSTRAINT `user_id_fk_contactgroups` FOREIGN KEY (`user_id`) + REFERENCES `users`(`user_id`) ON DELETE CASCADE ON UPDATE CASCADE, + INDEX `contactgroups_user_index` (`user_id`,`del`) +) /*!40000 ENGINE=INNODB */ /*!40101 CHARACTER SET utf8 COLLATE utf8_general_ci */; + +CREATE TABLE `contactgroupmembers` ( + `contactgroup_id` int(10) UNSIGNED NOT NULL, + `contact_id` int(10) UNSIGNED NOT NULL DEFAULT '0', + `created` datetime NOT NULL DEFAULT '0000-00-00 00:00:00', + PRIMARY KEY (`contactgroup_id`, `contact_id`), + CONSTRAINT `contactgroup_id_fk_contactgroups` FOREIGN KEY (`contactgroup_id`) + REFERENCES `contactgroups`(`contactgroup_id`) ON DELETE CASCADE ON UPDATE CASCADE, + CONSTRAINT `contact_id_fk_contacts` FOREIGN KEY (`contact_id`) + REFERENCES `contacts`(`contact_id`) ON DELETE CASCADE ON UPDATE CASCADE +) /*!40000 ENGINE=INNODB */; + -- Table structure for table `identities` diff --git a/SQL/mysql.update.sql b/SQL/mysql.update.sql index c4a897903..5de1f57b2 100644 --- a/SQL/mysql.update.sql +++ b/SQL/mysql.update.sql @@ -83,4 +83,27 @@ ALTER TABLE `contacts` ALTER `surname` SET DEFAULT ''; ALTER TABLE `identities` ADD INDEX `user_identities_index` (`user_id`, `del`); +CREATE TABLE `contactgroups` ( + `contactgroup_id` int(10) UNSIGNED NOT NULL AUTO_INCREMENT, + `user_id` int(10) UNSIGNED NOT NULL DEFAULT '0', + `changed` datetime NOT NULL DEFAULT '0000-00-00 00:00:00', + `del` tinyint(1) NOT NULL DEFAULT '0', + `name` varchar(128) NOT NULL DEFAULT '', + PRIMARY KEY(`contactgroup_id`), + CONSTRAINT `user_id_fk_contactgroups` FOREIGN KEY (`user_id`) + REFERENCES `users`(`user_id`) ON DELETE CASCADE ON UPDATE CASCADE, + INDEX `contactgroups_user_index` (`user_id`,`del`) +) /*!40000 ENGINE=INNODB */ /*!40101 CHARACTER SET utf8 COLLATE utf8_general_ci */; + +CREATE TABLE `contactgroupmembers` ( + `contactgroup_id` int(10) UNSIGNED NOT NULL, + `contact_id` int(10) UNSIGNED NOT NULL DEFAULT '0', + `created` datetime NOT NULL DEFAULT '0000-00-00 00:00:00', + PRIMARY KEY (`contactgroup_id`, `contact_id`), + CONSTRAINT `contactgroup_id_fk_contactgroups` FOREIGN KEY (`contactgroup_id`) + REFERENCES `contactgroups`(`contactgroup_id`) ON DELETE CASCADE ON UPDATE CASCADE, + CONSTRAINT `contact_id_fk_contacts` FOREIGN KEY (`contact_id`) + REFERENCES `contacts`(`contact_id`) ON DELETE CASCADE ON UPDATE CASCADE +) /*!40000 ENGINE=INNODB */; + /*!40014 SET FOREIGN_KEY_CHECKS=1 */; diff --git a/SQL/sqlite.initial.sql b/SQL/sqlite.initial.sql index 30546bbb2..8929c62ea 100644 --- a/SQL/sqlite.initial.sql +++ b/SQL/sqlite.initial.sql @@ -19,7 +19,7 @@ CREATE INDEX ix_cache_created ON cache(created); -- -------------------------------------------------------- -- --- Table structure for table contacts +-- Table structure for table contacts and related -- CREATE TABLE contacts ( @@ -36,6 +36,26 @@ CREATE TABLE contacts ( CREATE INDEX ix_contacts_user_id ON contacts(user_id, email); + +CREATE TABLE contactgroups ( + contactgroup_id integer NOT NULL PRIMARY KEY, + user_id integer NOT NULL default '0', + changed datetime NOT NULL default '0000-00-00 00:00:00', + del tinyint NOT NULL default '0', + name varchar(128) NOT NULL default '' +); + +CREATE INDEX ix_contactgroups_user_id ON contactgroups(user_id, del); + + +CREATE TABLE contactgroupmembers ( + contactgroup_id integer NOT NULL, + contact_id integer NOT NULL default '0', + created datetime NOT NULL default '0000-00-00 00:00:00', + PRIMARY KEY (contactgroup_id, contact_id) +); + + -- -------------------------------------------------------- -- diff --git a/SQL/sqlite.update.sql b/SQL/sqlite.update.sql index fd90d48f6..1011f52c2 100644 --- a/SQL/sqlite.update.sql +++ b/SQL/sqlite.update.sql @@ -47,3 +47,21 @@ CREATE INDEX ix_contacts_user_id ON contacts(user_id, email); DROP INDEX ix_identities_user_id; CREATE INDEX ix_identities_user_id ON identities (user_id, del); + +CREATE TABLE contactgroups ( + contactgroup_id integer NOT NULL PRIMARY KEY, + user_id integer NOT NULL default '0', + changed datetime NOT NULL default '0000-00-00 00:00:00', + del tinyint NOT NULL default '0', + name varchar(128) NOT NULL default '' +); + +CREATE INDEX ix_contactgroups_user_id ON contactgroups(user_id, del); + +CREATE TABLE contactgroupmembers ( + contactgroup_id integer NOT NULL, + contact_id integer NOT NULL default '0', + created datetime NOT NULL default '0000-00-00 00:00:00', + PRIMARY KEY (contactgroup_id, contact_id) +); + diff --git a/config/db.inc.php.dist b/config/db.inc.php.dist index eeca64718..a7cfe2120 100644 --- a/config/db.inc.php.dist +++ b/config/db.inc.php.dist @@ -44,6 +44,10 @@ $rcmail_config['db_table_identities'] = 'identities'; $rcmail_config['db_table_contacts'] = 'contacts'; +$rcmail_config['db_table_contactgroups'] = 'contactgroups'; + +$rcmail_config['db_table_contactgroupmembers'] = 'contactgroupmembers'; + $rcmail_config['db_table_session'] = 'session'; $rcmail_config['db_table_cache'] = 'cache'; diff --git a/index.php b/index.php index ee4aa45cf..02a6f14be 100644 --- a/index.php +++ b/index.php @@ -217,6 +217,10 @@ $action_map = array( 'addressbook' => array( 'add' => 'edit.inc', + 'create-group' => 'groups.inc', + 'delete-group' => 'groups.inc', + 'removefromgroup' => 'groups.inc', + 'add2group' => 'groups.inc', ), 'settings' => array( diff --git a/program/include/rcmail.php b/program/include/rcmail.php index 548ca6b12..1224822c6 100644 --- a/program/include/rcmail.php +++ b/program/include/rcmail.php @@ -300,19 +300,21 @@ class rcmail if ($abook_type != 'ldap') { $list['0'] = array( 'id' => 0, - 'name' => rcube_label('personaladrbook'), + 'name' => rcube_label('personaladrbook'), + 'groups' => true, 'readonly' => false, - 'autocomplete' => in_array('sql', $autocomplete) + 'autocomplete' => in_array('sql', $autocomplete) ); } if (is_array($ldap_config)) { foreach ($ldap_config as $id => $prop) $list[$id] = array( - 'id' => $id, - 'name' => $prop['name'], - 'readonly' => !$prop['writable'], - 'autocomplete' => in_array('sql', $autocomplete) + 'id' => $id, + 'name' => $prop['name'], + 'groups' => false, + 'readonly' => !$prop['writable'], + 'autocomplete' => in_array('sql', $autocomplete) ); } diff --git a/program/include/rcube_addressbook.php b/program/include/rcube_addressbook.php index b691ee8d5..ff525bc11 100644 --- a/program/include/rcube_addressbook.php +++ b/program/include/rcube_addressbook.php @@ -29,6 +29,7 @@ abstract class rcube_addressbook { /** public properties */ var $primary_key; + var $groups = false; var $readonly = true; var $ready = false; var $list_page = 1; @@ -62,6 +63,13 @@ abstract class rcube_addressbook */ abstract function list_records($cols=null, $subset=0); + /** + * List all active contact groups of this source + * + * @return array Indexed list of contact groups, each a hash array + */ + function list_groups() { } + /** * Search records * @@ -123,6 +131,12 @@ abstract class rcube_addressbook $this->page_size = (int)$size; } + /** + * Setter for the current group + * (empty, has to be re-implemented by extending class) + */ + function set_group($gid) { } + /** * Create a new contact record * diff --git a/program/include/rcube_contacts.php b/program/include/rcube_contacts.php index 23f86e852..41f47eae3 100644 --- a/program/include/rcube_contacts.php +++ b/program/include/rcube_contacts.php @@ -39,8 +39,10 @@ class rcube_contacts extends rcube_addressbook /** public properties */ var $primary_key = 'contact_id'; var $readonly = false; + var $groups = true; var $list_page = 1; var $page_size = 10; + var $group_id = 0; var $ready = false; @@ -81,6 +83,16 @@ class rcube_contacts extends rcube_addressbook } + /** + * Setter for the current group + * (empty, has to be re-implemented by extending class) + */ + function set_group($gid) + { + $this->group_id = $gid; + } + + /** * Reset all saved results and search parameters */ @@ -92,6 +104,32 @@ class rcube_contacts extends rcube_addressbook $this->search_string = null; } + /** + * List all active contact groups of this source + * + * @param string Search string to match group name + * @return array Indexed list of contact groups, each a hash array + */ + function list_groups($search = null) + { + $results = array(); + $sql_filter = $search ? "AND " . $this->db->ilike('name', '%'.$search.'%') : ''; + + $sql_result = $this->db->query( + "SELECT * FROM ".get_table_name('contactgroups')." + WHERE del<>1 + AND user_id=? + $sql_filter + ORDER BY name", + $this->user_id); + + while ($sql_result && ($sql_arr = $this->db->fetch_assoc($sql_result))) { + $sql_arr['ID'] = $sql_arr['contactgroup_id']; + $results[] = $sql_arr; + } + + return $results; + } /** * List the current set of contact records @@ -109,18 +147,24 @@ class rcube_contacts extends rcube_addressbook // get contacts from DB if ($this->result->count) { + if ($this->group_id) + $join = "LEFT JOIN ".get_table_name('contactgroupmembers')." AS rcmgrouplinks". + " ON (rcmgrouplinks.contact_id=rcmcontacts.".$this->primary_key.")"; + $start_row = $subset < 0 ? $this->result->first + $this->page_size + $subset : $this->result->first; $length = $subset != 0 ? abs($subset) : $this->page_size; $sql_result = $this->db->limitquery( - "SELECT * FROM ".$this->db_name." - WHERE del<>1 - AND user_id=?" . + "SELECT * FROM ".$this->db_name." AS rcmcontacts ".$join." + WHERE rcmcontacts.del<>1 + AND rcmcontacts.user_id=?" . + ($this->group_id ? " AND rcmgrouplinks.contactgroup_id=?" : ""). ($this->filter ? " AND (".$this->filter.")" : "") . - " ORDER BY name", + " ORDER BY rcmcontacts.name", $start_row, $length, - $this->user_id); + $this->user_id, + $this->group_id); } while ($sql_result && ($sql_arr = $this->db->fetch_assoc($sql_result))) @@ -184,14 +228,20 @@ class rcube_contacts extends rcube_addressbook */ function count() { + if ($this->group_id) + $join = "LEFT JOIN ".get_table_name('contactgroupmembers')." AS rcmgrouplinks". + " ON (rcmgrouplinks.contact_id=rcmcontacts.".$this->primary_key.")"; + // count contacts for this user $sql_result = $this->db->query( - "SELECT COUNT(contact_id) AS rows - FROM ".$this->db_name." - WHERE del<>1 - AND user_id=?". + "SELECT COUNT(rcmcontacts.contact_id) AS rows + FROM ".$this->db_name." AS rcmcontacts ".$join." + WHERE rcmcontacts.del<>1 + AND rcmcontacts.user_id=?". + ($this->group_id ? " AND rcmgrouplinks.contactgroup_id=?" : ""). ($this->filter ? " AND (".$this->filter.")" : ""), - $this->user_id); + $this->user_id, + $this->group_id); $sql_arr = $this->db->fetch_assoc($sql_result); return new rcube_result_set($sql_arr['rows'], ($this->list_page-1) * $this->page_size);; @@ -357,4 +407,114 @@ class rcube_contacts extends rcube_addressbook return $this->db->affected_rows(); } + + /** + * Create a contact group with the given name + * + * @param string The group name + * @return False on error, array with record props in success + */ + function create_group($name) + { + $result = false; + + $sql_result = $this->db->query( + "SELECT * FROM ".get_table_name('contactgroups')." + WHERE del<>1 + AND user_id=? + AND name LIKE ?", + $this->user_id, + $name . '%'); + + // make sure we have a unique name + if ($num = $this->db->num_rows($sql_result)) + $name .= ' ' . ($num+1); + + $this->db->query( + "INSERT INTO ".get_table_name('contactgroups')." (user_id, changed, name) + VALUES (".intval($this->user_id).", ".$this->db->now().", ".$this->db->quote($name).")" + ); + + if ($insert_id = $this->db->insert_id('contactgroups')) + $result = array('id' => $insert_id, 'name' => $name); + + return $result; + } + + /** + * Delete the given group and all linked group members + * + * @param string Group identifier + */ + function delete_group($gid) + { + $sql_result = $this->db->query( + "DELETE FROM ".get_table_name('contactgroupmembers')." + WHERE contactgroup_id=?", + $gid); + + $sql_result = $this->db->query( + "UPDATE ".get_table_name('contactgroups')." + SET del=1, changed=".$this->db->now()." + WHERE contactgroup_id=?", + $gid); + + return $this->db->affected_rows(); + } + + /** + * Add the given contact records the a certain group + * + * @param string Group identifier + * @param array List of contact identifiers to be added + */ + function add_to_group($group_id, $ids) + { + if (!is_array($ids)) + $ids = explode(',', $ids); + + $added = 0; + + foreach ($ids as $contact_id) { + $sql_result = $this->db->query( + "SELECT 1 FROM ".get_table_name('contactgroupmembers')." + WHERE contactgroup_id=? + AND contact_id=?", + $group_id, + $contact_id); + + if (!$this->db->num_rows($sql_result)) { + $this->db->query( + "INSERT INTO ".get_table_name('contactgroupmembers')." (contactgroup_id, contact_id, created) + VALUES (".intval($group_id).", ".intval($contact_id).", ".$this->db->now().")" + ); + if (!$this->db->db_error) + $added++; + } + } + + return $added; + } + + + /** + * Remove the given contact records from a certain group + * + * @param string Group identifier + * @param array List of contact identifiers to be removed + */ + function remove_from_group($group_id, $ids) + { + if (!is_array($ids)) + $ids = explode(',', $ids); + + $sql_result = $this->db->query( + "DELETE FROM ".get_table_name('contactgroupmembers')." + WHERE contactgroup_id=? + AND contact_id IN (".join(',', array_map(array($this->db, 'quote'), $ids)).")", + $group_id); + + return $this->db->affected_rows(); + } + } diff --git a/program/js/app.js b/program/js/app.js index a9838bc57..5330fb0da 100644 --- a/program/js/app.js +++ b/program/js/app.js @@ -284,9 +284,13 @@ function rcube_webmail() case 'addressbook': + if (this.gui_objects.folderlist) + this.env.contactfolders = $.extend($.extend({}, this.env.address_sources), this.env.contactgroups); + if (this.gui_objects.contactslist) { - this.contact_list = new rcube_list_widget(this.gui_objects.contactslist, {multiselect:true, draggable:true, keyboard:true}); + this.contact_list = new rcube_list_widget(this.gui_objects.contactslist, + {multiselect:true, draggable:this.gui_objects.folderlist?true:false, keyboard:true}); this.contact_list.row_init = function(row){ p.triggerEvent('insertrow', { cid:row.uid, row:row }); }; this.contact_list.addEventListener('keypress', function(o){ p.contactlist_keypress(o); }); this.contact_list.addEventListener('select', function(o){ p.contactlist_select(o); }); @@ -306,13 +310,15 @@ function rcube_webmail() else this.contact_list.focus(); - this.gui_objects.folderlist = this.gui_objects.contactslist; + //this.gui_objects.folderlist = this.gui_objects.contactslist; } this.set_page_buttons(); - if (this.env.address_sources && this.env.address_sources[this.env.source] && !this.env.address_sources[this.env.source].readonly) + if (this.env.address_sources && this.env.address_sources[this.env.source] && !this.env.address_sources[this.env.source].readonly) { this.enable_command('add', 'import', true); + this.enable_command('add-group', this.env.address_sources[this.env.source].groups); + } if (this.env.cid) this.enable_command('show', 'edit', true); @@ -325,7 +331,7 @@ function rcube_webmail() if (this.contact_list && this.contact_list.rowcount > 0) this.enable_command('export', true); - this.enable_command('list', true); + this.enable_command('list', 'listgroup', true); break; @@ -489,9 +495,8 @@ function rcube_webmail() case 'menu-open': case 'menu-save': - this.triggerEvent(command, {props:props}); - return false; - break; + this.triggerEvent(command, {props:props}); + return false; case 'open': var uid; @@ -525,6 +530,11 @@ function rcube_webmail() break; + case 'listgroup': + this.list_contacts(null, props); + break; + + case 'load-headers': this.load_headers(obj); break; @@ -969,7 +979,7 @@ function rcube_webmail() case 'add-contact': this.add_contact(props); break; - + // quicksearch case 'search': if (!props && this.gui_objects.qsearchbox) @@ -988,7 +998,11 @@ function rcube_webmail() if (s && this.env.mailbox) this.list_mailbox(this.env.mailbox); else if (s && this.task == 'addressbook') - this.list_contacts(this.env.source); + this.list_contacts(this.env.source, this.env.group); + break; + + case 'add-group': + this.add_contact_group(props) break; case 'import': @@ -1191,7 +1205,7 @@ function rcube_webmail() if (!rcube_mouse_is_over(e, this.contact_list.list)) this.contact_list.blur(); list = this.contact_list; - model = this.env.address_sources; + model = this.env.contactfolders; } else if (this.ksearch_value) { this.ksearch_blur(); @@ -1199,14 +1213,14 @@ function rcube_webmail() // handle mouse release when dragging if (this.drag_active && model && this.env.last_folder_target) { - var mbox = model[this.env.last_folder_target].id; - + var target = model[this.env.last_folder_target]; + $(this.get_folder_li(this.env.last_folder_target)).removeClass('droptarget'); this.env.last_folder_target = null; list.draglayer.hide(); - if (!this.drag_menu(e, mbox)) - this.command('moveto', mbox); + if (!this.drag_menu(e, target)) + this.command('moveto', target); } // reset 'pressed' buttons @@ -1218,17 +1232,19 @@ function rcube_webmail() } }; - this.drag_menu = function(e, mbox) + this.drag_menu = function(e, target) { var modkey = rcube_event.get_modifier(e); var menu = $('#'+this.gui_objects.message_dragmenu); - if (menu && modkey == SHIFT_KEY) { + if (menu && modkey == SHIFT_KEY && this.commands['copy']) { var pos = rcube_event.get_mouse_pos(e); - this.env.drag_mbox = mbox; + this.env.drag_target = target; menu.css({top: (pos.y-10)+'px', left: (pos.x-10)+'px'}).show(); return true; } + + return false; }; this.drag_menu_action = function(action) @@ -1237,13 +1253,13 @@ function rcube_webmail() if (menu) { menu.hide(); } - this.command(action, this.env.drag_mbox); - this.env.drag_mbox = null; + this.command(action, this.env.drag_target); + this.env.drag_target = null; }; this.drag_start = function(list) { - var model = this.task == 'mail' ? this.env.mailboxes : this.env.address_sources; + var model = this.task == 'mail' ? this.env.mailboxes : this.env.contactfolders; this.drag_active = true; if (this.preview_timer) @@ -1484,7 +1500,9 @@ function rcube_webmail() if (this.task == 'mail') return (this.env.mailboxes[id] && this.env.mailboxes[id].id != this.env.mailbox && !this.env.mailboxes[id].virtual); else if (this.task == 'addressbook') - return (id != this.env.source && this.env.address_sources[id] && !this.env.address_sources[id].readonly); + return (id != this.env.source && this.env.contactfolders[id] && !this.env.contactfolders[id].readonly && + !(!this.env.source && this.env.contactfolders[id].group) && + !(this.env.contactfolders[id].type == 'group' && this.env.contactfolders[id].id == this.env.group)); else if (this.task == 'settings') return (id != this.env.folder); }; @@ -1784,7 +1802,7 @@ function rcube_webmail() if (this.task=='mail') this.list_mailbox(this.env.mailbox, page); else if (this.task=='addressbook') - this.list_contacts(this.env.source, page); + this.list_contacts(this.env.source, null, page); } }; @@ -2174,6 +2192,9 @@ function rcube_webmail() // move selected messages to the specified mailbox this.move_messages = function(mbox) { + if (mbox && typeof mbox == 'object') + mbox = mbox.id; + // exit if current or no mailbox specified or if selection is empty if (!mbox || mbox == this.env.mailbox || (!this.env.uid && (!this.message_list || !this.message_list.get_selection().length))) return; @@ -3097,6 +3118,7 @@ function rcube_webmail() this.http_request('search', '_q='+urlencode(value) + (this.env.mailbox ? '&_mbox='+urlencode(this.env.mailbox) : '') + (this.env.source ? '&_source='+urlencode(this.env.source) : '') + + (this.env.group ? '&_gid='+urlencode(this.env.group) : '') + (addurl ? addurl : ''), true); } return true; @@ -3208,7 +3230,16 @@ function rcube_webmail() // replace search string with full address var pre = this.ksearch_input.value.substring(0, p); var end = this.ksearch_input.value.substring(p+this.ksearch_value.length, this.ksearch_input.value.length); - var insert = this.env.contacts[id]+', '; + var insert = ''; + + // insert all members of a group + if (typeof this.env.contacts[id] == 'object' && this.env.contacts[id].members) { + for (var i=0; i < this.env.contacts[id].members.length; i++) + insert += this.env.contacts[id].members[i] + ', '; + } + else if (typeof this.env.contacts[id] == 'string') + insert = this.env.contacts[id] + ', '; + this.ksearch_input.value = pre + insert + end; // set caret to insert pos @@ -3269,7 +3300,7 @@ function rcube_webmail() { // display search results if (a_results.length && this.ksearch_input && this.ksearch_value) { - var p, ul, li, s_val = this.ksearch_value; + var p, ul, li, text, s_val = this.ksearch_value; // create results pane if not present if (!this.ksearch_pane) { @@ -3283,9 +3314,10 @@ function rcube_webmail() ul.innerHTML = ''; // add each result line to list - for (i=0; i/g, '>').replace(/##([^%]+)%%/g, '$1'); + li.innerHTML = text.replace(new RegExp('('+s_val+')', 'ig'), '##$1%%').replace(//g, '>').replace(/##([^%]+)%%/g, '$1'); li.onmouseover = function(){ ref.ksearch_select(this); }; li.onmouseup = function(){ ref.ksearch_click(this) }; li._rcm_id = i; @@ -3363,15 +3395,19 @@ function rcube_webmail() return false; }; - this.list_contacts = function(src, page) + this.list_contacts = function(src, group, page) { var add_url = ''; var target = window; + // currently all groups belong to the local address book + if (group) + src = 0; + if (!src) src = this.env.source; - if (page && this.current_page==page && src == this.env.source) + if (page && this.current_page == page && src == this.env.source && group == this.env.group) return false; if (src != this.env.source) @@ -3380,14 +3416,19 @@ function rcube_webmail() this.env.current_page = page; this.reset_qsearch(); } + else if (group != this.env.group) + page = this.env.current_page = 1; this.select_folder(src, this.env.source); + this.select_folder(group, this.env.group, 'rcmliG'); + this.env.source = src; + this.env.group = group; // load contacts remotely if (this.gui_objects.contactslist) { - this.list_contacts_remote(src, page); + this.list_contacts_remote(src, group, page); return; } @@ -3396,17 +3437,22 @@ function rcube_webmail() target = window.frames[this.env.contentframe]; add_url = '&_framed=1'; } + + if (group) + add_url += '&_gid='+group; + if (page) + add_url += '&_page='+page; // also send search request to get the correct listing if (this.env.search_request) add_url += '&_search='+this.env.search_request; this.set_busy(true, 'loading'); - target.location.href = this.env.comm_path+(src ? '&_source='+urlencode(src) : '')+(page ? '&_page='+page : '')+add_url; + target.location.href = this.env.comm_path + (src ? '&_source='+urlencode(src) : '') + add_url; }; // send remote request to load contacts list - this.list_contacts_remote = function(src, page) + this.list_contacts_remote = function(src, group, page) { // clear message list first this.contact_list.clear(true); @@ -3416,6 +3462,10 @@ function rcube_webmail() // send request to server var url = (src ? '_source='+urlencode(src) : '') + (page ? (src?'&':'') + '_page='+page : ''); this.env.source = src; + this.env.group = group; + + if (group) + url += '&_gid='+group; // also send search request to get the right messages if (this.env.search_request) @@ -3453,8 +3503,10 @@ function rcube_webmail() if (!cid) cid = this.contact_list.get_selection().join(','); - if (to != this.env.source && cid && this.env.address_sources[to] && !this.env.address_sources[to].readonly) - this.http_post('copy', '_cid='+urlencode(cid)+'&_source='+urlencode(this.env.source)+'&_to='+urlencode(to)); + if (to.type == 'group') + this.http_post('add2group', '_cid='+urlencode(cid)+'&_source='+urlencode(this.env.source)+'&_gid='+urlencode(to.id)); + else if (to.id != this.env.source && cid && this.env.address_sources[to.id] && !this.env.address_sources[to.id].readonly) + this.http_post('copy', '_cid='+urlencode(cid)+'&_source='+urlencode(this.env.source)+'&_to='+urlencode(to.id)); }; @@ -3462,7 +3514,7 @@ function rcube_webmail() { // exit if no mailbox specified or if selection is empty var selection = this.contact_list.get_selection(); - if (!(selection.length || this.env.cid) || !confirm(this.get_label('deletecontactconfirm'))) + if (!(selection.length || this.env.cid) || (!this.env.group && !confirm(this.get_label('deletecontactconfirm')))) return; var a_cids = new Array(); @@ -3490,7 +3542,11 @@ function rcube_webmail() qs += '&_search='+this.env.search_request; // send request to server - this.http_post('delete', '_cid='+urlencode(a_cids.join(','))+'&_source='+urlencode(this.env.source)+'&_from='+(this.env.action ? this.env.action : '')+qs); + if (this.env.group) + this.http_post('removefromgroup', '_cid='+urlencode(a_cids.join(','))+'&_source='+urlencode(this.env.source)+'&_gid='+urlencode(this.env.group)+qs); + else + this.http_post('delete', '_cid='+urlencode(a_cids.join(','))+'&_source='+urlencode(this.env.source)+'&_from='+(this.env.action ? this.env.action : '')+qs); + return true; }; @@ -3505,11 +3561,11 @@ function rcube_webmail() // cid change if (newcid) { - row.id = 'rcmrow' + newcid; + row.id = 'rcmrow' + newcid; this.contact_list.remove_row(cid); this.contact_list.init_row(row); - this.contact_list.selection[0] = newcid; - row.style.display = ''; + this.contact_list.selection[0] = newcid; + ow.style.display = ''; } return true; @@ -3547,6 +3603,59 @@ function rcube_webmail() this.enable_command('export', (this.contact_list.rowcount > 0)); }; + + + this.add_contact_group = function() + { + if (!this.gui_objects.folderlist || !this.env.address_sources[this.env.source].groups) + return; + + if (!this.name_input) { + this.name_input = document.createElement('input'); + this.name_input.type = 'text'; + this.name_input.onkeypress = function(e){ return rcmail.add_input_keypress(e); }; + + this.gui_objects.folderlist.parentNode.appendChild(this.name_input); + } + + this.name_input.select(); + }; + + // handler for keyboard events on the input field + this.add_input_keypress = function(e) + { + var key = rcube_event.get_keycode(e); + + // enter + if (key == 13) { + var newname = this.name_input.value; + + if (newname) { + this.set_busy(true, 'loading'); + this.http_post('create-group', '_source='+urlencode(this.env.source)+'&_name='+urlencode(newname), true); + } + return false; + } + // escape + else if (key == 27) + this.reset_add_input(); + + return true; + }; + + this.reset_add_input = function() + { + if (this.name_input) { + this.name_input.parentNode.removeChild(this.name_input); + this.name_input = null; + } + }; + + // callback for creating a new contact group + this.insert_contact_group = function(prop) + { + this.reset_add_input(); + } /*********************************************************/ @@ -4256,31 +4365,33 @@ function rcube_webmail() }; // mark a mailbox as selected and set environment variable - this.select_folder = function(name, old) + this.select_folder = function(name, old, prefix) { if (this.gui_objects.folderlist) { var current_li, target_li; - if ((current_li = this.get_folder_li(old))) { + if ((current_li = this.get_folder_li(old, prefix))) { $(current_li).removeClass('selected').removeClass('unfocused'); } - if ((target_li = this.get_folder_li(name))) { + if ((target_li = this.get_folder_li(name, prefix))) { $(target_li).removeClass('unfocused').addClass('selected'); } // trigger event hook - this.triggerEvent('selectfolder', { folder:name, old:old }); + this.triggerEvent('selectfolder', { folder:name, old:old, prefix:prefix }); } }; // helper method to find a folder list item - this.get_folder_li = function(name) + this.get_folder_li = function(name, prefix) { + if (!prefix) + prefix = 'rcmli'; if (this.gui_objects.folderlist) { name = String(name).replace(this.identifier_expr, '_'); - return document.getElementById('rcmli'+name); + return document.getElementById(prefix+name); } return null; @@ -4713,8 +4824,11 @@ function rcube_webmail() else if (this.task == 'addressbook') { this.enable_command('export', (this.contact_list && this.contact_list.rowcount > 0)); - if (response.action == 'list') + if (response.action == 'list') { + this.enable_command('add-group', this.env.address_sources[this.env.source].groups); + // disabeld for now: this.enable_command('rename-group', 'delete-group', this.env.address_sources[this.env.source].groups && this.env.group); this.triggerEvent('listupdate', { folder:this.env.source, rowcount:this.contact_list.rowcount }); + } } break; } diff --git a/program/localization/de_CH/labels.inc b/program/localization/de_CH/labels.inc index cbc5b0898..29252aa30 100644 --- a/program/localization/de_CH/labels.inc +++ b/program/localization/de_CH/labels.inc @@ -214,10 +214,13 @@ $labels['contactsfromto'] = 'Kontakte $from bis $to von $count'; $labels['print'] = 'Drucken'; $labels['export'] = 'Exportieren'; $labels['exportvcards'] = 'Kontakte im vCard-Format exportieren'; +$labels['newcontactgroup'] = 'Neue Adressgruppe erstellen'; +$labels['groupactions'] = 'Aktionen für Kontaktgruppen...'; $labels['previouspage'] = 'Eine Seite zurück'; $labels['firstpage'] = 'Erste Seite'; $labels['nextpage'] = 'Nächste Seite'; $labels['lastpage'] = 'Letzte Seite'; +$labels['group'] = 'Gruppe'; $labels['groups'] = 'Gruppen'; $labels['personaladrbook'] = 'Persönliches Adressbuch'; $labels['import'] = 'Importieren'; diff --git a/program/localization/de_CH/messages.inc b/program/localization/de_CH/messages.inc index 706a690e6..be61cc78d 100644 --- a/program/localization/de_CH/messages.inc +++ b/program/localization/de_CH/messages.inc @@ -93,6 +93,8 @@ $messages['addsubfolderhint'] = 'Wird als Unterordner des aktuell selektierten O $messages['forbiddencharacter'] = 'Der Ordnername enthält ein ungültiges Zeichen'; $messages['selectimportfile'] = 'Bitte wählen Sie eine Datei zum Importieren aus'; $messages['addresswriterror'] = 'Das gewählte Adressbuch kann nicht verändert werden'; +$messages['contactaddedtogroup'] = 'Kontakte wurden dieser Gruppe hinzugefügt'; +$messages['contactremovedfromgroup'] = 'Kontakte wurden aus dieser Gruppe entfernt'; $messages['importwait'] = 'Daten werden importiert, bitte warten...'; $messages['importerror'] = 'Import fehlgeschlagen! Die hochgeladene Datei ist nicht im vCard-Format.'; $messages['importconfirm'] = 'Es wurden $inserted Adressen erfolgreich importiert und $skipped bestehende Einträge übersprungen:

$names

'; diff --git a/program/localization/en_US/labels.inc b/program/localization/en_US/labels.inc index b8c36ec87..10c04ec0f 100644 --- a/program/localization/en_US/labels.inc +++ b/program/localization/en_US/labels.inc @@ -258,12 +258,15 @@ $labels['contactsfromto'] = 'Contacts $from to $to of $count'; $labels['print'] = 'Print'; $labels['export'] = 'Export'; $labels['exportvcards'] = 'Export contacts in vCard format'; +$labels['newcontactgroup'] = 'Create new contact group'; +$labels['groupactions'] = 'Actions for contact groups...'; $labels['previouspage'] = 'Show previous set'; $labels['firstpage'] = 'Show first set'; $labels['nextpage'] = 'Show next set'; $labels['lastpage'] = 'Show last set'; +$labels['group'] = 'Group'; $labels['groups'] = 'Groups'; $labels['personaladrbook'] = 'Personal Addresses'; diff --git a/program/localization/en_US/messages.inc b/program/localization/en_US/messages.inc index ab24e070e..f87cfb579 100644 --- a/program/localization/en_US/messages.inc +++ b/program/localization/en_US/messages.inc @@ -95,6 +95,8 @@ $messages['addsubfolderhint'] = 'This folder will be created as subfolder of the $messages['forbiddencharacter'] = 'Folder name contains a forbidden character'; $messages['selectimportfile'] = 'Please select a file to upload'; $messages['addresswriterror'] = 'The selected address book is not writeable'; +$messages['contactaddedtogroup'] = 'Successfully added the contacts to this group'; +$messages['contactremovedfromgroup'] = 'Successfully remove contacts from this group'; $messages['importwait'] = 'Importing, please wait...'; $messages['importerror'] = 'Import failed! The uploaded file is not a valid vCard file.'; $messages['importconfirm'] = 'Successfully imported $inserted contacts, $skipped existing entries skipped:

$names

'; diff --git a/program/steps/addressbook/func.inc b/program/steps/addressbook/func.inc index ca1fd03fc..ec0662b24 100644 --- a/program/steps/addressbook/func.inc +++ b/program/steps/addressbook/func.inc @@ -39,6 +39,9 @@ if (!empty($_GET['_page'])) $CONTACTS->set_page(($_SESSION['page'] = intval($_GET['_page']))); else $CONTACTS->set_page(isset($_SESSION['page']) ?$_SESSION['page'] : 1); + +if (!empty($_REQUEST['_gid'])) + $CONTACTS->set_group(get_input_value('_gid', RCUBE_INPUT_GPC)); // set message set for search result if (!empty($_REQUEST['_search']) && isset($_SESSION['search'][$_REQUEST['_search']])) @@ -60,7 +63,7 @@ function rcmail_directory_list($attrib) $out = ''; $local_id = '0'; $current = get_input_value('_source', RCUBE_INPUT_GPC); - $line_templ = html::tag('li', array('id' => 'rcmli%s', 'class' => '%s'), + $line_templ = html::tag('li', array('id' => 'rcmli%s', 'class' => 'addressbook %s'), html::a(array('href' => '%s', 'onclick' => "return ".JS_OBJECT_NAME.".command('list','%s',this)"), '%s')); if (!$current && strtolower($RCMAIL->config->get('address_book_type', 'sql')) != 'ldap') { @@ -79,6 +82,8 @@ function rcmail_directory_list($attrib) $out .= sprintf($line_templ, $dom_id, ($current == $id ? 'selected' : ''), Q(rcmail_url(null, array('_source' => $id))), $js_id, (!empty($source['name']) ? Q($source['name']) : Q($id))); } + + $out .= rcmail_contact_groups(array('items' => true)); $OUTPUT->add_gui_object('folderlist', $attrib['id']); @@ -86,6 +91,30 @@ function rcmail_directory_list($attrib) } +function rcmail_contact_groups($attrib) +{ + global $CONTACTS, $OUTPUT; + + if (!$attrib['id']) + $attrib['id'] = 'rcmgroupslist'; + + $groups = $CONTACTS->list_groups(); + $line_templ = html::tag('li', array('id' => 'rcmliG%s', 'class' => 'contactgroup'), + html::a(array('href' => '#', 'onclick' => "return ".JS_OBJECT_NAME.".command('listgroup','%s',this)"), '%s')); + + $jsdata = array(); + foreach ($groups as $group) { + $out .= sprintf($line_templ, $group['ID'], $group['ID'], Q($group['name'])); + $jsdata['G'.$group['ID']] = array('id' => $group['ID'], 'name' => $group['name'], 'type' => 'group'); + } + + $OUTPUT->set_env('contactgroups', $jsdata); + //$OUTPUT->add_gui_object('groupslist', $attrib['id']); + + return $attrib['items'] ? $out : html::tag('ul', $attrib, $out, html::$common_attrib); +} + + // return the message list as HTML table function rcmail_contacts_list($attrib) { @@ -200,6 +229,7 @@ $OUTPUT->set_pagetitle(rcube_label('addressbook')); // register UI objects $OUTPUT->add_handlers(array( 'directorylist' => 'rcmail_directory_list', +// 'groupslist' => 'rcmail_contact_groups', 'addresslist' => 'rcmail_contacts_list', 'addressframe' => 'rcmail_contact_frame', 'recordscountdisplay' => 'rcmail_rowcount_display', diff --git a/program/steps/addressbook/groups.inc b/program/steps/addressbook/groups.inc new file mode 100644 index 000000000..8c7aef64b --- /dev/null +++ b/program/steps/addressbook/groups.inc @@ -0,0 +1,61 @@ + | + +-----------------------------------------------------------------------+ + + $Id$ + +*/ + + +if ($CONTACTS->readonly || !$CONTACTS->groups) { + $OUTPUT->show_message('sourceisreadonly', 'warning'); + $OUTPUT->send(); +} + +if ($RCMAIL->action == 'create-group') { + if (!empty($_POST['_name'])) { + $name = trim(get_input_value('_name', RCUBE_INPUT_POST)); + $created = $CONTACTS->create_group($name); + } + + if ($created && $OUTPUT->ajax_call) { + $OUTPUT->command('insert_contact_group', $created); + } + else if (!$create) { + $OUTPUT->show_message('errorsaving', 'error'); + } +} + +else if ($RCMAIL->action == 'add2group') { + if (($gid = get_input_value('_gid', RCUBE_INPUT_POST)) && ($ids = get_input_value('_cid', RCUBE_INPUT_POST))) + if ($CONTACTS->add_to_group($gid, $ids)) + $OUTPUT->show_message('contactaddedtogroup'); + //else + // $OUTPUT->show_message('erroraddingcontact', 'warning'); +} + +else if ($RCMAIL->action == 'removefromgroup') { + if (($gid = get_input_value('_gid', RCUBE_INPUT_POST)) && ($ids = get_input_value('_cid', RCUBE_INPUT_POST))) + if ($CONTACTS->remove_from_group($gid, $ids)) + $OUTPUT->show_message('contactremovedfromgroup'); + //else + // $OUTPUT->show_message('erroraddingcontact', 'warning'); +} + +// send response +$OUTPUT->send(); + +?> \ No newline at end of file diff --git a/program/steps/mail/autocomplete.inc b/program/steps/mail/autocomplete.inc index 06209c41b..1e56fb222 100644 --- a/program/steps/mail/autocomplete.inc +++ b/program/steps/mail/autocomplete.inc @@ -5,7 +5,7 @@ | program/steps/mail/autocomplete.inc | | | | This file is part of the RoundCube Webmail client | - | Copyright (C) 2008-2009, RoundCube Dev Team | + | Copyright (C) 2008-2010, RoundCube Dev Team | | Licensed under the GNU GPL | | | | PURPOSE: | @@ -23,7 +23,7 @@ $MAXNUM = 15; $contacts = array(); $book_types = (array) $RCMAIL->config->get('autocomplete_addressbooks', 'sql'); -if ($book_types && $search = get_input_value('_search', RCUBE_INPUT_POST, true)) { +if ($book_types && $search = get_input_value('_search', RCUBE_INPUT_GPC, true)) { foreach ($book_types as $id) { $abook = $RCMAIL->get_address_book($id); @@ -32,16 +32,42 @@ if ($book_types && $search = get_input_value('_search', RCUBE_INPUT_POST, true)) if ($result = $abook->search(array('email','name'), $search)) { while ($sql_arr = $result->iterate()) { $contacts[] = format_email_recipient($sql_arr['email'], $sql_arr['name']); - if (count($contacts) >= $MAXNUM) + if (count($contacts) >= $MAXNUM) break 2; } } + + // also list matching contact groups + if ($abook->groups) { + foreach ($abook->list_groups($search) as $group) { + $members = array(); + $abook->reset(); + $abook->set_group($group['ID']); + $result = $abook->list_records(array('email','name')); + while ($result && ($sql_arr = $result->iterate())) + $members[] = format_email_recipient($sql_arr['email'], $sql_arr['name']); + + if (count($members)) { + $contacts[] = array('name' => $group['name'] . ' (' . rcube_label('group') . ')', 'members' => $members); + if (count($contacts) >= $MAXNUM) + break; + } + } + } } - sort($contacts); + usort($contacts, 'contact_results_sort'); } $OUTPUT->command('ksearch_query_results', $contacts, $search); $OUTPUT->send(); + +function contact_results_sort($a, $b) +{ + $name_a = is_array($a) ? $a['name'] : $a; + $name_b = is_array($b) ? $b['name'] : $b; + return strcmp(trim($name_a, '" '), trim($name_b, '" ')); +} + ?> diff --git a/skins/default/addressbook.css b/skins/default/addressbook.css index ce82a1594..456d137b1 100644 --- a/skins/default/addressbook.css +++ b/skins/default/addressbook.css @@ -4,7 +4,7 @@ { position: absolute; top: 45px; - left: 205px; + left: 225px; height: 35px; } @@ -76,8 +76,8 @@ #abookcountbar { position: absolute; - bottom: 16px; - left: 200px; + bottom: 6px; + left: 225px; width: 240px; height: 20px; text-align: left; @@ -94,22 +94,80 @@ position: absolute; top: 85px; right: 20px; - bottom: 40px; - left: 205px; + bottom: 30px; + left: 225px; } -#directorylist +#directorylistbox { position: absolute; top: 85px; - bottom: 40px; + bottom: 30px; left: 20px; - width: 175px; + width: 195px; border: 1px solid #999999; background-color: #F9F9F9; overflow: hidden; } +#directorylistwarp +{ + position: absolute; + top: 20px; + bottom: 22px; + left: 0; + right: 0; + overflow: auto; +} + +#groups-title +{ + position: absolute; + top: 0; + left: 0; + right: 0; +} + +#directorylistbox input +{ + display: absolute; + margin: 2px; +} + +#directoylistbuttons +{ + display: block; + position: absolute; + bottom: 0px; + left: 0px; + right: 0px; + height: 22px; + border-top: 1px solid #999; + background: url('images/listheader.gif') top left repeat-x #CCC; +} + +#directoylistbuttons a.button, +#directoylistbuttons a.buttonPas +{ + display: block; + float: left; + width: 34px; + height: 22px; + padding: 0px; + margin: 0; + overflow: hidden; + background: url('images/icons/groupactions.png') 0 0 no-repeat transparent; + opacity: 0.99; /* this is needed to make buttons appear correctly in Chrome */ +} + +#directoylistbuttons a.groupactions { + background-position: 0 -26px; +} + +#directoylistbuttons a.buttonPas { + opacity: 0.35; +} + #addresslist { position: absolute; @@ -120,6 +178,11 @@ overflow: auto; } +#contactgroupslist +{ + border-top: 1px solid #999; +} + #importbox { position: absolute; @@ -136,7 +199,7 @@ #addresslist { left: 0px; - width: 340px; + width: 280px; } #importbox a @@ -144,7 +207,7 @@ color: blue; } -#directorylist ul +#directorylist { list-style: none; margin: 0; @@ -152,23 +215,31 @@ background-color: #FFFFFF; } -#directorylist ul li +#directorylist li { + display: block; font-size: 11px; + background: url(images/icons/folders.png) 5px -108px no-repeat; border-bottom: 1px solid #EBEBEB; white-space: nowrap; } -#directorylist ul li a +#directorylist li a { + cursor: default; display: block; - padding-left: 6px; + padding-left: 25px; padding-top: 2px; padding-bottom: 2px; text-decoration: none; white-space: nowrap; } +#directorylist li.contactgroup +{ + background-position: 5px -144px; +} + #directorylist li.selected { background-color: #929292; @@ -201,7 +272,7 @@ { position: absolute; top: 0px; - left: 555px; + left: 290px; right: 0px; bottom: 0px; border: 1px solid #999999; diff --git a/skins/default/functions.js b/skins/default/functions.js index ca4c6694b..6d99e921d 100644 --- a/skins/default/functions.js +++ b/skins/default/functions.js @@ -126,37 +126,52 @@ function rcmail_init_compose_form() function rcube_mail_ui() { - this.markmenu = $('#markmessagemenu'); - this.searchmenu = $('#searchmenu'); - this.messagemenu = $('#messagemenu'); - this.listmenu = $('#listmenu'); - this.dragmessagemenu = $('#dragmessagemenu'); + this.popupmenus = { + markmenu:'markmessagemenu', + searchmenu:'searchmenu', + messagemenu:'messagemenu', + listmenu:'listmenu', + dragmessagemenu:'dragmessagemenu', + groupmenu:'groupoptionsmenu' + }; + + var obj; + for (var k in this.popupmenus) { + obj = $('#'+this.popupmenus[k]) + if (obj.length) + this[k] = obj; + } } rcube_mail_ui.prototype = { -show_markmenu: function(show) +show_popupmenu: function(obj, refname, show, above) { if (typeof show == 'undefined') - show = this.markmenu.is(':visible') ? false : true; + show = obj.is(':visible') ? false : true; - var ref = rcube_find_object('markreadbutton'); - if (show && ref) - this.markmenu.css({ left:ref.offsetLeft, top:(ref.offsetTop + ref.offsetHeight) }); + var ref = rcube_find_object(refname); + if (show && ref) { + var pos = $(ref).offset(); + obj.css({ left:pos.left, top:(pos.top + (above ? -obj.height() : ref.offsetHeight)) }); + } - this.markmenu[show?'show':'hide'](); + obj[show?'show':'hide'](); }, -show_messagemenu: function(show) +show_markmenu: function(show) { - if (typeof show == 'undefined') - show = this.messagemenu.is(':visible') ? false : true; + this.show_popupmenu(this.markmenu, 'markreadbutton', show); +}, - var ref = rcube_find_object('messagemenulink'); - if (show && ref) - this.messagemenu.css({ left:ref.offsetLeft, top:(ref.offsetTop + ref.offsetHeight) }); +show_messagemenu: function(show) +{ + this.show_popupmenu(this.messagemenu, 'messagemenulink', show); +}, - this.messagemenu[show?'show':'hide'](); +show_groupmenu: function(show) +{ + this.show_popupmenu(this.groupmenu, 'groupactionslink', show, true); }, show_searchmenu: function(show) @@ -267,6 +282,8 @@ body_mouseup: function(evt, p) this.show_messagemenu(false); else if (this.dragmessagemenu && this.dragmessagemenu.is(':visible') && !rcube_mouse_is_over(evt, rcube_find_object('dragmessagemenu'))) this.dragmessagemenu.hide(); + else if (this.groupmenu && this.groupmenu.is(':visible') && target != rcube_find_object('groupactionslink')) + this.show_groupmenu(false); else if (this.listmenu && this.listmenu.is(':visible') && target != rcube_find_object('listmenulink')) { var menu = rcube_find_object('listmenu'); while (target.parentNode) { @@ -290,16 +307,10 @@ body_mouseup: function(evt, p) body_keypress: function(evt, p) { if (rcube_event.get_keycode(evt) == 27) { - if (this.markmenu && this.markmenu.is(':visible')) - this.show_markmenu(false); - if (this.searchmenu && this.searchmenu.is(':visible')) - this.show_searchmenu(false); - if (this.messagemenu && this.messagemenu.is(':visible')) - this.show_messagemenu(false); - if (this.listmenu && this.listmenu.is(':visible')) - this.show_listmenu(false); - if (this.dragmessagemenu && this.dragmessagemenu.is(':visible')) - this.dragmessagemenu.hide(); + for (var k in this.popupmenus) { + if (this[k] && this[k].is(':visible')) + this[k].hide(); + } } } @@ -312,7 +323,9 @@ function rcube_init_mail_ui() rcmail_ui = new rcube_mail_ui(); rcube_event.add_listener({ object:rcmail_ui, method:'body_mouseup', event:'mouseup' }); rcube_event.add_listener({ object:rcmail_ui, method:'body_keypress', event:'keypress' }); - rcmail.addEventListener('menu-open', 'open_listmenu', rcmail_ui); - rcmail.addEventListener('menu-save', 'save_listmenu', rcmail_ui); - rcmail.gui_object('message_dragmenu', 'dragmessagemenu'); + if (rcmail.env.task == 'mail') { + rcmail.addEventListener('menu-open', 'open_listmenu', rcmail_ui); + rcmail.addEventListener('menu-save', 'save_listmenu', rcmail_ui); + rcmail.gui_object('message_dragmenu', 'dragmessagemenu'); + } } diff --git a/skins/default/images/icons/folders.png b/skins/default/images/icons/folders.png index dcd68842526a1174e37df0915455728e5b76c4b7..5013318f8584a6c7d9661d2d3d7ffa245e0b4475 100644 GIT binary patch delta 4756 zcmV;F5^L?i8lxqU7Yd>X1^@s6NbyY{ks&945=BWwK~#9!>{@r2R8^M$y~?SpbIvFk zX=s8Hl%VXkfXtdeQ9=C_*YBHgWE^L45EJU)NEk+BO#>=Ax{VAf#@SIs3=Jy{EHts9 zb9Jh!uIkEFuio2pZ-uU+o22{4e6#yLF7@iwJHLC*xhLM+f-KAUDVOk*0b)06*RH*P z!R2yoJM#sz**t&g(xqF^3$SI&mgbo=XBHO}6!7n%PzVhT4cNYY`$4DEIVKzqgK--h zYu2pc@2O${ZEbDE!C(+ajvRp~ieUE)Wcu{!6(4-?fqb^f`G1=}{VVGJ_PIQF?p(Ot zZb*{E2b(oBGc!)XW@gXcefM4La!nF{>Z|XGb6~lwY7T5c8NCWUJfaM5P=Y%^!|s9C z8-|BQz(-;bNt-m0(R@c^G6xXDD);;;6JatL!Fi?sWa7<>$Z*nd(dZ4*b0zR?jT>)m z*lLgKz!QMo>!V+_Mg+p(!!HssapDdU01Zz_X753%r|(uD1VOMeS_KCPD!Mv<++K*1 z3_raX2ubjTh;1eY4Id$TiG6=Cj1Yl3ZEn(@qN|SszueN|@*$sQ_LAJbu*_$75mPp= zh#r?n?=T<3|aitpZgUNh#J`Di?L41SM?j76f^ zXhtz&IAWL)OOPc23Y`I>-;MU~tIdp7h+3nDhBnHPGCG6-%$J!-I68cPH2nCfSA3R7 zS3CiSqvwK%($doL^vWlay^+b9ot=$k%YJiC;#I4jhu`1Fc^n`V3S!96!KkU(i_;Cw z&}p?WrKCWuR+BS%Id|pdr6?;a!J$Kk2LcFT#PH#0Y-}c%cJX1gS(9dOZ|}gcVZ%^c zyFXb6On$m5X_kE#sDp8T^ij+7po+;)*UcB9gIgFtTm&XRUDGc@2UjNPfXPqS$cxay z$RvQ26mzB9<4!3nEycdtgUHCp;L4$(L8d3i0wQtYL~&6Oxq~9bWHePWF=9HnIX#Ws zlvrgivUOG}6a*qf)JzzXEb-smN|Gg>IN8Ww zR4Nr%`rrek17*~Idn1=9hVkw9dx?`Ei6ZtlY+S52o)Zy22cTdLb`)o(>u2VP^@Z73rqt&{{ zEw5Pd*pFf$?#_=twt|OkK{!hxnNX^-|DIgAvTqrMDAnoH4O|5FZQl?3aNt0s8XGfa z491V25I5oSLFG^iiatwZ8JH0&6tO6zL|xr~Vf=H|wS3go)$wNnC0qp?G=oy5>f=DE zq#p^96;zR`M8N2Z3Vi>k?zm$SmOr|jbDboA zg4JrJB6AOJyZsK_ddmW;#{B5&>WmYXloU~e{uaLZW;c4By~zMf1jgP9vIURZgW)4a zQcc*46f-Y_NP!xZQ58qsJ*ujzSbcORfWVdL^Z77t{5UezVkndy&>M_V05!6+GBJ_r zg@XqV(sKh}i9QinK}85wn;ivIjj^VGi@lJVYUVE;wn$j|z8j+pieR>V$!#Tx18zJd zf|W83tB-SXvQbi$AD7=Dg^0k39&ZT#uC92>>l1-i;k|8tfyH9MV~;-(c_G0If^0cy zf;PH05c}KS+ZG3iT})UOw%PtiVQg$}U5>=>(UHxzR%YBaywBOD-Fz7`%2Sblo|nQO z1^uCTc_&eA7?#7lRt!s#dsmka9!miGKWU7JV9qijw=@m8Lo>J;hdd#qsIxFEeLRYc zmte=K*I@N`MhZ^ShgIu@GLWD|bEXjnvj*9vsr;eeD`VlH2hr(i#jwl@=(3k!{nyJN zE5dxWh#zPSLV`PkI(1h7K~Y41SGN@zz9O{uG~<&4@8IQco`N7Lu(0Am_{cily>7Uz zK6*FU$AMoAz-#egO6_fOu&o_zYhp)Wx9E1I1}r|WT|fWxl00)UubX-8><>eo=u9q` zgPIBnx=KAdihqH&b}OnM&S$@=9(eFq#$7L+luafJ$}*~unVE57yZ^<1p&Da?9XPt& z%%k`NFPXE;;?(ITZa8d{%IEbd^V7;?r(}7VNdGmW16O!)#*h@8IB^`iQ%n?0 zM0k8&WTctV)YJfz(FiwxiCItW5Ht7Op=D^PYgilyC@wCLW-qx4rZf|+#|fy&nGE3& zda3!CJ$^8)kR4vR8<$K9p(JF0$L>N}r%{ZH(B16x@A&tx4HspJodI8;A!V6tWQiFU zX^Gv5me@*`IQSwh@pPgkHvb4qY)G`kW6Tn9EJ&T2g!^XQLUCVzK(no$VuccVl?ek_ zVtt||9%Po7>gW4X@C7|+wx7U?*)PHF>B41&m&4(+Vawh>K;{54$P$kyTH+CAi6Jq> zBhRW&AHc-&nY_h2(fQx7c{=gop|?0Ouc;!Icq|rBm>$Y2=N5k=s1!pID#w|9 z|LEc5);ACcqMKHrsHg}vUwnar!U7B$Gzf(YfZ5HgW-H%ZtFT;chAHe+i3yzb|4`I6%vu9rm0>Y?Kqfl5_h^FRdv^2NyEwE0?*bayD z=*Aac7?C7@0@cf9CQ}NBeC3r_!0tOcJ9r=|E-vQVAVY@^Lm?^I;piFMPY2`1RoW+B zensB?+5_-YSd%2SqZxu;uSaffE`O(D>=;}*^-3J6JM7;0;)}@uDwWD^HmBv?aMO*r zcJ3Tn9vNw=X(%Yji*d*|>}&SyVY@xnejHGdVXLiw-N$RfonL;zZ6GZz4d3sn;f_^O zQUU|53M;ONyPke_Kpb?Zrl->Sx8l-E2Xn>k+4Em0CM6CTGML**hoz05IZ*x3(GP%@ ztKCMHSU%_yZZK@iYy5;t96fp*2dFAxWljMV80i_A&}uc^{X|f!Rqf0dn#dAMO3VBC zLThV(Yoae4>jyx-&~OpHU?0dAPF;jA)D7ed^%vm_tpoW&(emYw?lzfBWpn1t;aX>Y z&y0xK0Lx0ul&Sb^dFrXBCe!<^{Q!u#aLJM-4|qJDNko)O1(iXo)iKF>&1SQcK!4b{ zapUt;)wT=-z@kJ-wBRyC_uADl6)`UuTuvXr0-7?fU#f zAr3%3f88_BJhR5q-Z?n4a1hzS$Y@lTlOwzwf$CZRF$^K${X4%|cYO@tQEHP{Hj@Lr z`erp%ERo}}NFI?B4(WgqgUgWjW*xR4-P!oZqaT*XEb$aM)57Vqs-QQTVC(9D+i8b? zr`Jx&)XwjnP8+OUZT;%#m?d^U{q)oK96M3ZBT-6fDwZs~9*!P6c744I)2B^?DJ2yf z-`ENT!z7OZ^oaflRkn7L}TYQCTT?mO>r2h!`ckzHPLtk+#( z{NPhr69KFw!iK8r{sl%;Dn%V-d;^YuLOYkAnKR2?tfNcS=R&74h7$pl_uqd%%1cZ6 zj<(ky;J(04HS6o^she9Di5WUQG{l`JOl6QDAu6H4%ZBtn=k zsYIopia1cJB?4B-iK}3|@x~j29o8OR)-nGVsF-FAcOL+ieWH>8RPbu8kaXVMM1Z{N z>gv@@gg2{qK%-?E*YJ{0kYi`9B8jHt9E8SygMj^5XClC95@GI$iU}Xxec#VfnxDm6 z{a9Pp~!~kLkb|ulyO&MKc;S8sgqa_jR0Gn7x_>Z-P;nh5-w08au810!SD7;; z5nwopu%M}_2{mso$DmAqav}qna)_)>CLrPU}|N5BDC6R-FQwoKmiXzfyn--TMHK&Mg2XJ=T8j@AYM$&_i+CmjSpM~kt!kR>1 zV4Rp{-*ktw!nOT^oauL;yIGs-ihE-eL>nE#IcOOH+d%OXt@QtAZBc-?|NM@uqt@np z7i)!XTX?I`?|`DCvkSK_xcR&QFTeWQz$YBB)7+h3?ZUZ#I2k>9G`4MjUrywN0|bK+ zfFHw2;@=V~irjymHpF{o0lUTcM4fRwR zNKH%U`wT-ajcm09*?Wn9?G_uqdgwRug00RJpj^H|$K@OJy0000(?IYPFyV-0K zLf%j0{SEROmmY&?z0Jh$1)+ilB~#BDO*YXS5(iM4>t&Dx!fx z9)<7-kAx(|qy1RFG@7~+*oPA_t!Xw%lduLAe?%nhK?)QD?_nq@QCn`lz@O>_S zHNNMiirnnjvEyNv%XR469gIff(zR>X9=t2S{{8!#=g*&?pOcfrzXt*VG&VNk@ZrOC z4u@k>Fc<_2H`2Cm-_HAKBLHTzIp6R1-Z32f;UkjWAeluAOLDYKw2h+?A;Us+M)m+pX!ExrOk=UMG`DF5&l5vZ3GM-x$kGLQ z5&=+kS96gz$T;d8dzm@NZPfBoLJ_(=~gMbnil z5CwtABGcRGjZ%m>Qji(TP-GR<2}zJdH`=~AU}UzcFdO5k>B1gq!c8@Q0DT!vg5l<+ zCgOiq&QEXL1iQWGo{UVUR6M_BQ{QT2v1VjsVEy_(-qCpL)@=~QUeBWdfq)+)Mh-_+ z)mhwZY(|1k2Sai);^N|{m^|FO;^IOS6&2vZg^L3L1TbdwXf!o7Q%Spcv)ZhE-rm;M zj!~mVp}M-JZw^@ebd}P7Th8BO4yMpYHOqrG58j+P3Ho25`C{`2RnapU?mWM*X{F(DCJN}!rNVN<*)dSPyDp?N{X z?Af!TdEfv;hYn4iIAH?Hrj+uz|K%O8;_~H-&}hN{`-TB)KjLGG5$e68xnXos19Hp7Kn`cB6M6|WHq01Q` zs>UB&;{5Oc+Iyb9sWK}oE1w|wQ%UmYwSxx_zR&V^&j8H0thjjZk{L5_puT=CNuej{ z@ji6~{Z5S;&nV5^J7elpBvR4Mbh(fQymC(=knyI#t*CR~M-j zU(;QaNXC1AD<5XYr3lbmqp4d3q^9DHrl#6XQJfc*ZnrZ&dE{k937k~?_Ro~Ij}|Lm zzd2V~_~7iC>C>m1B7$>30B-!j_OS6uGOkcg?_tKXxN+pg0|G>irP~|d79eCyZcCrv z(Kr?$n#2_=R&1*H@q{Nb3bPP!t6)359XHKxY&j5rsI|L;^Qx+%a$#BV&?3*CcofClJyKOw&-#EtoyFUQ2@s88KvWJ?xQzx?DS=P zUemFU=ESvkGmcIu4?Xmdl4gjjZS8ctHZ&`{Av<$uJ5e}+K!Wr;TQ~#klb*4Ai9RJ- zDtmB`QpqI1wNJ>p1^y^5n^gkJliyc7|R!004an1F=29tXZ>3NX+hCwMnHA zt!QK4+E))I7DXys`stz4f;{j@#ppufT?cLs`*fB+L4%~4+F#@V@Nut zkzS~)tE1;hd?b29V1kMethR3CkQ!run=aNNCB?`)U9`wp`@S0!a`Irbea5-c#{*6t z3c*S%4%5d&hi0H4FFP8)T@H|e13jJq#Lmv>lGhsojlz3}{t1i4f@hxH6zULD1%5W1 z^a3;8Cz1Py-a8Zph+HgKHf^)>kK9Px+qwa<-@`3~9j(lJa&&L8{pgYLNGnc%L26bq zf8-YfQF$kmHVi63t`&oF=-$=og<$ca=2%lG1mh3`G7C+}9GS*x91sFXjvInesb$F1 z55v(Le}z@-2nkNghgIi*#ut-BW4azm#&~2DrtpWNN5P{dYtSLIVpRG>BxJ0_YhSL1 zq7L%WB7YE{q>3qL&`#augI|(=(Ai~0nl}$^J$#E?>&elRo+M1mnsu=7AsvdjWLD_}w0RV3pJ9#Pbyt zjyC}|n~mILGqwjeZr}nlHh%{uHpVjX3S%O!_-UsmW7UG?H1Dg>Y`ac>bA<+pS_1}P;`LZ2 z)-figi2Pg%UcZ3m?gnfu-;Hje6XSF5huv$#{zxw|EU5|A9^Dzy}xJ=Ehu8g_w9HGNG_M6qi=yA5&@7BVv@~w|5-TTMF}9&4(?E z=(dr0z+i7YwfIkeBXzn&65}SC10}vX3q;)o3l<=r)&UpM+(Wzi#>Sg`5w7`gUEeK_ z&*w)MjX+*r9;!~9Ku&HBN=iy_;o=3pW{w^+8ks{g(bLldtya@tWbtDVa~!9tEZp z6LNC0A|CPs`>Hc%*lCZop9dr|Y}M80xh6dM*$K{n0h7suZ_ZTl!YU{zKoX4#6IbM2 zPd^@z2VE(tDK!4AxbME21C)}#fy2?S=}=dF%Ga*iBXxv zXUpczn`cu0)_wqFoV#-6%GH7(%pjvo5>!b#T>^{DX*3!g1bS}Qu3g(m)wT=-z-Ec$ xu;5AxpS8zDQpEBc{+Ejo{+A11p8qSr09z*#(z&{TbG85g002ovPDHLkV1nN=bISk# diff --git a/skins/default/images/icons/groupactions.png b/skins/default/images/icons/groupactions.png new file mode 100644 index 0000000000000000000000000000000000000000..2e9c5f101f6985097789ca897d1be37fcee9062a GIT binary patch literal 660 zcmV;F0&D$=P)& zbZ{*cx=LCV9HNv`2RDZ&q^nS{lhlH1us8&op~3&41bu@`OQ3YIQ0UOLLxVyDQ$(l` zd;5Ln3HD)0<;El=_rS+_=ezg5_q}t^fp?#fQi{vOa5#JelTL9(ObgLM7Aho>NJv6w z(x8^905?olc8dtv2>F@*L+p+tSa#GC+kq2{u)0 z=bR4_2E0KrldXxzr=@9*#L@bB*L@RgI3(9h4An3$@m zs->f&%*@Qq&CS@@*z@!A>Fev)*Vgg!^6KmB=<4dv&(G1((DL%~)6&z@)6(AE-QM2a z=<4ay($e$v^Uco9-rnBM&d<-!&h75*^Yirb^YYl&*wWI|*4Eb6*4NI?&fVVK=;`Xv z(b3V+(9zJ*l+uPf{y}Y)zx992Uo}Qh`$;#m1;MLXD^z`)E+1cmk z=Z%bvA^8LV00000EC2ui01N;u000L6K%Q_&B$A58hw?%4EFYeWL|U3kO418p%PnxX z1VX_;E{Ms5k$N;(588u(d`_1`CXoF7KEZQXEEIwi929{ZH;Iaii5e@8D;pXcAQ~VW zJ(-%DoSm8uprN5A4ki&1DW@o?D60_yusgCl0<^UlCmAOl85kJ6x-TvkEiJ%?CLu2@bT|7^!4`iv-$c6{Qds? zIRXkC7;uKcg9r`QSje#9LK_exN`x4rqQ#396=>YZQR9M-AVYq{L9(Ptk_l3(T$%EK d%a<@+%9u&B=FA^BbL!m5v!~AlpE-j706Q(D8bbg8 literal 538 zcmZ?wbhEHblw;6fxXQpVbHc7cQK?e&fc43m0zOxN+|MxzC?Jzj^!S#fukb&z^nt>eah<@7}(7a}FqR?%dnA zZ=O7PGNHf!@Zm$ZZ{Oa#chA+USGR53{_NSadGqF;I(h2DhYt@PJXpGTac@u0moHz= zoH@fl8BqMm0yaPgM1uUp!1n*ZlmZVOss0m7icaQO&F@}O;?*0m{zgvb>9RV+uA!gg}ZxtdzB_koHTjL)Cnvz zX3m;DXYLH%1q&A~=3lyO`3jy@tJkdMTfbrBCW$Ruw@FCuloZ&rci(=FgNF_u;Sf1~ z;^axKGiT48zi{!=>-L?y_ijo*eDwIq(`OIWUcP$$=Iy(eavwi^{_^$P zM}?oie*gLV?MOMjcj>pDDM?2(=vhJMNxcGRFqEnZQ z=cXkmCol!AiaEJy>FFt~NvGy`ZeDhFhI-MfJ0~|UKW{!qxrs~GYs-p@3m7J8#h%)- w^70bKMP75gwywImLS|Fe-BVjvUtgnfs7uy++nSr3GcK=+J-u!1Z3PBv0Q0aP{{R30 diff --git a/skins/default/mail.css b/skins/default/mail.css index 7b233f861..a3bb4ab9b 100644 --- a/skins/default/mail.css +++ b/skins/default/mail.css @@ -686,7 +686,7 @@ body.messagelist #messagelist thead tr td.sortedASC, #messagelist thead tr td.sortedDESC { - background-position: 0 -20px; + background-position: 0 -22px; } #messagelist thead tr td.sortedASC a diff --git a/skins/default/templates/addressbook.html b/skins/default/templates/addressbook.html index fe9019208..f47931502 100644 --- a/skins/default/templates/addressbook.html +++ b/skins/default/templates/addressbook.html @@ -4,27 +4,16 @@ <roundcube:object name="pagetitle" /> - + - + @@ -45,12 +34,24 @@ - -
+
- +
+ + +
+
+ + +
+
+ +
+
    +
    • +
    • +
-
diff --git a/skins/default/templates/mail.html b/skins/default/templates/mail.html index a1084b096..9ade5641f 100644 --- a/skins/default/templates/mail.html +++ b/skins/default/templates/mail.html @@ -119,6 +119,7 @@ +
    @@ -132,8 +133,6 @@ -
-