34353 - editorShell.wrapColumn busted now

Status: VERIFIED INVALID

(Core :: DOM: Editor, defect, P3)

Description

[Pete Collins (MDG)]

I had this code working the beginning of last week. 
From what i understand it is just an attribute that you can simply change the
value of. 

function setWrap(){ 
        dump("Setting Wrap\n\n"); 
        dump("Old Wrap = "+editorShell.wrapColumn+"\n\n"); 
        editorShell.wrapColumn = 1000; 
}

Comment 1

[Akkana Peck]

What is it doing now?  I.e. how do you test that it is not working?

Comment 2

[Pete Collins (MDG)]

Easy, the function i posted is called right before i call 

InsertText('some really long string')

Last week, i would get scrollbars on the bottom of the edit field to accomodate
that large column number.

1000 is *pretty* big.


Also if you run the code, the dump of the existing column size doesn't work att
all.

I will put a test in composer and see what happens.

pete

Comment 3

[Akkana Peck]

This is in a plaintext editor, I assume?  Wrapping is meaningless in an html
editor since html doesn't have hard wrapping.  Is this in the regular plaintext
editor window?

Why are you setting wrap column to 1000?  Why not just set it to -1 (no wrap)
instead?

Comment 4

[Akkana Peck]

Seems to work for me in the plaintext editor.

Comment 5

[pete collins]

Ah . . . I don't beleive it.

I am real sorry. Mark this invalid. I changed over from plaintext to html a few
days ago.

Well, i guess i have successfully just wasted your time.

pete

Comment 6

[Akkana Peck]

No problem -- I didn't spend much time on it since I suspected that might be the
problem. :-)

Comment 7

[sujay]

verified in 4/3 build.

Comment 8

[rubydoo123]

setting to an approximate milestone so it can be off of the no TFV list