Clean JS for Search (#698)

epfl-si · Feb 23, 2024 · 84213b0 · 84213b0
1 parent 2cf42ee
commit 84213b0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/assets/components/molecules/search/search-mobile.js b/assets/components/molecules/search/search-mobile.js
@@ -2,24 +2,23 @@
 
 export default () => {
 
-  $('#search-mobile-toggle').click(function(event){
+  $('#search-mobile-toggle').on('click', () => {
 
-    var searchContainer = $('.search-mobile');
-    var searchField = searchContainer.find('.form-control');
+    const searchContainer = $('.search-mobile');
+    const searchField = searchContainer.find('.form-control');
 
     searchContainer.toggleClass('show');
     $('body').toggleClass('search-open');
 
-    if ( searchContainer.hasClass('show') ) {
-      searchField.focus();
+    if (searchContainer.hasClass('show')) {
+      searchField.trigger('focus');
     }
 
   });
 
-  $('#search-mobile-close').click(function(event){
+  $('#search-mobile-close').on('click', () => {
 
-    var searchContainer = $('.search-mobile');
-    var searchField = searchContainer.find('.form-control');
+    const searchContainer = $('.search-mobile');
 
     searchContainer.removeClass('show');
     $('body').removeClass('search-open');

diff --git a/assets/components/molecules/search/search.js b/assets/components/molecules/search/search.js
@@ -2,6 +2,6 @@
 
 export default () => {
   $('.search').on('shown.bs.dropdown', () => {
-    $('.search input[type="text"]').focus();
+    $('.search input[type="text"]').trigger('focus');
   });
 };