summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoralecpl <alec@alec.pl>2010-02-23 17:22:39 +0000
committeralecpl <alec@alec.pl>2010-02-23 17:22:39 +0000
commit1617dbb6a45ccd1fa6fc74ab6c47026804c7a1e0 (patch)
tree03a737a92dcb539914189967d11311ba9e7149df
parent3e7c3694ef2b516610c0592712e95046a2bde227 (diff)
- Fix IE spellcheck suggestion popup issue (#1486471)
-rw-r--r--CHANGELOG1
-rw-r--r--program/js/googiespell.js24
2 files changed, 12 insertions, 13 deletions
diff --git a/CHANGELOG b/CHANGELOG
index 5e0460ae3..34c3e76aa 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -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)
diff --git a/program/js/googiespell.js b/program/js/googiespell.js
index abd3462ec..5f201b7a1 100644
--- a/program/js/googiespell.js
+++ b/program/js/googiespell.js
@@ -80,7 +80,7 @@ function GoogieSpell(img_dir, server_url) {
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.decorateTextarea = function(id) {
/////
this.setSpellContainer = function(id) {
this.spell_container = typeof(id) == 'string' ? document.getElementById(id) : id;
-
}
this.setLanguages = function(lang_dict) {
@@ -225,7 +224,7 @@ this.spellCheck = function(ignore) {
this.removeIndicator();
return;
}
-
+
this.createEditLayer(this.text_area.offsetWidth, this.text_area.offsetHeight);
this.createErrorWindow();
$('body').append(this.error_window);
@@ -309,6 +308,7 @@ this.parseResult = function(r_text) {
}
results.push(item);
}
+
return results;
}
@@ -402,13 +402,11 @@ this.showErrorWindow = function(elm, id) {
//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 @@ this.showErrorWindow = function(elm, id) {
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 @@ this.showErrorWindow = function(elm, id) {
//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});