Snow: > OK, good point. I've attached a revised patch that adds a field > Editor::currentPageWidth. This is updated in Paint(); and Idle() calls > SetScrollBars() when currentPageWidth != scrollWidth.
This change makes the scroll width too variable I think: it will extend the scroll range when painting a wide line, but may also, sometimes, shrink the scroll range when moving to a narrow portion of the file. It should really only act to widen the range automatically. I don't think that all interactions between scrollWidth and currentPageWidth are well defined: currentPageWidth should have a defined invalid setting, probably 0 so that setting it is a signal rather than a state. For clients that call SCI_SETSCROLLWIDTH, it should be possible to enable / disable the automatic sizing mechanism. The use of SIF_DISABLENOSCROLL means that there is always a visible scroll bar even when not needed which decreases the amount of text visible and will make Scintilla less useful for those using it for small entry boxes. The workaround double call to ShowScrollBar looks like covering up a problem that may be more tightly definable. Neil _______________________________________________ Scintilla-interest mailing list [email protected] http://mailman.lyra.org/mailman/listinfo/scintilla-interest
