From 7e0dacb92bfd7de073c61b8b474cc397edd9bfdd Mon Sep 17 00:00:00 2001
From: Simon Legner <Simon.Legner@gmail.com>
Date: Fri, 4 Nov 2022 21:54:01 +0100
Subject: [PATCH] Settings: disable autofocus of search input

Fixes #1856.
---
 assets/javascripts/app/settings.coffee                 |  1 +
 .../javascripts/templates/pages/settings_tmpl.coffee   |  3 +++
 assets/javascripts/views/content/settings_page.coffee  |  1 +
 assets/javascripts/views/search/search.coffee          | 10 +++++++---
 4 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/assets/javascripts/app/settings.coffee b/assets/javascripts/app/settings.coffee
index 38b4f910..74e32a65 100644
--- a/assets/javascripts/app/settings.coffee
+++ b/assets/javascripts/app/settings.coffee
@@ -12,6 +12,7 @@ class app.Settings
     'layout'
     'size'
     'tips'
+    'noAutofocus'
     'autoInstall'
     'spaceScroll'
     'spaceTimeout'
diff --git a/assets/javascripts/templates/pages/settings_tmpl.coffee b/assets/javascripts/templates/pages/settings_tmpl.coffee
index 73c65650..048afa1a 100644
--- a/assets/javascripts/templates/pages/settings_tmpl.coffee
+++ b/assets/javascripts/templates/pages/settings_tmpl.coffee
@@ -34,6 +34,9 @@ app.templates.settingsPage = (settings) -> """
         <input type="checkbox" form="settings" name="layout" value="_sidebar-hidden"#{if settings['_sidebar-hidden'] then ' checked' else ''}>Automatically hide and show the sidebar
         <small>Tip: drag the edge of the sidebar to resize it.</small>
       </label>
+      <label class="_settings-label _hide-on-mobile">
+        <input type="checkbox" form="settings" name="noAutofocus" value="_no-autofocus"#{if settings.noAutofocus then ' checked' else ''}>Disable autofocus of search input
+      </label>
       <label class="_settings-label">
         <input type="checkbox" form="settings" name="autoInstall" value="_auto-install"#{if settings.autoInstall then ' checked' else ''}>Automatically download documentation for offline use
         <small>Only enable this when bandwidth isn't a concern to you.</small>
diff --git a/assets/javascripts/views/content/settings_page.coffee b/assets/javascripts/views/content/settings_page.coffee
index f0adebf7..7a7b6246 100644
--- a/assets/javascripts/views/content/settings_page.coffee
+++ b/assets/javascripts/views/content/settings_page.coffee
@@ -14,6 +14,7 @@ class app.views.SettingsPage extends app.View
     settings.theme = app.settings.get('theme')
     settings.smoothScroll = !app.settings.get('fastScroll')
     settings.arrowScroll = app.settings.get('arrowScroll')
+    settings.noAutofocus = app.settings.get('noAutofocus')
     settings.autoInstall = app.settings.get('autoInstall')
     settings.analyticsConsent = app.settings.get('analyticsConsent')
     settings.spaceScroll = app.settings.get('spaceScroll')
diff --git a/assets/javascripts/views/search/search.coffee b/assets/javascripts/views/search/search.coffee
index 7d05f0a0..55c31276 100644
--- a/assets/javascripts/views/search/search.coffee
+++ b/assets/javascripts/views/search/search.coffee
@@ -39,12 +39,16 @@ class app.views.Search extends app.View
     return
 
   focus: =>
-    @input.focus() unless document.activeElement is @input
+    return if document.activeElement is @input
+    return if app.settings.get('noAutofocus')
+    @input.focus() 
     return
 
   autoFocus: =>
-    unless app.isMobile() or $.isAndroid() or $.isIOS()
-      @input.focus() unless document.activeElement?.tagName is 'INPUT'
+    return if app.isMobile() or $.isAndroid() or $.isIOS()
+    return if document.activeElement?.tagName is 'INPUT'
+    return if app.settings.get('noAutofocus')
+    @input.focus()
     return
 
   onWindowFocus: (event) =>