- fixed identation + some code improvements

1 files changed, 292 insertions, 290 deletions

diff --git a/program/js/googiespell.js b/program/js/googiespell.js
index c7be29e57..51acb732f 100644
--- a/program/js/googiespell.js
+++ b/program/js/googiespell.js
@@ -7,29 +7,29 @@
          GPL
      AUTHORS
          4mir Salihefendic (http://amix.dk) - amix@amix.dk
-	 Aleksander Machniak - alec [at] alec.pl
+	    Aleksander Machniak - alec [at] alec.pl
 */
 
 var SPELL_CUR_LANG = null;
 var GOOGIE_DEFAULT_LANG = 'en';
 
 function GoogieSpell(img_dir, server_url) {
-    var ref = this;
+    var ref = this,
+        cookie_value = getCookie('language');
+
+    GOOGIE_CUR_LANG = cookie_value != null ? cookie_value : GOOGIE_DEFAULT_LANG;
 
     this.array_keys = function(arr) {
-	var res = [];
-	for (var key in arr) { res.push([key]); }
-	return res;
+	    var res = [];
+	    for (var key in arr) { res.push([key]); }
+	    return res;
     }
-    
-    var cookie_value = getCookie('language');
-    GOOGIE_CUR_LANG = cookie_value != null ? cookie_value : GOOGIE_DEFAULT_LANG;
 
     this.img_dir = img_dir;
     this.server_url = server_url;
 
     this.org_lang_to_word = {
-	"da": "Dansk", "de": "Deutsch", "en": "English",
+	    "da": "Dansk", "de": "Deutsch", "en": "English",
         "es": "Español", "fr": "Français", "it": "Italiano", 
         "nl": "Nederlands", "pl": "Polski", "pt": "Português",
         "fi": "Suomi", "sv": "Svenska"
@@ -49,42 +49,42 @@ function GoogieSpell(img_dir, server_url) {
     this.lang_rsm_edt = "Resume editing";
     this.lang_no_error_found = "No spelling errors found";
     this.lang_no_suggestions = "No suggestions";
-    
+
     this.show_spell_img = false; // roundcube mod.
     this.decoration = true;
     this.use_close_btn = true;
     this.edit_layer_dbl_click = true;
     this.report_ta_not_found = true;
 
-    //Extensions
+    // Extensions
     this.custom_ajax_error = null;
     this.custom_no_spelling_error = null;
-    this.custom_menu_builder = []; //Should take an eval function and a build menu function
-    this.custom_item_evaulator = null; //Should take an eval function and a build menu function
+    this.custom_menu_builder = []; // Should take an eval function and a build menu function
+    this.custom_item_evaulator = null; // Should take an eval function and a build menu function
     this.extra_menu_items = [];
     this.custom_spellcheck_starter = null;
     this.main_controller = true;
 
-    //Observers
+    // Observers
     this.lang_state_observer = null;
     this.spelling_state_observer = null;
     this.show_menu_observer = null;
     this.all_errors_fixed_observer = null;
 
-    //Focus links - used to give the text box focus
+    // Focus links - used to give the text box focus
     this.use_focus = false;
     this.focus_link_t = null;
     this.focus_link_b = null;
 
-    //Counters
+    // Counters
     this.cnt_errors = 0;
     this.cnt_errors_fixed = 0;
-    
-    //Set document's onclick to hide the language and error menu
+
+    // Set document's onclick to hide the language and error menu
     $(document).bind('click', function(e) {
         if($(e.target).attr('googie_action_btn') != '1' && ref.isLangWindowShown())
-	    ref.hideLangWindow();
-	if($(e.target).attr('googie_action_btn') != '1' && ref.isErrorWindowShown())
+	        ref.hideLangWindow();
+	    if($(e.target).attr('googie_action_btn') != '1' && ref.isErrorWindowShown())
             ref.hideErrorWindow();
     });
 
@@ -94,13 +94,12 @@ this.decorateTextarea = function(id) {
 
     if (this.text_area) {
         if (!this.spell_container && this.decoration) {
-            var table = document.createElement('table');
-            var tbody = document.createElement('tbody');
-            var tr = document.createElement('tr');
-            var spell_container = document.createElement('td');
-
-            var r_width = this.isDefined(this.force_width) ? this.force_width : this.text_area.offsetWidth;
-            var r_height = this.isDefined(this.force_height) ? this.force_height : 16;
+            var table = document.createElement('table'),
+                tbody = document.createElement('tbody'),
+                tr = document.createElement('tr'),
+                spell_container = document.createElement('td'),
+                r_width = this.isDefined(this.force_width) ? this.force_width : this.text_area.offsetWidth,
+                r_height = this.isDefined(this.force_height) ? this.force_height : 16;
 
             tr.appendChild(spell_container);
             tbody.appendChild(tr);
@@ -115,19 +114,19 @@ this.decorateTextarea = function(id) {
     else 
         if (this.report_ta_not_found)
             alert('Text area not found');
-}
+};
 
 //////
 // API Functions (the ones that you can call)
 /////
 this.setSpellContainer = function(id) {
     this.spell_container = typeof(id) == 'string' ? document.getElementById(id) : id;
-}
+};
 
 this.setLanguages = function(lang_dict) {
     this.lang_to_word = lang_dict;
     this.langlist_codes = this.array_keys(lang_dict);
-}
+};
 
 this.setCurrentLanguage = function(lan_code) {
     GOOGIE_CUR_LANG = lan_code;
@@ -136,29 +135,29 @@ this.setCurrentLanguage = function(lan_code) {
     var now = new Date();
     now.setTime(now.getTime() + 365 * 24 * 60 * 60 * 1000);
     setCookie('language', lan_code, now);
-}
+};
 
 this.setForceWidthHeight = function(width, height) {
     // Set to null if you want to use one of them
     this.force_width = width;
     this.force_height = height;
-}
+};
 
 this.setDecoration = function(bool) {
     this.decoration = bool;
-}
+};
 
 this.dontUseCloseButtons = function() {
     this.use_close_btn = false;
-}
+};
 
 this.appendNewMenuItem = function(name, call_back_fn, checker) {
     this.extra_menu_items.push([name, call_back_fn, checker]);
-}
+};
 
 this.appendCustomMenuBuilder = function(eval, builder) {
     this.custom_menu_builder.push([eval, builder]);
-}
+};
 
 this.setFocus = function() {
     try {
@@ -169,7 +168,7 @@ this.setFocus = function() {
     catch(e) {
         return false;
     }
-}
+};
 
 
 //////
@@ -179,11 +178,11 @@ this.setStateChanged = function(current_state) {
     this.state = current_state;
     if (this.spelling_state_observer != null && this.report_state_change)
         this.spelling_state_observer(current_state, this);
-}
+};
 
 this.setReportStateChange = function(bool) {
     this.report_state_change = bool;
-}
+};
 
 
 //////
@@ -191,17 +190,17 @@ this.setReportStateChange = function(bool) {
 /////
 this.getUrl = function() {
     return this.server_url + GOOGIE_CUR_LANG;
-}
+};
 
 this.escapeSpecial = function(val) {
     return val.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
-}
+};
 
 this.createXMLReq = function (text) {
     return '<?xml version="1.0" encoding="utf-8" ?>'
 	+ '<spellrequest textalreadyclipped="0" ignoredups="0" ignoredigits="1" ignoreallcaps="1">'
 	+ '<text>' + text + '</text></spellrequest>';
-}
+};
 
 this.spellCheck = function(ignore) {
     this.cnt_errors_fixed = 0;
@@ -229,7 +228,7 @@ this.spellCheck = function(ignore) {
     this.createErrorWindow();
     $('body').append(this.error_window);
 
-    try { netscape.security.PrivilegeManager.enablePrivilege("UniversalBrowserRead"); } 
+    try { netscape.security.PrivilegeManager.enablePrivilege("UniversalBrowserRead"); }
     catch (e) { }
 
     if (this.main_controller)
@@ -241,34 +240,34 @@ this.spellCheck = function(ignore) {
 
     $.ajax({ type: 'POST', url: this.getUrl(),
 	data: this.createXMLReq(req_text), dataType: 'text',
-	error: function(o) {
-    	    if (ref.custom_ajax_error)
-        	ref.custom_ajax_error(ref);
-    	    else
-        	alert('An error was encountered on the server. Please try again later.');
-    	    if (ref.main_controller) {
-        	$(ref.spell_span).remove();
-        	ref.removeIndicator();
-    	    }
-    	    ref.checkSpellingState();
-	},
+	    error: function(o) {
+            if (ref.custom_ajax_error)
+        	    ref.custom_ajax_error(ref);
+            else
+        	    alert('An error was encountered on the server. Please try again later.');
+            if (ref.main_controller) {
+        	    $(ref.spell_span).remove();
+        	    ref.removeIndicator();
+            }
+            ref.checkSpellingState();
+	    },
         success: function(data) {
-	    var r_text = data;
+	        var r_text = data;
     	    ref.results = ref.parseResult(r_text);
     	    if (r_text.match(/<c.*>/) != null) {
-        	//Before parsing be sure that errors were found
-        	ref.showErrorsInIframe();
-        	ref.resumeEditingState();
+        	    // Before parsing be sure that errors were found
+        	    ref.showErrorsInIframe();
+        	    ref.resumeEditingState();
     	    } else {
-        	if (!ref.custom_no_spelling_error)
-		    ref.flashNoSpellingErrorState();
+        	    if (!ref.custom_no_spelling_error)
+		        ref.flashNoSpellingErrorState();
         	else
-            	    ref.custom_no_spelling_error(ref);
+            	ref.custom_no_spelling_error(ref);
     	    }
     	    ref.removeIndicator();
-	}
+	    }
     });
-}
+};
 
 
 //////
@@ -276,32 +275,32 @@ this.spellCheck = function(ignore) {
 /////
 this.parseResult = function(r_text) {
     // Returns an array: result[item] -> ['attrs'], ['suggestions']
-    var re_split_attr_c = /\w+="(\d+|true)"/g;
-    var re_split_text = /\t/g;
-
-    var matched_c = r_text.match(/<c[^>]*>[^<]*<\/c>/g);
-    var results = new Array();
+    var re_split_attr_c = /\w+="(\d+|true)"/g,
+        re_split_text = /\t/g,
+        matched_c = r_text.match(/<c[^>]*>[^<]*<\/c>/g),
+        results = [];
 
     if (matched_c == null)
         return results;
-    
-    for (var i=0; i < matched_c.length; i++) {
-        var item = new Array();
+
+    for (var i=0, len=matched_c.length; i < len; i++) {
+        var item = [];
         this.errorFound();
 
-        //Get attributes
-        item['attrs'] = new Array();
-        var split_c = matched_c[i].match(re_split_attr_c);
+        // Get attributes
+        item['attrs'] = [];
+        var c_attr, val,
+            split_c = matched_c[i].match(re_split_attr_c);
         for (var j=0; j < split_c.length; j++) {
-            var c_attr = split_c[j].split(/=/);
-            var val = c_attr[1].replace(/"/g, '');
+            c_attr = split_c[j].split(/=/);
+            val = c_attr[1].replace(/"/g, '');
             item['attrs'][c_attr[0]] = val != 'true' ? parseInt(val) : val;
         }
 
-        //Get suggestions
-        item['suggestions'] = new Array();
-        var only_text = matched_c[i].replace(/<[^>]*>/g, '');
-        var split_t = only_text.split(re_split_text);
+        // Get suggestions
+        item['suggestions'] = [];
+        var only_text = matched_c[i].replace(/<[^>]*>/g, ''),
+            split_t = only_text.split(re_split_text);
         for (var k=0; k < split_t.length; k++) {
     	    if(split_t[k] != '')
         	item['suggestions'].push(split_t[k]);
@@ -310,7 +309,7 @@ this.parseResult = function(r_text) {
     }
 
     return results;
-}
+};
 
 
 //////
@@ -319,50 +318,51 @@ this.parseResult = function(r_text) {
 this.createErrorWindow = function() {
     this.error_window = document.createElement('div');
     $(this.error_window).addClass('googie_window').attr('googie_action_btn', '1');
-}
+};
 
 this.isErrorWindowShown = function() {
     return $(this.error_window).is(':visible');
-}
+};
 
 this.hideErrorWindow = function() {
     $(this.error_window).css('visibility', 'hidden');
     $(this.error_window_iframe).css('visibility', 'hidden');
-}
+};
 
 this.updateOrginalText = function(offset, old_value, new_value, id) {
-    var part_1 = this.orginal_text.substring(0, offset);
-    var part_2 = this.orginal_text.substring(offset+old_value.length);
+    var part_1 = this.orginal_text.substring(0, offset),
+        part_2 = this.orginal_text.substring(offset+old_value.length),
+        add_2_offset = new_value.length - old_value.length;
+
     this.orginal_text = part_1 + new_value + part_2;
     $(this.text_area).val(this.orginal_text);
-    var add_2_offset = new_value.length - old_value.length;
-    for (var j=0; j < this.results.length; j++) {
-        //Don't edit the offset of the current item
+    for (var j=0, len=this.results.length; j<len; j++) {
+        // Don't edit the offset of the current item
         if (j != id && j > id)
             this.results[j]['attrs']['o'] += add_2_offset;
     }
-}
+};
 
 this.saveOldValue = function(elm, old_value) {
     elm.is_changed = true;
     elm.old_value = old_value;
-}
+};
 
 this.createListSeparator = function() {
-    var td = document.createElement('td');
-    var tr = document.createElement('tr');
+    var td = document.createElement('td'),
+        tr = document.createElement('tr');
 
     $(td).html(' ').attr('googie_action_btn', '1')
 	.css({'cursor': 'default', 'font-size': '3px', 'border-top': '1px solid #ccc', 'padding-top': '3px'});
     tr.appendChild(td);
 
     return tr;
-}
+};
 
 this.correctError = function(id, elm, l_elm, rm_pre_space) {
-    var old_value = elm.innerHTML;
-    var new_value = l_elm.nodeType == 3 ? l_elm.nodeValue : l_elm.innerHTML;
-    var offset = this.results[id]['attrs']['o'];
+    var old_value = elm.innerHTML,
+        new_value = l_elm.nodeType == 3 ? l_elm.nodeValue : l_elm.innerHTML,
+        offset = this.results[id]['attrs']['o'];
 
     if (rm_pre_space) {
         var pre_length = elm.previousSibling.innerHTML;
@@ -380,27 +380,27 @@ this.correctError = function(id, elm, l_elm, rm_pre_space) {
 
     if (!this.isDefined(elm.old_value))
         this.saveOldValue(elm, old_value);
-    
+
     this.errorFixed();
-}
+};
 
 this.showErrorWindow = function(elm, id) {
     if (this.show_menu_observer)
         this.show_menu_observer(this);
 
-    var ref = this;
-    var pos = $(elm).offset();
+    var ref = this,
+        pos = $(elm).offset(),
+        table = document.createElement('table'),
+        list = document.createElement('tbody');
+
     pos.top -= this.edit_layer.scrollTop;
 
     $(this.error_window).css({'visibility': 'visible',
 	'top': (pos.top+20)+'px', 'left': (pos.left)+'px'}).html('');
 
-    var table = document.createElement('table');
-    var list = document.createElement('tbody');
-
     $(table).addClass('googie_list').attr('googie_action_btn', '1');
 
-    //Check if we should use custom menu builder, if not we use the default
+    // Check if we should use custom menu builder, if not we use the default
     var changed = false;
     for (var k=0; k<this.custom_menu_builder.length; k++) {
         var eb = this.custom_menu_builder[k];
@@ -410,15 +410,16 @@ this.showErrorWindow = function(elm, id) {
         }
     }
     if (!changed) {
-        //Build up the result list
-        var suggestions = this.results[id]['suggestions'];
-        var offset = this.results[id]['attrs']['o'];
-        var len = this.results[id]['attrs']['l'];
+        // Build up the result list
+        var suggestions = this.results[id]['suggestions'],
+            offset = this.results[id]['attrs']['o'],
+            len = this.results[id]['attrs']['l'],
+            row, item, dummy;
 
         if (suggestions.length == 0) {
-            var row = document.createElement('tr');
-            var item = document.createElement('td');
-            var dummy = document.createElement('span');
+            row = document.createElement('tr'),
+            item = document.createElement('td'),
+            dummy = document.createElement('span');
 
             $(dummy).text(this.lang_no_suggestions);
             $(item).attr('googie_action_btn', '1').css('cursor', 'default');
@@ -428,16 +429,16 @@ this.showErrorWindow = function(elm, id) {
             list.appendChild(row);
         }
 
-        for (i=0; i < suggestions.length; i++) {
-            var row = document.createElement('tr');
-            var item = document.createElement('td');
-            var dummy = document.createElement('span');
+        for (var i=0, len=suggestions.length; i < len; i++) {
+            row = document.createElement('tr'),
+            item = document.createElement('td'),
+            dummy = document.createElement('span');
 
             $(dummy).html(suggestions[i]);
-            
+
             $(item).bind('mouseover', this.item_onmouseover)
-        	.bind('mouseout', this.item_onmouseout)
-        	.bind('click', function(e) { ref.correctError(id, elm, e.target.firstChild) });
+        	    .bind('mouseout', this.item_onmouseout)
+        	    .bind('click', function(e) { ref.correctError(id, elm, e.target.firstChild) });
 
             item.appendChild(dummy);
             row.appendChild(item);
@@ -446,32 +447,32 @@ this.showErrorWindow = function(elm, id) {
 
         //The element is changed, append the revert
         if (elm.is_changed && elm.innerHTML != elm.old_value) {
-            var old_value = elm.old_value;
-            var revert_row = document.createElement('tr');
-            var revert = document.createElement('td');
-            var rev_span = document.createElement('span');
-	    
-	    $(rev_span).addClass('googie_list_revert').html(this.lang_revert + ' ' + old_value);
+            var old_value = elm.old_value,
+                revert_row = document.createElement('tr'),
+                revert = document.createElement('td'),
+                rev_span = document.createElement('span');
+
+	        $(rev_span).addClass('googie_list_revert').html(this.lang_revert + ' ' + old_value);
 
             $(revert).bind('mouseover', this.item_onmouseover)
-        	.bind('mouseout', this.item_onmouseout)
-        	.bind('click', function(e) {
+        	    .bind('mouseout', this.item_onmouseout)
+        	    .bind('click', function(e) {
             	    ref.updateOrginalText(offset, elm.innerHTML, old_value, id);
             	    $(elm).attr('is_corrected', true).css('color', '#b91414').html(old_value);
             	    ref.hideErrorWindow();
-        	});
+        	    });
 
             revert.appendChild(rev_span);
             revert_row.appendChild(revert);
             list.appendChild(revert_row);
         }
 
-        //Append the edit box
-        var edit_row = document.createElement('tr');
-        var edit = document.createElement('td');
-        var edit_input = document.createElement('input');
-        var ok_pic = document.createElement('img');
-	var edit_form = document.createElement('form');
+        // Append the edit box
+        var edit_row = document.createElement('tr'),
+            edit = document.createElement('td'),
+            edit_input = document.createElement('input'),
+            ok_pic = document.createElement('img'),
+	        edit_form = document.createElement('form');
 
         var onsub = function () {
             if (edit_input.value != '') {
@@ -479,55 +480,55 @@ this.showErrorWindow = function(elm, id) {
                     ref.saveOldValue(elm, elm.innerHTML);
 
                 ref.updateOrginalText(offset, elm.innerHTML, edit_input.value, id);
-		$(elm).attr('is_corrected', true).css('color', 'green').html(edit_input.value);
+		        $(elm).attr('is_corrected', true).css('color', 'green').html(edit_input.value);
                 ref.hideErrorWindow();
             }
             return false;
         };
 
-	$(edit_input).width(120).css({'margin': 0, 'padding': 0});
-	$(edit_input).val(elm.innerHTML).attr('googie_action_btn', '1');
-	$(edit).css('cursor', 'default').attr('googie_action_btn', '1');
+	    $(edit_input).width(120).css({'margin': 0, 'padding': 0});
+	    $(edit_input).val(elm.innerHTML).attr('googie_action_btn', '1');
+	    $(edit).css('cursor', 'default').attr('googie_action_btn', '1');
 
-	$(ok_pic).attr('src', this.img_dir + 'ok.gif')
-	    .width(32).height(16)
-	    .css({'cursor': 'pointer', 'margin-left': '2px', 'margin-right': '2px'})
-	    .bind('click', onsub);
+	    $(ok_pic).attr('src', this.img_dir + 'ok.gif')
+	        .width(32).height(16)
+    	    .css({'cursor': 'pointer', 'margin-left': '2px', 'margin-right': '2px'})
+	        .bind('click', onsub);
 
         $(edit_form).attr('googie_action_btn', '1')
-	    .css({'margin': 0, 'padding': 0, 'cursor': 'default', 'white-space': 'nowrap'})
-	    .bind('submit', onsub);
-        
-	edit_form.appendChild(edit_input);
-	edit_form.appendChild(ok_pic);
+	        .css({'margin': 0, 'padding': 0, 'cursor': 'default', 'white-space': 'nowrap'})
+	        .bind('submit', onsub);
+
+	    edit_form.appendChild(edit_input);
+	    edit_form.appendChild(ok_pic);
         edit.appendChild(edit_form);
         edit_row.appendChild(edit);
         list.appendChild(edit_row);
 
-        //Append extra menu items
+        // Append extra menu items
         if (this.extra_menu_items.length > 0)
-	    list.appendChild(this.createListSeparator());
-	
+	        list.appendChild(this.createListSeparator());
+
         var loop = function(i) {
-                if (i < ref.extra_menu_items.length) {
-                    var e_elm = ref.extra_menu_items[i];
-
-                    if (!e_elm[2] || e_elm[2](elm, ref)) {
-                        var e_row = document.createElement('tr');
-                        var e_col = document.createElement('td');
-
-			$(e_col).html(e_elm[0])
-                    	    .bind('mouseover', ref.item_onmouseover)
-                    	    .bind('mouseout', ref.item_onmouseout)
-			    .bind('click', function() { return e_elm[1](elm, ref) });
-			
-			e_row.appendChild(e_col);
-                        list.appendChild(e_row);
-                    }
-                    loop(i+1);
+            if (i < ref.extra_menu_items.length) {
+                var e_elm = ref.extra_menu_items[i];
+
+                if (!e_elm[2] || e_elm[2](elm, ref)) {
+                    var e_row = document.createElement('tr'),
+                      e_col = document.createElement('td');
+
+			        $(e_col).html(e_elm[0])
+                        .bind('mouseover', ref.item_onmouseover)
+                    	.bind('mouseout', ref.item_onmouseout)
+			            .bind('click', function() { return e_elm[1](elm, ref) });
+
+			        e_row.appendChild(e_col);
+                    list.appendChild(e_row);
                 }
-        }
-	
+                loop(i+1);
+            }
+        };
+
         loop(0);
         loop = null;
 
@@ -540,19 +541,19 @@ this.showErrorWindow = function(elm, id) {
     table.appendChild(list);
     this.error_window.appendChild(table);
 
-    //Dummy for IE - dropdown bug fix
+    // Dummy for IE - dropdown bug fix
     if ($.browser.msie) {
-	if (!this.error_window_iframe) {
+	    if (!this.error_window_iframe) {
             var iframe = $('<iframe>').css({'position': 'absolute', 'z-index': -1});
-	    $('body').append(iframe);
+	        $('body').append(iframe);
     	    this.error_window_iframe = iframe;
         }
 
-	$(this.error_window_iframe).css({'visibility': 'visible',
-	    'top': this.error_window.offsetTop, 'left': this.error_window.offsetLeft,
+	    $(this.error_window_iframe).css({'visibility': 'visible',
+	        'top': this.error_window.offsetTop, 'left': this.error_window.offsetLeft,
     	    'width': this.error_window.offsetWidth, 'height': this.error_window.offsetHeight});
     }
-}
+};
 
 
 //////
@@ -582,7 +583,7 @@ this.createEditLayer = function(width, height) {
             return false;
         });
     }
-}
+};
 
 this.resumeEditing = function() {
     this.setStateChanged('ready');
@@ -608,22 +609,22 @@ this.resumeEditing = function() {
             this.text_area.scrollTop = this.el_scroll_top;
     }
     this.checkSpellingState(false);
-}
+};
 
 this.createErrorLink = function(text, id) {
-    var elm = document.createElement('span');
-    var ref = this;
-    var d = function (e) {
+    var elm = document.createElement('span'),
+        ref = this,
+        d = function (e) {
     	    ref.showErrorWindow(elm, id);
     	    d = null;
     	    return false;
-    };
-    
+        };
+
     $(elm).html(text).addClass('googie_link').bind('click', d)
 	.attr({'googie_action_btn' : '1', 'g_id' : id, 'is_corrected' : false});
 
     return elm;
-}
+};
 
 this.createPart = function(txt_part) {
     if (txt_part == " ")
@@ -634,36 +635,37 @@ this.createPart = function(txt_part) {
     txt_part = txt_part.replace(/    /g, " &nbsp;");
     txt_part = txt_part.replace(/^ /g, "&nbsp;");
     txt_part = txt_part.replace(/ $/g, "&nbsp;");
-    
+
     var span = document.createElement('span');
     $(span).html(txt_part);
     return span;
-}
+};
 
 this.showErrorsInIframe = function() {
-    var output = document.createElement('div')
-    var pointer = 0;
-    var results = this.results;
+    var output = document.createElement('div'),
+        pointer = 0,
+        results = this.results;
 
     if (results.length > 0) {
-        for (var i=0; i < results.length; i++) {
-            var offset = results[i]['attrs']['o'];
-            var len = results[i]['attrs']['l'];
-            var part_1_text = this.orginal_text.substring(pointer, offset);
-            var part_1 = this.createPart(part_1_text);
-    
+        for (var i=0, length=results.length; i < length; i++) {
+            var offset = results[i]['attrs']['o'],
+                len = results[i]['attrs']['l'],
+                part_1_text = this.orginal_text.substring(pointer, offset),
+                part_1 = this.createPart(part_1_text);
+
             output.appendChild(part_1);
             pointer += offset - pointer;
-            
-            //If the last child was an error, then insert some space
+
+            // If the last child was an error, then insert some space
             var err_link = this.createErrorLink(this.orginal_text.substr(offset, len), i);
             this.error_links.push(err_link);
             output.appendChild(err_link);
             pointer += len;
         }
-        //Insert the rest of the orginal text
-        var part_2_text = this.orginal_text.substr(pointer, this.orginal_text.length);
-        var part_2 = this.createPart(part_2_text);
+
+        // Insert the rest of the orginal text
+        var part_2_text = this.orginal_text.substr(pointer, this.orginal_text.length),
+            part_2 = this.createPart(part_2_text);
 
         output.appendChild(part_2);
     }
@@ -675,10 +677,10 @@ this.showErrorsInIframe = function() {
     var me = this;
     if (this.custom_item_evaulator)
         $.map(this.error_links, function(elm){me.custom_item_evaulator(me, elm)});
-    
+
     $(this.edit_layer).append(output);
 
-    //Hide text area and show edit layer
+    // Hide text area and show edit layer
     $(this.text_area).hide();
     $(this.edit_layer).insertBefore(this.text_area);
 
@@ -691,7 +693,7 @@ this.showErrorsInIframe = function() {
     }
 
 //    this.edit_layer.scrollTop = this.ta_scroll_top;
-}
+};
 
 
 //////
@@ -702,49 +704,50 @@ this.createLangWindow = function() {
     $(this.language_window).addClass('googie_window')
 	.width(100).attr('googie_action_btn', '1');
 
-    //Build up the result list
-    var table = document.createElement('table');
-    var list = document.createElement('tbody');
-    var ref = this;
+    // Build up the result list
+    var table = document.createElement('table'),
+        list = document.createElement('tbody'),
+        ref = this,
+        row, item, span;
 
     $(table).addClass('googie_list').width('100%');
-    this.lang_elms = new Array();
+    this.lang_elms = [];
 
     for (i=0; i < this.langlist_codes.length; i++) {
-        var row = document.createElement('tr');
-        var item = document.createElement('td');
-        var span = document.createElement('span');
-	
-	$(span).text(this.lang_to_word[this.langlist_codes[i]]);
+        row = document.createElement('tr');
+        item = document.createElement('td');
+        span = document.createElement('span');
+
+	    $(span).text(this.lang_to_word[this.langlist_codes[i]]);
         this.lang_elms.push(item);
 
         $(item).attr('googieId', this.langlist_codes[i])
     	    .bind('click', function(e) {
-        	ref.deHighlightCurSel();
-        	ref.setCurrentLanguage($(this).attr('googieId'));
+        	    ref.deHighlightCurSel();
+        	    ref.setCurrentLanguage($(this).attr('googieId'));
 
-        	if (ref.lang_state_observer != null) {
+        	    if (ref.lang_state_observer != null) {
             	    ref.lang_state_observer();
-        	}
+        	    }
 
-        	ref.highlightCurSel();
-        	ref.hideLangWindow();
+        	    ref.highlightCurSel();
+        	    ref.hideLangWindow();
     	    })
-    	    .bind('mouseover', function(e) { 
-        	if (this.className != "googie_list_selected")
+    	    .bind('mouseover', function(e) {
+        	    if (this.className != "googie_list_selected")
             	    this.className = "googie_list_onhover";
     	    })
-    	    .bind('mouseout', function(e) { 
-        	if (this.className != "googie_list_selected")
-            	    this.className = "googie_list_onout"; 
+    	    .bind('mouseout', function(e) {
+        	    if (this.className != "googie_list_selected")
+            	    this.className = "googie_list_onout";
     	    });
 
-	item.appendChild(span);
+	    item.appendChild(span);
         row.appendChild(item);
         list.appendChild(row);
     }
 
-    //Close button
+    // Close button
     if (this.use_close_btn) {
         list.appendChild(this.createCloseButton(function () { ref.hideLangWindow.apply(ref) }));
     }
@@ -753,20 +756,20 @@ this.createLangWindow = function() {
 
     table.appendChild(list);
     this.language_window.appendChild(table);
-}
+};
 
 this.isLangWindowShown = function() {
     return $(this.language_window).is(':hidden');
-}
+};
 
 this.hideLangWindow = function() {
     $(this.language_window).css('visibility', 'hidden');
     $(this.switch_lan_pic).removeClass().addClass('googie_lang_3d_on');
-}
+};
 
 this.deHighlightCurSel = function() {
     $(this.lang_cur_elm).removeClass().addClass('googie_list_onout');
-}
+};
 
 this.highlightCurSel = function() {
     if (GOOGIE_CUR_LANG == null)
@@ -780,7 +783,7 @@ this.highlightCurSel = function() {
             this.lang_elms[i].className = "googie_list_onout";
         }
     }
-}
+};
 
 this.showLangWindow = function(elm) {
     if (this.show_menu_observer)
@@ -789,39 +792,38 @@ this.showLangWindow = function(elm) {
     this.createLangWindow();
     $('body').append(this.language_window);
 
-    var pos = $(elm).offset();
-    var top = pos.top + $(elm).height();
-    var left = this.change_lang_pic_placement == 'right' ? 
-	pos.left - 100 + $(elm).width() : pos.left + $(elm).width();
+    var pos = $(elm).offset(),
+        top = pos.top + $(elm).height(),
+        left = this.change_lang_pic_placement == 'right' ?
+	        pos.left - 100 + $(elm).width() : pos.left + $(elm).width();
 
     $(this.language_window).css({'visibility': 'visible', 'top' : top+'px','left' : left+'px'});
 
     this.highlightCurSel();
-}
+};
 
 this.createChangeLangPic = function() {
     var img = $('<img>')
-	.attr({src: this.img_dir + 'change_lang.gif', 'alt': 'Change language', 'googie_action_btn': '1'});
-
-    var switch_lan = document.createElement('span');
-    var ref = this;
+	    .attr({src: this.img_dir + 'change_lang.gif', 'alt': 'Change language', 'googie_action_btn': '1'}),
+        switch_lan = document.createElement('span');
+        ref = this;
 
     $(switch_lan).addClass('googie_lang_3d_on')
-	.append(img)
-	.bind('click', function(e) {
+	    .append(img)
+	    .bind('click', function(e) {
     	    var elm = this.tagName.toLowerCase() == 'img' ? this.parentNode : this;
     	    if($(elm).hasClass('googie_lang_3d_click')) {
-        	elm.className = 'googie_lang_3d_on';
-        	ref.hideLangWindow();
+            	elm.className = 'googie_lang_3d_on';
+        	    ref.hideLangWindow();
     	    }
     	    else {
-        	elm.className = 'googie_lang_3d_click';
-        	ref.showLangWindow(elm);
+        	    elm.className = 'googie_lang_3d_click';
+        	    ref.showLangWindow(elm);
     	    }
-	});
+	    });
 
     return switch_lan;
-}
+};
 
 this.createSpellDiv = function() {
     var span = document.createElement('span');
@@ -829,10 +831,10 @@ this.createSpellDiv = function() {
     $(span).addClass('googie_check_spelling_link').text(this.lang_chck_spell);
 
     if (this.show_spell_img) {
-	$(span).append(' ').append($('<img>').attr('src', this.img_dir + 'spellc.gif'));
+	    $(span).append(' ').append($('<img>').attr('src', this.img_dir + 'spellc.gif'));
     }
     return span;
-}
+};
 
 
 //////
@@ -843,15 +845,15 @@ this.flashNoSpellingErrorState = function(on_finish) {
 
     var ref = this;
     if (this.main_controller) {
-	var no_spell_errors;
-	if (on_finish) {
+	    var no_spell_errors;
+	    if (on_finish) {
     	    var fn = function() {
-        	on_finish();
-        	ref.checkSpellingState();
+            	on_finish();
+        	    ref.checkSpellingState();
     	    };
     	    no_spell_errors = fn;
-	}
-	else
+	    }
+	    else
     	    no_spell_errors = function () { ref.checkSpellingState() };
 
         var rsm = $('<span>').text(this.lang_no_error_found);
@@ -862,7 +864,7 @@ this.flashNoSpellingErrorState = function(on_finish) {
 
         window.setTimeout(no_spell_errors, 1000);
     }
-}
+};
 
 this.resumeEditingState = function() {
     this.setStateChanged('resume_editing');
@@ -880,7 +882,7 @@ this.resumeEditingState = function() {
 
     try { this.edit_layer.scrollTop = this.ta_scroll_top; }
     catch (e) {};
-}
+};
 
 this.checkSpellingState = function(fire) {
     if (fire)
@@ -891,8 +893,8 @@ this.checkSpellingState = function(fire) {
     else
         this.switch_lan_pic = document.createElement('span');
 
-    var span_chck = this.createSpellDiv();
-    var ref = this;
+    var span_chck = this.createSpellDiv(),
+        ref = this;
 
     if (this.custom_spellcheck_starter)
         $(span_chck).bind('click', function(e) { ref.custom_spellcheck_starter() });
@@ -902,14 +904,14 @@ this.checkSpellingState = function(fire) {
 
     if (this.main_controller) {
         if (this.change_lang_pic_placement == 'left') {
-	    $(this.spell_container).empty().append(this.switch_lan_pic).append(' ').append(span_chck);
+	        $(this.spell_container).empty().append(this.switch_lan_pic).append(' ').append(span_chck);
         } else {
-	    $(this.spell_container).empty().append(span_chck).append(' ').append(this.switch_lan_pic);
-	}
+	        $(this.spell_container).empty().append(span_chck).append(' ').append(this.switch_lan_pic);
+	    }
     }
 
     this.spell_span = span_chck;
-}
+};
 
 
 //////
@@ -917,7 +919,7 @@ this.checkSpellingState = function(fire) {
 /////
 this.isDefined = function(o) {
     return (o != 'undefined' && o != null)
-}
+};
 
 this.errorFixed = function() { 
     this.cnt_errors_fixed++; 
@@ -926,79 +928,79 @@ this.errorFixed = function() {
             this.hideErrorWindow();
             this.all_errors_fixed_observer();
         }
-}
+};
 
 this.errorFound = function() {
     this.cnt_errors++;
-}
+};
 
 this.createCloseButton = function(c_fn) {
     return this.createButton(this.lang_close, 'googie_list_close', c_fn);
-}
+};
 
 this.createButton = function(name, css_class, c_fn) {
-    var btn_row = document.createElement('tr');
-    var btn = document.createElement('td');
-    var spn_btn;
+    var btn_row = document.createElement('tr'),
+        btn = document.createElement('td'),
+        spn_btn;
 
     if (css_class) {
         spn_btn = document.createElement('span');
-	$(spn_btn).addClass(css_class).html(name);
+	    $(spn_btn).addClass(css_class).html(name);
     } else {
         spn_btn = document.createTextNode(name);
     }
 
     $(btn).bind('click', c_fn)
-	.bind('mouseover', this.item_onmouseover)
-	.bind('mouseout', this.item_onmouseout);
+	    .bind('mouseover', this.item_onmouseover)
+	    .bind('mouseout', this.item_onmouseout);
 
     btn.appendChild(spn_btn);
     btn_row.appendChild(btn);
 
     return btn_row;
-}
+};
 
 this.removeIndicator = function(elm) {
     //$(this.indicator).remove();
     // roundcube mod.
     if (window.rcmail)
 	rcmail.set_busy(false);
-}
+};
 
 this.appendIndicator = function(elm) {
     // modified by roundcube
     if (window.rcmail)
-	rcmail.set_busy(true, 'checking');
-/*    
+	    rcmail.set_busy(true, 'checking');
+/*
     this.indicator = document.createElement('img');
     $(this.indicator).attr('src', this.img_dir + 'indicator.gif')
-	.css({'margin-right': '5px', 'text-decoration': 'none'}).width(16).height(16);
-    
+	    .css({'margin-right': '5px', 'text-decoration': 'none'}).width(16).height(16);
+
     if (elm)
-	$(this.indicator).insertBefore(elm);
+	    $(this.indicator).insertBefore(elm);
     else
-	$('body').append(this.indicator);
-*/				    
+    	$('body').append(this.indicator);
+*/
 }
 
 this.createFocusLink = function(name) {
     var link = document.createElement('a');
     $(link).attr({'href': 'javascript:;', 'name': name});
     return link;
-}
+};
 
 this.item_onmouseover = function(e) {
-    if (this.className != "googie_list_revert" && this.className != "googie_list_close")
-        this.className = "googie_list_onhover";
+    if (this.className != 'googie_list_revert' && this.className != 'googie_list_close')
+        this.className = 'googie_list_onhover';
     else
-        this.parentNode.className = "googie_list_onhover";
-}
+        this.parentNode.className = 'googie_list_onhover';
+};
 this.item_onmouseout = function(e) {
-    if (this.className != "googie_list_revert" && this.className != "googie_list_close")
-        this.className = "googie_list_onout";
+    if (this.className != 'googie_list_revert' && this.className != 'googie_list_close')
+        this.className = 'googie_list_onout';
     else
-        this.parentNode.className = "googie_list_onout";
-}
+        this.parentNode.className = 'googie_list_onout';
+};
 
 
 };