Changeset 382347

Timestamp:

05/08/2011 08:10:13 AM (15 years ago)

Author:

kei51

Message:

Fixed issues on IE browsers.

Location:

prompty/trunk

Files:

• prompty.js (modified) (8 diffs)
• prompty.php (modified) (3 diffs)
• readme.txt (modified) (3 diffs)
• test.html (modified) (1 diff)

prompty/trunk/prompty.js

@@ -18 +18 @@
 
 var prompty = {};
-prompty.str = null;
 prompty.target = null;
 prompty.data = null;
-prompty.prompts = ['% ', '# ', '> '];
+prompty.prompts = ['% ', '# ', '> ', '$ '];
 prompty.speed = 100;
+prompty.buttonIdPrefix = '_promtpy:';
 prompty.buttons = [];
+prompty.contents = [];
+prompty.isIE = navigator.appName == 'Microsoft Internet Explorer';
 
 prompty.style = {};
@@ -30 +32 @@
 prompty.style.color = '#999999';
 prompty.style.backgroundColor = '#ffffff';
-prompty.style.font = '8pt courier,helvetica,arial,sans-serif';
+prompty.style.font = '8pt courier,sans-serif';
 prompty.style.innerHTML = '>Play';
-
 
 prompty.setup = function()
@@ -39 +40 @@
   for (var i=0; i<elems.length; i++)
   {
-    if ( /prompty/.test(elems[i].className))
+    if (/prompty/.test(elems[i].className))
     {
-//      elems[i].onclick = function() 
-//      {
-//        prompty.hideButtons(); 
-//        prompty.start(this); 
-//      };
       var pos = prompty.getPosition(elems[i]);
-      var label = document.createElement('div');
-      label.style.position = 'absolute';
-      label.style.top = pos.y + 'px';
-      label.style.left = pos.x + 'px';
-      label.style.padding = prompty.style.padding;
-      label.style.border = prompty.style.border;
-      label.style.color = prompty.style.color;
-      label.style.backgroundColor = prompty.style.backgroundColor;
-      label.style.cursor = 'pointer';
-      label.style.font = prompty.style.font;
-      label.innerHTML = prompty.style.innerHTML;
-      label.onclick = function() 
+      var button = document.createElement('div');
+      button.style.position = 'absolute';
+      button.style.top = pos.y + 'px';
+      button.style.left = pos.x + 'px';
+      button.style.padding = prompty.style.padding;
+      button.style.border = prompty.style.border;
+      button.style.color = prompty.style.color;
+      button.style.backgroundColor = prompty.style.backgroundColor;
+      button.style.cursor = 'pointer';
+      button.style.font = prompty.style.font;
+      button.innerHTML = prompty.style.innerHTML;
+      button.id = prompty.buttonIdPrefix + i;
+      button.onclick = function() 
       { 
         prompty.hideButtons(); 
-        prompty.start(this.nextSibling); 
+        prompty.start(this); 
       };
-      elems[i].parentNode.insertBefore(label, elems[i]);
-      prompty.buttons.push(label);
+      elems[i].parentNode.insertBefore(button, elems[i]);
+      // store all the button elements for show/hide buttons.
+      prompty.buttons.push(button);
+      // store all the pre tag text contents.
+      prompty.contents.push(elems[i].textContent ? elems[i].textContent : elems[i].innerText);
     }
   }
@@ -83 +83 @@
 prompty.start = function(elem)
 {
-  prompty.str = elem.textContent;
-  prompty.target = elem;
-  prompty.target.innerHTML = '';
-  prompty.data = prompty.parse(prompty.str);
+  var idx = elem.id.replace(prompty.buttonIdPrefix, '');
+  prompty.target = elem.nextSibling;   // target pre tag
+  prompty.target.innerHTML = '';       // clear the pre tag contents
+  prompty.data = prompty.parse(prompty.contents[idx]);
   prompty.render();
 }
@@ -92 +92 @@
 prompty.parse = function(str)
 {
-  var lines = str.split('\n'); // line feed letter itself will be removed here.
+  var lfstr = prompty.isIE ? '<br/>' : '\n';
+  var lines = str.split('\n'); 
   var data = []; 
   
@@ -98 +99 @@
   {    
     var p = prompty.isPrompt(lines[i]);
-    var lf = (i==lines.length - 1) ? '' : '\n';
-    if (p==null)
-    {
-      data.push(lines[i] + lf);
-    }
-    else
+    var lf = (i==lines.length - 1 && !prompty.isIE ) ? '' : lfstr;
+    if (p)
     {
       data.push(p);
@@ -110 +107 @@
       data = data.concat(chars);
       data.push(lf);
+    }
+    else
+    {
+      data.push(lines[i] + lf);
     }
   }
@@ -137 +138 @@
 prompty.getPosition = function (elem)
 {
-  var left = top = 0;
+  var left = 0, top = 0;
   if (elem.offsetParent)
   {

prompty/trunk/prompty.php

@@ -4 +4 @@
 Plugin URI: http://wordpress.org/extend/plugins/prompty/
 Description: This plugin adds the prompt-like effect on the pre tag contents in your post. 
-Version: 1.0
+Version: 1.1
 Author: Kei Saito
 Author URI: 
@@ -64 +64 @@
     //  as prompt lines, you can modify the line like following.
   //
-  //  prompty.prompts = ['% ', '# ', '> ', 'SQL> '];
+  //  prompty.prompts = ['% ', '# ', '> ', '$ ', 'SQL> '];
   //
-  prompty.prompts = ['% ', '# ', '> ' ];
+  prompty.prompts = ['% ', '# ', '> ', '$ ' ];
 
     //  You can change the speed here.
@@ -83 +83 @@
   prompty.style.color = '#999999';
   prompty.style.backgroundColor = '#ffffff';
-  prompty.style.font = '8pt courier,helvetica,arial,sans-serif';
+  prompty.style.font = '8pt courier,sans-serif';
   prompty.style.innerHTML = '>Play';
 

prompty/trunk/readme.txt

@@ -5 +5 @@
 Requires at least: 2.7.0
 Tested up to: 3.1.2
-Stable tag: 1.0
+Stable tag: 1.1
 
 This plugin adds the prompt-like effect on the pre tag contents in your post.
@@ -48 +48 @@
 Please edit the `prompty.php` with the plugin editor for all the customizations. 
 
+= Buttons are not positioned/aligned correctly =
+I see some small alignment issues especially on IE. 
+Those are known issues and I will fix in later release.
+
 == Screenshots ==
 
@@ -54 +58 @@
 
 == Changelog ==
+= 1.1 =
+* Fixed IE issues.
+
 = 1.0 =
 * Initinal version.
 
 == Upgrade Notice ==
-No upgrade notice here since no previous versions.
 

prompty/trunk/test.html

@@ -22 +22 @@
 [scott@server1] % pwd
 /home/paul/dev
+
+
 
 </pre>