Improvements/fixes to contacts popup

diff --git a/imp/js/contacts.js b/imp/js/contacts.js
index 6170dfb..f812012 100644 (file)
--- a/imp/js/contacts.js
+++ b/imp/js/contacts.js
@@ -62,7 +62,7 @@ var ImpContacts = {
 
     updateMessage: function()
     {
-        if (parent.opener.closed) {
+        if (!parent.opener) {
             alert(IMP.text.contacts_closed);
             window.close();
             return;
@@ -122,6 +122,9 @@ var ImpContacts = {
 
     onDomLoad: function()
     {
+        if ($('search').present()) {
+            $('btn_clear').show();
+        }
         $('contacts').observe('submit', this._passAddresses.bind(this));
         document.observe('change', this._changeHandler.bindAsEventListener(this));
         document.observe('click', this._clickHandler.bindAsEventListener(this));
@@ -151,6 +154,10 @@ var ImpContacts = {
             id = elt.readAttribute('id');
 
             switch (id) {
+            case 'btn_clear':
+                $('search').setValue('');
+                break;
+
             case 'btn_add_to':
             case 'btn_add_cc':
             case 'btn_add_bcc':

diff --git a/imp/templates/contacts/contacts.html b/imp/templates/contacts/contacts.html
index 15384d2..1e74a9e 100644 (file)
--- a/imp/templates/contacts/contacts.html
+++ b/imp/templates/contacts/contacts.html
@@ -23,6 +23,7 @@
   <input name="source" type="hidden" value="<tag:source_list />" />
 </else:multiple_source></if:multiple_source>
   <input type="submit" class="button" value="<gettext>Search</gettext>" />
+  <input id="btn_clear" type="submit" class="button" style="display:none" value="<gettext>Reset</gettext>" />
  </p>
 
  <table width="100%" cellspacing="2">

diff --git a/imp/templates/javascript_defs.php b/imp/templates/javascript_defs.php
index bc3c434..2dd3f64 100644 (file)
--- a/imp/templates/javascript_defs.php
+++ b/imp/templates/javascript_defs.php
@@ -39,7 +39,7 @@ $code = array(
 
         /* Strings used in contacts.js */
         'contacts_select' => _("You must select an address first."),
-        'contacts_closed' => _("The message being composed has been closed. Exiting."),
+        'contacts_closed' => _("The message being composed has been closed."),
         'contacts_called' => _("This window must be called from a compose window."),
 
         /* Strings used in folders.js */