From a4865adf44c318efb8f894ca36e5e3dd9330f49e Mon Sep 17 00:00:00 2001 From: alecpl Date: Sun, 7 Mar 2010 17:42:09 +0000 Subject: - code cleanup --- skins/default/functions.js | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'skins/default/functions.js') diff --git a/skins/default/functions.js b/skins/default/functions.js index 3a295decb..47a121db1 100644 --- a/skins/default/functions.js +++ b/skins/default/functions.js @@ -13,7 +13,7 @@ function rcube_init_settings_tabs() tab = '#settingstab' + (rcmail.env.action=='preferences' ? 'default' : (rcmail.env.action.indexOf('identity')>0 ? 'identities' : rcmail.env.action.replace(/\./g, ''))); $(tab).addClass('tablink-selected'); - $(tab + '> a').removeAttr('onclick').unbind('click').bind('click', function(){return false}); + $(tab + '> a').removeAttr('onclick').unbind('click').bind('click', function(){return false;}); } function rcube_show_advanced(visible) @@ -39,12 +39,12 @@ function rcmail_show_header_form(id) link.style.display = 'none'; - if (row = document.getElementById('compose-' + id)) + if ((row = document.getElementById('compose-' + id))) { var div = document.getElementById('compose-div'); var headers_div = document.getElementById('compose-headers-div'); row.style.display = (document.all && !window.opera) ? 'block' : 'table-row'; - div.style.top = (parseInt(headers_div.offsetHeight)) + 'px'; + div.style.top = parseInt(headers_div.offsetHeight, 10) + 'px'; } return false; @@ -52,7 +52,7 @@ function rcmail_show_header_form(id) function rcmail_hide_header_form(id) { - var row, parent, ns, ps, link, links; + var row, parent, ns, link, links; link = document.getElementById(id + '-link'); link.style.display = ''; @@ -71,12 +71,12 @@ function rcmail_hide_header_form(id) document.getElementById('_' + id).value = ''; - if (row = document.getElementById('compose-' + id)) + if ((row = document.getElementById('compose-' + id))) { var div = document.getElementById('compose-div'); var headers_div = document.getElementById('compose-headers-div'); row.style.display = 'none'; - div.style.top = (parseInt(headers_div.offsetHeight)) + 'px'; + div.style.top = parseInt(headers_div.offsetHeight, 10) + 'px'; } return false; @@ -175,7 +175,7 @@ show_searchmenu: function(show) set_searchmod: function(elem) { if (!rcmail.env.search_mods) - rcmail.env.search_mods = new Object(); + rcmail.env.search_mods = {}; if (!rcmail.env.search_mods[rcmail.env.mailbox]) rcmail.env.search_mods[rcmail.env.mailbox] = rcube_clone_object(rcmail.env.search_mods['*']); -- cgit v1.2.3