Permalink
Browse files

Fixes interface lock-up issues and minor bugs (richs)

  • Loading branch information...
thomascube committed Aug 9, 2007
1 parent 6d969b4 commit 4b9efbb9f49911b17bde2d46b86df825e987101e
Showing with 37 additions and 17 deletions.
  1. +1 −1 .htaccess
  2. +12 −0 CHANGELOG
  3. +3 −0 INSTALL
  4. +19 −14 program/js/app.js
  5. +1 −1 skins/default/mail.css
  6. +1 −1 skins/default/templates/compose.html
View
@@ -2,7 +2,7 @@
php_flag display_errors Off
php_flag log_errors On
php_value error_log logs/errors
-php_value upload_max_filesize 2M
+php_value upload_max_filesize 5M
<FilesMatch "(\.inc|\~)$">
Order allow,deny
View
@@ -1,6 +1,18 @@
CHANGELOG RoundCube Webmail
---------------------------
+2007/07/09 (richs)
+----------
+- Fixed bug with buttons not dimming/enabling properly after switching folders
+- Fixed compose window becoming unresponsive after saving a draft (#1484487)
+- Re-enabled "Back" button in compose window now that bug #1484487 is fixed
+- Fixed unresponsive interface issue when downloading attachments (#1484496)
+- Lowered status message time from 5 to 3 seconds to improve responsiveness
+- Added note to INSTALL about .htaccess limiting upload_max_filesize
+- Raised .htaccess upload_max_filesize from 2M to 5M to differ from default php.ini
+- Increased "mailboxcontrols" mail.css width from 160 to 170px to fix non-english languages
+- Fixed empty-message sending with TinyMCE plain-text mode, or if it's not installed
+
2007/07/03 (thomasb)
----------
- Added Macedonian (Slavic FYROM) localization
View
@@ -103,6 +103,9 @@ CONFIGURATION
Change the files in config/* according your to environment and your needs.
Details about the config paramaters can be found in the config files.
+You can also modify the default .htaccess file. This is necessary to
+increase the allowed size of file attachments, for example:
+ php_value upload_max_filesize 2M
UPGRADING
=========
View
@@ -35,7 +35,7 @@ function rcube_webmail()
// webmail client settings
this.dblclick_time = 500;
- this.message_time = 5000;
+ this.message_time = 3000;
this.identifier_expr = new RegExp('[^0-9a-z\-_]', 'gi');
@@ -694,7 +694,7 @@ function rcube_webmail()
}
}
- this.goto_url('get', qstring+'&_download=1');
+ this.goto_url('get', qstring+'&_download=1', false);
break;
case 'select-all':
@@ -1063,7 +1063,9 @@ function rcube_webmail()
this.unfocus_folder(id);
this.command('moveto', id);
}
-
+
+ // Hide message command buttons until a message is selected
+ this.enable_command('reply', 'reply-all', 'forward', 'delete', 'print', false);
return false;
};
@@ -1088,15 +1090,16 @@ function rcube_webmail()
clearTimeout(this.preview_timer);
var selected = list.selection.length==1;
+
+ // Hide certain command buttons when Drafts folder is selected
if (this.env.mailbox == this.env.drafts_mailbox)
{
- this.enable_command('show', selected);
- this.enable_command('delete', 'moveto', list.selection.length>0 ? true : false);
+ this.enable_command('reply', 'reply-all', 'forward', false);
+ this.enable_command('show', 'delete', 'moveto', selected);
}
else
{
- this.enable_command('show', 'reply', 'reply-all', 'forward', 'print', selected);
- this.enable_command('delete', 'moveto', list.selection.length>0 ? true : false);
+ this.enable_command('show', 'reply', 'reply-all', 'forward', 'print', 'delete', 'moveto', selected);
}
// start timer for message preview (wait for double click)
@@ -1680,7 +1683,7 @@ function rcube_webmail()
}
// check for empty body
- if ((input_message.value=='')&&(tinyMCE.getContent()==''))
+ if ((input_message.value=='')&&(tinyMCE == null ? true : (tinyMCE.getContent()=='' || tinyMCE.getContent() == null)))
{
if (!confirm(this.get_label('nobodywarning')))
{
@@ -1711,6 +1714,9 @@ function rcube_webmail()
{
if (this.env.draft_autosave)
this.save_timer = self.setTimeout(function(){ ref.command("savedraft"); }, this.env.draft_autosave * 1000);
+
+ // Unlock interface now that saving is complete
+ this.busy = false;
};
@@ -3187,9 +3193,11 @@ function rcube_webmail()
/********* remote request methods *********/
/********************************************************/
- this.redirect = function(url)
+ this.redirect = function(url, lock)
{
- this.set_busy(true);
+ if (lock || lock == NULL)
+ this.set_busy(true);
+
if (this.env.framed && window.parent)
parent.location.href = url;
else
@@ -3198,11 +3206,8 @@ function rcube_webmail()
this.goto_url = function(action, query, lock)
{
- if (lock)
- this.set_busy(true);
-
var querystring = query ? '&'+query : '';
- this.redirect(this.env.comm_path+'&_action='+action+querystring);
+ this.redirect(this.env.comm_path+'&_action='+action+querystring, lock);
};
View
@@ -338,7 +338,7 @@ html>body*#messagecontframe
{
position: absolute;
left: 20px;
- width: 160px;
+ width: 170px;
bottom: 20px;
height: 16px;
overflow: hidden;
@@ -31,7 +31,7 @@
<form name="form" action="./" method="post">
<div id="messagetoolbar">
-<!--<roundcube:button command="list" image="/images/buttons/back_act.png" imageSel="/images/buttons/back_sel.png" imageAct="/images/buttons/back_act.png" width="32" height="32" title="backtolist" />-->
+<roundcube:button command="list" image="/images/buttons/back_act.png" imageSel="/images/buttons/back_sel.png" imageAct="/images/buttons/back_act.png" width="32" height="32" title="backtolist" />
<roundcube:button command="send" imageSel="/images/buttons/send_sel.png" imageAct="/images/buttons/send_act.png" imagePas="/images/buttons/send_pas.png" width="32" height="32" title="sendmessage" />
<roundcube:button command="spellcheck" imageSel="/images/buttons/spellcheck_sel.png" imageAct="/images/buttons/spellcheck_act.png" imagePas="/images/buttons/spellcheck_pas.png" width="32" height="32" title="checkspelling" />
<roundcube:button command="add-attachment" imageSel="/images/buttons/attach_sel.png" imageAct="/images/buttons/attach_act.png" imagePas="/images/buttons/attach_pas.png" width="32" height="32" title="addattachment" />

0 comments on commit 4b9efbb

Please sign in to comment.