On Mon, Mar 22, 2010 at 9:08 AM, P T Withington <[email protected]>wrote:
> On 2010-03-21, at 18:53, Henry Minsky wrote: > > >> > >> 2) I don't understand the change to scrollevent. If an inputtextview is > >> autosizing, it won't have scrollevents enabled will it? It should only > be > >> getting scrollevents if it has a fixed size (and hence would scroll any > text > >> that does not fit). > >> > > > > Oh yeah you're right. I need to ensure that when the text changes that > new > > text height info is stuffed into the LFC some other way > > in that case. > > > > Actually, I'm wrong. From LzTextSprite.js line 267: > > > // NOTE [2009-04-08 ptw] We always send lineHeight events, even if > > // scrollevents are not requested. scrollevent should probably be > > // renamed to spriteevent or something > > So, your fix does work -- scrollevent is just poorly named. No, I think you were right the first time, the lineHeight event specifies the interline spacing, but I need the LFC to get the entire text content height, So I think we also need that code to send 'scrollHeight' events unconditionally also, to inform the LFC of the total text height. -- Henry Minsky Software Architect [email protected]
