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
signature.asc
Description: PGP signature