Skip to content

Commit

Permalink
(refactor)ui: Refactoring/simplifying TUI state (#8650)
Browse files Browse the repository at this point in the history
### Description

Previously, we held the state for the TUI in the sub-components of
`app.rs`. In this PR, we're hoisting those state variables up so we can
have better access to that shared state and create more robust UI.

### Testing Instructions

There two behavioral changes in this PR: 

- Reordering of the task list so that running tasks are first in the
list, then planned tasks, then done tasks.
- We are removing the spinner in favor of the "two arrow" indicator.
Users mentioned that the visual noise was more annoying than helpful, so
let's quiet it down.

Beyond that, you should see no behavioral changes.

---------

Co-authored-by: Chris Olszewski <chris.olszewski@vercel.com>
  • Loading branch information
anthonyshew and chris-olszewski authored Jul 3, 2024
1 parent 2503bb6 commit 43473b4
Show file tree
Hide file tree
Showing 9 changed files with 700 additions and 880 deletions.
124 changes: 0 additions & 124 deletions crates/turborepo-ui/examples/pane.rs

This file was deleted.

131 changes: 0 additions & 131 deletions crates/turborepo-ui/examples/table.rs

This file was deleted.

Loading

0 comments on commit 43473b4

Please sign in to comment.