I don't often use continuous spellcheck, but I was surprised when I
toggled it and LyX recentered. I am guessing that when users are
scrolling and then toggle it, they do not expect LyX to recenter around
the cursor. What do others think?

The attached patch makes it so LyX does not recenter after toggling
continuous spellcheck.

Scott
From a0f58ac0dd1bd1cc75a4562cbfc8b773938fcd42 Mon Sep 17 00:00:00 2001
From: Scott Kostyshak <skost...@lyx.org>
Date: Sun, 30 Oct 2016 19:38:28 -0400
Subject: [PATCH] Do not recenter screen on spelling-continuously

If a user is scrolling and then toggles spellcheck, LyX used to
recenter the screen around the cursor. Now LyX does not recenter.
---
 src/frontends/qt4/GuiView.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/frontends/qt4/GuiView.cpp b/src/frontends/qt4/GuiView.cpp
index 42ee532..fb57c2e 100644
--- a/src/frontends/qt4/GuiView.cpp
+++ b/src/frontends/qt4/GuiView.cpp
@@ -4072,7 +4072,7 @@ void GuiView::dispatch(FuncRequest const & cmd, 
DispatchResult & dr)
 
                case LFUN_SPELLING_CONTINUOUSLY:
                        lyxrc.spellcheck_continuously = 
!lyxrc.spellcheck_continuously;
-                       dr.screenUpdate(Update::Force | Update::FitCursor);
+                       dr.screenUpdate(Update::Force);
                        break;
 
                default:
-- 
2.7.4

Attachment: signature.asc
Description: PGP signature

Reply via email to