Changeset b8d4feb in github

Timestamp:

Jun 30, 2010 7:17:51 AM (3 years ago)

Author:

alecpl <alec@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

3e91f08

Parents:

9ed374b

Message:

• googiespell layer resizing + small css improvements
• better googiespell suggestions popup positioning

Files:

• program/js/googiespell.js (modified) (6 diffs)
• skins/default/functions.js (modified) (1 diff)
• skins/default/googiespell.css (modified) (2 diffs)
• skins/default/mail.css (modified) (1 diff)

program/js/googiespell.js

@@ -112 +112 @@
         this.checkSpellingState();
     }
-    else 
-        if (this.report_ta_not_found)
-            alert('Text area not found');
+    else if (this.report_ta_not_found)
+        alert('Text area not found');
 };
 
@@ -394 +393 @@
         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('');
-
+    $(this.error_window).html('');
     $(table).addClass('googie_list').attr('googie_action_btn', '1');
 
@@ -542 +537 @@
     this.error_window.appendChild(table);
 
+    // calculate and set position
+    var height = $(this.error_window).height(),
+        width = $(this.error_window).width(),
+        pageheight = $(document).height(),
+        pagewidth = $(document).width(),
+        top = pos.top + height + 20 < pageheight ? pos.top + 20 : pos.top - height,
+        left = pos.left + width < pagewidth ? pos.left : pos.left - width;
+
+    $(this.error_window).css({'visibility': 'visible', 'top': top+'px', 'left': left+'px'});
+
     // Dummy for IE - dropdown bug fix
     if ($.browser.msie) {
@@ -562 +567 @@
 this.createEditLayer = function(width, height) {
     this.edit_layer = document.createElement('div');
-    $(this.edit_layer).addClass('googie_edit_layer').width(width-10).height(height);
+    $(this.edit_layer).addClass('googie_edit_layer').attr('id', 'googie_edit_layer')
+        .width(width).height(height);
 
     if (this.text_area.nodeName.toLowerCase() != 'input' || $(this.text_area).val() == '') {
@@ -622 +628 @@
 
     $(elm).html(text).addClass('googie_link').bind('click', d)
-        .attr({'googie_action_btn' : '1', 'g_id' : id, 'is_corrected' : false});
+            .attr({'googie_action_btn' : '1', 'g_id' : id, 'is_corrected' : false});
 
     return elm;
@@ -703 +709 @@
     this.language_window = document.createElement('div');
     $(this.language_window).addClass('googie_window')
-        .width(100).attr('googie_action_btn', '1');
+            .width(100).attr('googie_action_btn', '1');
 
     // Build up the result list

skins/default/functions.js

@@ -345 +345 @@
     $('#compose-body_ifr').width((w+2)+'px').height((h-54)+'px');
   }
+  else {
+    $('#googie_edit_layer').width(w-(bw.ie || bw.opera || bw.safari ? 2 : 0)+'px').height(h+'px');
+  }
 },
 

skins/default/googiespell.css

@@ -14 +14 @@
 
 .googie_edit_layer {
-  border: 1px solid #666666;
   background-color: #ffffff;
-  padding: 1px 4px;
-  margin: 1px 0px;
+  padding: 0 4px;
   font-size: 9pt;
   font-family: monospace;
@@ -76 +74 @@
   text-decoration: underline;
   cursor: pointer;
+  font-size: 9pt;
+  font-family: monospace;
 }
 

skins/default/mail.css

@@ -138 +138 @@
 }
 
-#messagetoolbar select,
-#compose-container select
+#messagetoolbar select
 {
   font-size: 11px;