Skip to content

Use trame-facade

Duggan, John requested to merge 13-use-trame-facade-sans into main

The main goal here is to get trame-facade into this application. In doing so, I've had to heavily change a few of the tabs in order to make things work well with the theme. If you don't like the way anything looks, this is the first chance to get some real feedback on the theming that I'm trying to set up, so I'll be more than happy to take feedback and update the theme based on it.

In addition to the theming, I've made the following more functional changes to the application:

  1. Gave the application a proper page title.
  2. I am hiding execution buttons rather than disabling them when they can't be used. I've also added visual separation between the configuration and running buttons.
  3. I've added a callback to run validation on the whole configuration when someone updates the "Sample Names" field. I'm thinking this probably needs to be removed before merge and needs to be worked on in a separate issue/branch, but I wanted to include it for now as a demonstration of one option for this problem.

Closes #13 (closed)

Edited by Duggan, John

Merge request reports