Skip to content

Commit

Permalink
fix nextflow ring classes, change how filterable columns is set
Browse files Browse the repository at this point in the history
  • Loading branch information
ChrisHuynh333 committed Feb 13, 2025
1 parent 0a21a0d commit 1dcb6b2
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 13 deletions.
2 changes: 1 addition & 1 deletion app/components/nextflow_component.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
<%= workflow.text_field :name,
value: instance.present? ? instance.name : nil,
class:
"bg-slate-50 border border-slate-300 text-slate-900 text-sm rounded-lg focus:ring-blue-500 focus:border-blue-500 block w-full p-2.5 dark:bg-slate-700 dark:border-slate-600 dark:placeholder-slate-400 dark:text-white dark:focus:ring-blue-500 dark:focus:border-blue-500" %>
"bg-slate-50 border border-slate-300 text-slate-900 text-sm rounded-lg focus:ring-primary-500 focus:border-primary-500 block w-full p-2.5 dark:bg-slate-700 dark:border-slate-600 dark:placeholder-slate-400 dark:text-white dark:focus:ring-primary-500 dark:focus:border-primary-500" %>
</section>

<%= workflow.hidden_field :namespace_id, value: @namespace_id %>
Expand Down
15 changes: 3 additions & 12 deletions app/javascript/controllers/nextflow/samplesheet_controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -575,22 +575,13 @@ export default class extends Controller {
}, 50);
}

// check first sample entry for which columns are available for filter so we don't need to do it for each sample
// while filtering
// check samplesheet properties for sample and sample_name and add them as filterable if present
#setFilterableColumns() {
if (
this.#samplesheetAttributes[0]["samplesheet_params"].hasOwnProperty(
"sample",
)
) {
if (this.#samplesheetProperties.hasOwnProperty("sample")) {
this.#filterableColumns.push("sample");
}

if (
this.#samplesheetAttributes[0]["samplesheet_params"].hasOwnProperty(
"sample_name",
)
) {
if (this.#samplesheetProperties.hasOwnProperty("sample_name")) {
this.#filterableColumns.push("sample_name");
}
}
Expand Down

0 comments on commit 1dcb6b2

Please sign in to comment.