Merge pull request #902 from opencb/TASK-6014

TASK-6014 - Sample Genotypes columns are hidden in SVB in some circunstancies
opencb · Apr 16, 2024 · d51f579 · d51f579
2 parents 9e1e14a + 8c1e2e6
commit d51f579
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/src/webcomponents/variant/interpretation/variant-interpreter-grid.js b/src/webcomponents/variant/interpretation/variant-interpreter-grid.js
@@ -562,7 +562,8 @@ export default class VariantInterpreterGrid extends LitElement {
                                     colspan: 1,
                                     formatter: this.vcfDataFormatter,
                                     halign: this.displayConfigDefault.header.horizontalAlign,
-                                    visible: this.gridCommons.isColumnVisible(columnId, "VCF_Data"),
+                                    excludeFromSettings: true,
+                                    visible: !this._config.hideVcfFileData,
                                 });
                             }
                         }
@@ -1082,7 +1083,8 @@ export default class VariantInterpreterGrid extends LitElement {
                         formatter: VariantInterpreterGridFormatter.sampleGenotypeFormatter,
                         align: "center",
                         nucleotideGenotype: true,
-                        visible: this.gridCommons.isColumnVisible(samples[i].id, "sampleGenotypes"),
+                        excludeFromSettings: true,
+                        visible: !this._config.hideSampleGenotypes,
                     });
                 }
             }
@@ -1131,7 +1133,8 @@ export default class VariantInterpreterGrid extends LitElement {
                         formatter: VariantInterpreterGridFormatter.sampleGenotypeFormatter,
                         align: "center",
                         nucleotideGenotype: true,
-                        visible: this.gridCommons.isColumnVisible(samples[i].id, "sampleGenotypes"),
+                        excludeFromSettings: true,
+                        visible: !this._config.hideSampleGenotypes,
                     });
                 }
             }
@@ -1549,6 +1552,8 @@ export default class VariantInterpreterGrid extends LitElement {
             hidePopulationFrequencies: false,
             hideClinicalInfo: false,
             hideDeleteriousness: false,
+            hideSampleGenotypes: false,
+            hideVcfFileData: false,
 
             quality: {
                 qual: 30,

diff --git a/src/webcomponents/variant/interpretation/variant-interpreter-rearrangement-grid.js b/src/webcomponents/variant/interpretation/variant-interpreter-rearrangement-grid.js
@@ -544,7 +544,8 @@ export default class VariantInterpreterRearrangementGrid extends LitElement {
                                 colspan: 1,
                                 formatter: (value, row) => this.vcfDataFormatter(value, row[index], field),
                                 halign: "center",
-                                visible: this.gridCommons.isColumnVisible(id),
+                                excludeFromSettings: true,
+                                visible: !this._config.hideVcfFileData,
                             });
                         });
                     });
@@ -1104,6 +1105,7 @@ export default class VariantInterpreterRearrangementGrid extends LitElement {
             showActions: false,
             multiSelection: false,
             nucleotideGenotype: true,
+            hideVcfFileData: false,
 
             alleleStringLengthMax: 50,