setOption( "firstLineNumber", 0 ) only works correctly on scrollable and scrolled content

[FredGandt]

Setting the firstLineNumber to zero fails unless the code displayed is scrollable and has been scrolled.

Setting editor.setOptions( { "firstLineNumber": 0, "scrollPastEnd": true } ); will trick the first line number to show as zero on shorter content, but still only after the content has been scrolled enough to push the first line out of view.
On shorter content, without scrollPastEnd set, the first line number will always be empty.

Adding a hack to set the HTML textContent to zero on initializing ace, causes the first visible line number to always show as zero, even if, whilst scrolling, it should be something else.

FYI: Linden Scripting Language (LSL)'s line numbering starts at zero.

BTW: ace.js is great.

[nightwing]

I can't reproduce this issue, when i run editor.setOptions( { "firstLineNumber": 0 } ); on https://ace.c9.io/ the 0 is shown correctly. Do you use an older version of the editor?

[FredGandt]

I downloaded the build from https://github.com/ajaxorg/ace-builds/ about 3 days ago.

Used the "editor.html" in the build as a template (just realised that it uses a <pre> instead of a <div> like the "...and use the code below" example at https://ace.c9.io/#nav=embedding which might make a difference?).

Am using the "src-min-noconflict", and to save explaining everything else, it's being served at http://fredgandts.website/lsl/editor/

You should see that until the code is scrolled, the zero is not shown.

Being JS, I should also have mentioned using Chrome (Version 57.0.2987.133) on Win7 Home Premium (SP1)

I am quite prepared to be told I'm doing it wrong, I just can't see how right now.

[nightwing]

The link was very helpful, thanks!
The fix is at 597bd32

[FredGandt]

I found var b=c=h?h.getText(t,v):v+l;b!=p.textNode.data&&(p.textNode.data=b),v++ in "ace.js" in "src-min-noconflict" and changed it to var b=c=h?h.getText(t,v):v+l;b!==p.textNode.data&&(p.textNode.data=b),v++ and am pleased to say it works. Thank you.

Reading minified code is slightly less fun than one might think ;-)