editorShell.wrapColumn busted now

VERIFIED INVALID

Status

()

P3
normal
VERIFIED INVALID
19 years ago
15 years ago

People

(Reporter: pete, Assigned: akkzilla)

Tracking

Trunk
x86
FreeBSD
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
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; 
}
(Assignee)

Comment 1

19 years ago
What is it doing now?  I.e. how do you test that it is not working?
Status: NEW → ASSIGNED
(Reporter)

Comment 2

19 years ago
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
(Assignee)

Comment 3

19 years ago
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?
(Assignee)

Comment 4

19 years ago
Seems to work for me in the plaintext editor.

Comment 5

19 years ago
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
(Assignee)

Comment 6

19 years ago
No problem -- I didn't spend much time on it since I suspected that might be the
problem. :-)
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → INVALID

Comment 7

19 years ago
verified in 4/3 build.
Status: RESOLVED → VERIFIED

Comment 8

19 years ago
setting to an approximate milestone so it can be off of the no TFV list
Target Milestone: --- → M15
You need to log in before you can comment on or make changes to this bug.