fixing bug where first gene would click if is_multigene set

IGS · Jan 22, 2024 · 08f0e43 · 08f0e43
1 parent 3ff046d
commit 08f0e43
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/www/js/expression.js b/www/js/expression.js
@@ -73,7 +73,7 @@ document.addEventListener('DOMContentLoaded', () => {
 
             // auto-select the first gene in the list
             const first_gene = document.querySelector('.gene-result-list-item');
-            if (first_gene) {
+            if (!is_multigene && first_gene) {
                 first_gene.click();
             }
 
@@ -94,7 +94,7 @@ document.addEventListener('DOMContentLoaded', () => {
             hideOrganismSelectorToolip();
             updateAnnotationDisplay();
 
-            // If the user is logged in and doesn't have a default org ID or it's different from their current, 
+            // If the user is logged in and doesn't have a default org ID or it's different from their current,
             //  show the control
             if (CURRENT_USER.session_id) {
                 if (CURRENT_USER.default_org_id === undefined || CURRENT_USER.default_org_id !== currently_selected_org_id) {

diff --git a/www/js/index.js b/www/js/index.js
@@ -37,11 +37,7 @@ document.addEventListener('DOMContentLoaded', () => {
             // get the value of the single-multi radio box
             const single_multi = document.querySelector('input[name="single-multi"]:checked').value;
 
-            if (single_multi === 'single') {
-                url += '&is_multigene=0';
-            } else {
-                url += '&is_multigene=1';
-            }
+            url += single_multi === 'single' ? '&is_multigene=0' : '&is_multigene=1';
 
             // add the gene lists
             //  TODO: This will only be for labeling purposes, since individual genes could have been