On Thursday, September 27, 2012 6:49:10 PM UTC-5, Edward K. Ream wrote: > > I now have a prototype for a more complete solution to the unwanted > scrolling.
Rev 5465 fixes some more scroll-related problems. These were much more tractable once w.setStyleSheet calls are mostly gone. Here is the checkin log: QQQ A number of important improvements for scroll position. - Activated onSliderChanged. - Call c.p.v.restoreCursorAndScroll() in onActivateEvent. - Removed the *evil* maintain_scroll setting. - p/v.restoreCursorAndScroll and p/v.saveCursorAndScroll no longer take a w argument. QQQ The scrolling code is getting more and more simple, but there are still subtleties. I'm not sure whether I put in the maintain_scroll setting, but I consider it completely evil in the present environment. Indeed, it appears that moving between panes *never* alters either insert point or the scroll position, which is of course how it should be. As always, I welcome all bug reports, but please DO NOT change any of the scrolling code without consulting with me first. In particular, calls to QTextBrowser.ensureCursorVisible, or equivalently, c.frame.body.seeInsertPoint or c.frame.body.bodyCtrl.seeInsertPoint must be used only in unusual circumstances, or perhaps even never :-) Edward -- You received this message because you are subscribed to the Google Groups "leo-editor" group. To view this discussion on the web visit https://groups.google.com/d/msg/leo-editor/-/7VaDlI3h-i8J. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en.
