[Svn] r3284 - in trunk/roundcubemail: . program/js

[trac at roundcube.net]

Author: alec
Date: 2010-02-23 11:22:39 -0600 (Tue, 23 Feb 2010)
New Revision: 3284

Modified:
   trunk/roundcubemail/CHANGELOG
   trunk/roundcubemail/program/js/googiespell.js
Log:
- Fix IE spellcheck suggestion popup issue (#1486471)


Modified: trunk/roundcubemail/CHANGELOG
===================================================================
--- trunk/roundcubemail/CHANGELOG	2010-02-23 16:28:38 UTC (rev 3283)
+++ trunk/roundcubemail/CHANGELOG	2010-02-23 17:22:39 UTC (rev 3284)
@@ -1,6 +1,7 @@
 CHANGELOG RoundCube Webmail
 ===========================
 
+- Fix IE spellcheck suggestion popup issue (#1486471)
 - Fix email address auto-completion shows regexp pattern (#1486258)
 - Fix merging of configuration parameters: user prefs always survive (#1486368) 
 - Fix quota indicator value after folder purge/expunge (#1486488)

Modified: trunk/roundcubemail/program/js/googiespell.js
===================================================================
--- trunk/roundcubemail/program/js/googiespell.js	2010-02-23 16:28:38 UTC (rev 3283)
+++ trunk/roundcubemail/program/js/googiespell.js	2010-02-23 17:22:39 UTC (rev 3284)
@@ -80,7 +80,7 @@
     this.cnt_errors = 0;
     this.cnt_errors_fixed = 0;
     
-    //Set document on click 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();
@@ -122,7 +122,6 @@
 /////
 this.setSpellContainer = function(id) {
     this.spell_container = typeof(id) == 'string' ? document.getElementById(id) : id;
-
 }
 
 this.setLanguages = function(lang_dict) {
@@ -225,7 +224,7 @@
         this.removeIndicator();
         return;
     }
-    
+
     this.createEditLayer(this.text_area.offsetWidth, this.text_area.offsetHeight);
     this.createErrorWindow();
     $('body').append(this.error_window);
@@ -309,6 +308,7 @@
     	}
         results.push(item);
     }
+
     return results;
 }
 
@@ -402,13 +402,11 @@
 
     //Check if we should use custom menu builder, if not we use the default
     var changed = false;
-    if (this.custom_menu_builder != []) {
-        for (var k=0; k<this.custom_menu_builder.length; k++) {
-            var eb = this.custom_menu_builder[k];
-            if(eb[0]((this.results[id]))){
-                changed = eb[1](this, list, elm);
-                break;
-            }
+    for (var k=0; k<this.custom_menu_builder.length; k++) {
+        var eb = this.custom_menu_builder[k];
+        if(eb[0]((this.results[id]))){
+            changed = eb[1](this, list, elm);
+            break;
         }
     }
     if (!changed) {
@@ -467,7 +465,7 @@
             revert_row.appendChild(revert);
             list.appendChild(revert_row);
         }
-        
+
         //Append the edit box
         var edit_row = document.createElement('tr');
         var edit = document.createElement('td');
@@ -545,11 +543,11 @@
     //Dummy for IE - dropdown bug fix
     if ($.browser.msie) {
 	if (!this.error_window_iframe) {
-            var iframe = $('<iframe>').css('position', 'absolute').css('z-index', 0);
+            var iframe = $('<iframe>').css({'position': 'absolute', 'z-index': -1});
 	    $('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,
     	    'width': this.error_window.offsetWidth, 'height': this.error_window.offsetHeight});

_______________________________________________
http://lists.roundcube.net/mailman/listinfo/svn