Calculate spinner and hide next button

[Jonas0000]

Description

The calculate button shows a loading spinner while calculating. The next button at the last step is now hidden.

Changes

ui/runs/static/runs/runs.js
ui/runs/templates/runs/details.html

Testing

A walkthrough with steps of a sample workflow.

PR checklist

Development

• If necessary, I have updated the documentation (README, docstrings, etc.)
• If necessary, I have created / updated tests.

Mergeability

• main-branch has been merged into local branch to resolve conflicts
• The tests and linter have passed AFTER local merge
• The code has been formatted with black

Code review

• I have self-reviewed my code.
• At least one other developer reviewed and approved the changes

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
protzilla
run.py
run_helper.py
runner.py
protzilla/data_analysis
differential_expression_anova.py
differential_expression_linear_model.py
differential_expression_mann_whitney.py					263
differential_expression_t_test.py
protein_graphs.py
ptm_analysis.py					115-116, 134-136
protzilla/data_integration
di_plots.py
enrichment_analysis_gsea.py
protzilla/data_preprocessing
filter_proteins.py
outlier_detection.py
peptide_filter.py
transformation.py
protzilla/importing
ms_data_import.py					122, 276
protzilla/methods
data_analysis.py
data_preprocessing.py
importing.py					142
protzilla/utilities
transform_dfs.py
utilities.py
ui/main
settings.py
ui/runs
fields.py
views.py
views_helper.py
Project Total

The report is truncated to 25 files out of 51. To see the full report, please visit the workflow summary page.

This report was generated by python-coverage-comment-action

[noLeonardo]

The button looks good and the code as well. Everything works :)

[maximilianKalff]

The spinning button looks amazing and works as expected. The code is understandable and well written :))

[sarahvgls]

Find ich sonst richtig gut und passt schön in den bisherigen Code rein :)

[sarahvgls]

ein kleine Frage: Ist Calculate hier einfach der Value des Buttons bevor er gedrückt wird und nur eine andere Schreibweise als vorher?

[Jonas0000]

Ja, genau. Calculate ist hier nur der Text, der auf dem Button angezeigt wird.
Wenn man zusätzlich noch name und value angeben würde, würden die Werte als name=value mit dem Form übermittelt werden. Vorher beim Input war der Unterscheid, dass ein Input kein Start und End-Tag hat, sondern nur aus einem Tag besteht, sodass hier der Text nicht zwischen die Tags geschrieben werden kann. Bei diesem Wird dann value als Anzeigetext verwendet.
Wenn dich das noch genauer interessiert, ist diese Website ganz gut zum herumspielen: https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_input_test

[sarahvgls]

Okay danke! Das schaue ich mir mal an, habe damit bisher noch nicht so viel Kontakt gehabt!

[sarahvgls]

Am besten dann mit Squash und merge auf dev tun, dann ist es dort etwas ordentlicher :)