From 0dc9da5df470b9c345e78dbe9553d81b9e4a7435 Mon Sep 17 00:00:00 2001 From: Jan Holesovsky Date: Wed, 24 Mar 2021 15:39:05 +0100 Subject: lok: Disable the "AutoInput" again. This partially reverts "lok: sc - suppress LOK editengine events for the calc input bar." The feature itself is very problematic in Online: 1) causes unwanted jumps to other cells, 2) causes the selection blinking in the cell when typing, and 3) it is very annoying in the form that in which it is implemented in LibreOffice anyway, compared to other office suites. Let's disable it, and enable again when we address the above issues. This (partially) reverts commit 91319ad56887f932b2da334db560d5d0a79a0280. Change-Id: I2234455c29069f74d13896474f3499035935931b Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113047 Tested-by: Jenkins CollaboraOffice Reviewed-by: Jan Holesovsky (cherry picked from commit 65990058f041c3f1d280a69d411eb4ceacf5a721) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113112 Tested-by: Jenkins --- sc/source/ui/app/inputhdl.cxx | 5 +++-- sc/source/ui/unoobj/docuno.cxx | 5 +++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/sc/source/ui/app/inputhdl.cxx b/sc/source/ui/app/inputhdl.cxx index 3abf07641963..bca87cd88aaa 100644 --- a/sc/source/ui/app/inputhdl.cxx +++ b/sc/source/ui/app/inputhdl.cxx @@ -801,7 +801,6 @@ ScInputHandler::ScInputHandler() if (comphelper::LibreOfficeKit::isActive()) { ScInputHandler::bOptLoaded = true; // Evaluate App options - ScInputHandler::bAutoComplete = true; // Is set in KeyInput } } @@ -2707,7 +2706,9 @@ void ScInputHandler::UpdateFormulaMode() if (pInputWin) pInputWin->SetFormulaMode(true); - if ( bAutoComplete ) + // in LOK, we always need to perform the GetFormulaData() call so + // that the formula insertion works + if (bAutoComplete || comphelper::LibreOfficeKit::isActive()) GetFormulaData(); UpdateParenthesis(); diff --git a/sc/source/ui/unoobj/docuno.cxx b/sc/source/ui/unoobj/docuno.cxx index 371cb395e139..1729c2ca1cdb 100644 --- a/sc/source/ui/unoobj/docuno.cxx +++ b/sc/source/ui/unoobj/docuno.cxx @@ -1179,6 +1179,11 @@ void ScModelObj::initializeForTiledRendering(const css::uno::SequenceGetAppOptions() ); + aAppOptions.SetAutoComplete(false); + SC_MOD()->SetAppOptions(aAppOptions); + for (const beans::PropertyValue& rValue : rArguments) { if (rValue.Name == ".uno:SpellOnline" && rValue.Value.has()) -- cgit