[Proto-Scripty] Re: Prototype and WYSIWYG editors weird FF bug

2010-07-21 Thread T.J. Crowder
Hi, Oh, of course. Remember that HTML and JavaScript are not parsed by the same parsers, and you can confuse the HTML parser when you put script tags in *inline* JavaScript you've included within a script tag. What happens if you change this: '' + "\n" to: 'http://localhost/PearCMS2/Pub

[Proto-Scripty] Re: Prototype and WYSIWYG editors weird FF bug

2010-07-20 Thread T.J. Crowder
Hi, I meant where in _your_ code (but that's still useful info), e.g., this call to setStyle: this.editorFrame.setStyle( 'border: 1px inset;' ); (BTW, the semicolon inside the quotes there is an error, although most browsers will ignore it.) ...or this one: this.editorFrame.setStyle( {

[Proto-Scripty] Re: Prototype and WYSIWYG editors weird FF bug

2010-07-20 Thread Gindi Bar Yahav
OK, the bug in here setStyle: function(element, styles) { 2206 element = $(element); 2207 var elementStyle = element.style, match; 2208 if (Object.isString(styles)) { 2209 element.style.cssText += ';' + styles; 2210 return styles.include('opacity') ? 2211 element.setOpacity(styles.match(/opacity:\s

[Proto-Scripty] Re: Prototype and WYSIWYG editors weird FF bug

2010-07-20 Thread T.J. Crowder
Hi, And at what point does the error occur? One of the setStyle calls? Which one? I mean, I could guess, but guessing is a waste of time. -- T.J. On Jul 20, 1:33 pm, Gindi Bar Yahav wrote: > OK, sorry about the lack of details > here is the define of editorDocument > > /**             Iframe ex

[Proto-Scripty] Re: Prototype and WYSIWYG editors weird FF bug

2010-07-20 Thread Gindi Bar Yahav
OK, sorry about the lack of details here is the define of editorDocument /** Iframe exists already? **/ if ( $("PearServicesEditor_" + this.editorID + "_IFrame") ) { this.editorFrame=

[Proto-Scripty] Re: Prototype and WYSIWYG editors weird FF bug

2010-07-20 Thread T.J. Crowder
Hi, I don't think it's possible to help with this without a lot more information. What's `editorDocument`, for instance? You said that the failure occurs in `setStyle`; so you're going to need to show us your call to `setStyle` at the very least (or your call to whatever else in Prototype called `