diff options
| author | Jan Holesovsky <kendy@collabora.com> | 2021-03-24 15:39:05 +0100 |
|---|---|---|
| committer | Jan Holesovsky <kendy@collabora.com> | 2021-04-29 17:53:57 +0200 |
| commit | 0dc9da5df470b9c345e78dbe9553d81b9e4a7435 (patch) | |
| tree | e58c3d18d8dc6854c47fd2f28be0240595b66fec | |
| parent | e53ac4207d2f9e53f7bfe4f927961ebe2bd17182 (diff) | |
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 <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
(cherry picked from commit 65990058f041c3f1d280a69d411eb4ceacf5a721)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/113112
Tested-by: Jenkins
| -rw-r--r-- | sc/source/ui/app/inputhdl.cxx | 5 | ||||
| -rw-r--r-- | 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::Sequence<css::beans { SolarMutexGuard aGuard; + // disable word autocompletion + ScAppOptions aAppOptions( SC_MOD()->GetAppOptions() ); + aAppOptions.SetAutoComplete(false); + SC_MOD()->SetAppOptions(aAppOptions); + for (const beans::PropertyValue& rValue : rArguments) { if (rValue.Name == ".uno:SpellOnline" && rValue.Value.has<bool>()) |