Changeset 3979 for BreadCrumbs

Timestamp:

05/13/09 13:39:10 (3 years ago)

Author:

skruk

Message:

added getText() to provide compatibility fallback
tested on FF2, FF3, Safari4, Opera, IE8
unfixed: IE6/7

Location:

BreadCrumbs/js

Files:

• breadcrumbslist.js (modified) (2 diffs)
• crumb.js (modified) (3 diffs)

BreadCrumbs/js/breadcrumbslist.js

@@ -97 +97 @@
                 return { 
                         fontSize: parseInt(cont.getStyle("font-size")),
-                        len:  cont.textContent.length, 
-                        text: cont.textContent
+                        len:  cont.getText().length, 
+                        text: cont.getText()
                 };
                 
@@ -131 +131 @@
                 var span = document.createElement("span");
                 span.style.fontSize = maxEl.fontSize;
-                span.innerHTML = maxEl.text;
+                span.innerHTML = maxEl.text.gsub(/\s+/, " ");
                 span.hide();
                 $$("body")[0].appendChild(span);

BreadCrumbs/js/crumb.js

@@ -114 +114 @@
                         var len = Math.floor(this.dimensions.width/fsize);
 
-                        console.log("TEXT: "+this.text);
-                        
                         if ( len > this.text.length )
                         {
@@ -183 +181 @@
                 }
 
-                console.log("CRUMB: "+this.acrumb.textContent);
-                
-                this.text = ( this.acrumb != null ) ? ( this.acrumb.textContent ) : "";
+                this.text = ( this.acrumb != null ) ? ( this.acrumb.getText() ) : "";
                 
                 Event.observe(this.licrumb, "mouseover", this.breadcrumbs.focusOn.bindAsEventListener(this.breadcrumbs, this, this));
@@ -191 +187 @@
         }
 });
+
+Object.extend(Element.prototype, {
+        getText: function(){
+                return (!Object.isUndefined(this.textContent)) ? this.textContent : this.innerText;
+        }
+});
+