mirror of
https://github.com/laurent22/joplin.git
synced 2024-12-24 10:27:10 +02:00
Desktop: Disable fuzzy search for now due to performance issues
This commit is contained in:
parent
947d81d96d
commit
3e86ae4a82
@ -736,18 +736,23 @@ class BaseApplication {
|
||||
setLocale(Setting.value('locale'));
|
||||
}
|
||||
|
||||
if (Setting.value('db.fuzzySearchEnabled') === -1) {
|
||||
const fuzzySearchEnabled = await this.database_.fuzzySearchEnabled();
|
||||
Setting.setValue('db.fuzzySearchEnabled', fuzzySearchEnabled ? 1 : 0);
|
||||
}
|
||||
// if (Setting.value('db.fuzzySearchEnabled') === -1) {
|
||||
// const fuzzySearchEnabled = await this.database_.fuzzySearchEnabled();
|
||||
// Setting.setValue('db.fuzzySearchEnabled', fuzzySearchEnabled ? 1 : 0);
|
||||
// }
|
||||
|
||||
// Always disable on CLI because building and packaging the extension is not working
|
||||
// and is too error-prone - requires gcc on the machine, or we should package the .so
|
||||
// and dylib files, but it's not sure it would work everywhere if not built from
|
||||
// source on the target machine.
|
||||
if (Setting.value('appType') !== 'desktop') {
|
||||
Setting.setValue('db.fuzzySearchEnabled', 0);
|
||||
}
|
||||
// // Always disable on CLI because building and packaging the extension is not working
|
||||
// // and is too error-prone - requires gcc on the machine, or we should package the .so
|
||||
// // and dylib files, but it's not sure it would work everywhere if not built from
|
||||
// // source on the target machine.
|
||||
// if (Setting.value('appType') !== 'desktop') {
|
||||
// Setting.setValue('db.fuzzySearchEnabled', 0);
|
||||
// }
|
||||
|
||||
// For now always disable fuzzy search due to performance issues:
|
||||
// https://discourse.joplinapp.org/t/1-1-4-keyboard-locks-up-while-typing/11231/11
|
||||
// https://discourse.joplinapp.org/t/serious-lagging-when-there-are-tens-of-thousands-of-notes/11215/23
|
||||
Setting.setValue('db.fuzzySearchEnabled', 0);
|
||||
|
||||
if (Setting.value('encryption.shouldReencrypt') < 0) {
|
||||
// We suggest re-encryption if the user has at least one notebook
|
||||
|
Loading…
Reference in New Issue
Block a user