From 96a0575c93f6150dc890e9ac272dbc6635723989 Mon Sep 17 00:00:00 2001 From: Simon Cambier Date: Mon, 31 Oct 2022 11:22:41 +0100 Subject: [PATCH] #120 - Disabling PDF Indexing on iOS --- src/settings.ts | 50 ++++++++++++++++++++++++++++--------------------- 1 file changed, 29 insertions(+), 21 deletions(-) diff --git a/src/settings.ts b/src/settings.ts index 1bfa861..36a7b01 100644 --- a/src/settings.ts +++ b/src/settings.ts @@ -143,27 +143,29 @@ export class SettingsTab extends PluginSettingTab { }) ) - // PDF Indexing - const indexPDFsDesc = new DocumentFragment() - indexPDFsDesc.createSpan({}, span => { - span.innerHTML = `Omnisearch will include PDFs in search results. - - Needs a restart to fully take effect.` - }) - new Setting(containerEl) - .setName('BETA - PDF Indexing') - .setDesc(indexPDFsDesc) - .addToggle(toggle => - toggle.setValue(settings.PDFIndexing).onChange(async v => { - settings.PDFIndexing = v - await saveSettings(this.plugin) - }) - ) + // PDF Indexing - disabled on iOS + if (!Platform.isIosApp) { + const indexPDFsDesc = new DocumentFragment() + indexPDFsDesc.createSpan({}, span => { + span.innerHTML = `Omnisearch will include PDFs in search results. + + Needs a restart to fully take effect.` + }) + new Setting(containerEl) + .setName('BETA - PDF Indexing') + .setDesc(indexPDFsDesc) + .addToggle(toggle => + toggle.setValue(settings.PDFIndexing).onChange(async v => { + settings.PDFIndexing = v + await saveSettings(this.plugin) + }) + ) + } // #endregion Behavior // #region User Interface @@ -324,6 +326,12 @@ export let settings = Object.assign({}, DEFAULT_SETTINGS) as OmnisearchSettings export async function loadSettings(plugin: Plugin): Promise { settings = Object.assign({}, DEFAULT_SETTINGS, await plugin.loadData()) + + // Make sure that PDF indexing is disabled on iOS + if (Platform.isIosApp) { + settings.PDFIndexing = false + } + showExcerpt.set(settings.showExcerpt) }