diff options
| author | Robin Haberkorn <rhaberkorn@fmsbw.de> | 2023-04-18 11:58:19 +0300 |
|---|---|---|
| committer | Robin Haberkorn <rhaberkorn@fmsbw.de> | 2025-11-09 20:31:36 +0100 |
| commit | d4fa2014808092ddfaa0c97c1755af87a182be6b (patch) | |
| tree | 1e13a29e1f3e3ca636f5137bb886562e2b467a54 /src | |
| parent | 46e22dc1fb24189bfd63473ea63023087276c639 (diff) | |
| download | scintilla-mirror-d4fa2014808092ddfaa0c97c1755af87a182be6b.tar.gz | |
disable automatic scrolling and choosing the X that caret sticks to
* SciTECO needs to avoid automatic scrolling as an optimization.
While this works partially by avoiding certain messages like SCI_GOTOPOS (in favor of SCI_SETEMPTYSELECTION),
there aren't alternatives for all messages that scroll.
For instance SCI_UNDO will always call EnsureCaretVisible().
Also, even if we could avoid all scrolling messages, there is no guarantee that will not suddenly
and unexpectedly break in the future.
* Instead, every scrolling is now disabled except for an explicit SCI_SCROLLCARET.
SciTECO can therefore use the more intuitive messages like SCI_GOTOPOS.
* SetLastXChosen() (choosing the X that caret sticks to) has been found to be a major slow down
during UNDO and serves no purpose on SciTECO anyway - we have to implement such an algorithm
at the SciTECO language level.
We therefore simply disable SetLastXChosen().
Diffstat (limited to 'src')
| -rw-r--r-- | src/Editor.cxx | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/Editor.cxx b/src/Editor.cxx index 56eb4bdda..e971e02f1 100644 --- a/src/Editor.cxx +++ b/src/Editor.cxx @@ -929,8 +929,10 @@ Point Editor::PointMainCaret() { * as it moves up and down. */ void Editor::SetLastXChosen() { +#if 0 const Point pt = PointMainCaret(); lastXChosen = static_cast<int>(pt.x) + xOffset; +#endif } void Editor::RememberSelectionForUndo(int index) { @@ -1442,11 +1444,13 @@ void Editor::ScrollRange(SelectionRange range) { } void Editor::EnsureCaretVisible(bool useMargin, bool vert, bool horiz) { +#if 0 SetXYScroll(XYScrollToMakeVisible(SelectionRange(posDrag.IsValid() ? posDrag : sel.RangeMain().caret), (useMargin?XYScrollOptions::useMargin:XYScrollOptions::none)| (vert?XYScrollOptions::vertical:XYScrollOptions::none)| (horiz?XYScrollOptions::horizontal:XYScrollOptions::none), caretPolicies)); +#endif } void Editor::ShowCaretAtCurrentPosition() { @@ -6601,7 +6605,9 @@ sptr_t Editor::WndProc(Message iMessage, uptr_t wParam, sptr_t lParam) { break; case Message::ScrollCaret: - EnsureCaretVisible(); + SetXYScroll(XYScrollToMakeVisible(SelectionRange(posDrag.IsValid() ? posDrag : sel.RangeMain().caret), + XYScrollOptions::useMargin | XYScrollOptions::vertical | XYScrollOptions::horizontal, + caretPolicies)); break; case Message::SetReadOnly: |