Skip to content

Conversation

@eeelie
Copy link
Collaborator

@eeelie eeelie commented Apr 26, 2023

  • refactor: moving old tui files to new directory
  • refactor: reverting Header and Footer to textual versions
  • WIP: Working browser with placeholder plots
  • Worked on quit w/Dump
  • fix: Minor cleanup
  • feat: Added Content Switcher to plot view
  • style: now using css to style browser.py
  • feat: added TreeWidget
  • WIP: working on implementing tree view
  • Tree and Panel/Error communication working
  • feat: light/dark mode
  • WIP: quit with dump
  • fix: quit with dump
  • fix: include error in quit with dump
  • style: cleaning up browser.css

need to do some fixes to pass ruff and mypy pre-commit hooks

@eeelie eeelie requested a review from henryiii April 26, 2023 20:15
@henryiii henryiii force-pushed the elie/feat/update-textual branch from 029d484 to 09f639f Compare April 26, 2023 21:16
@henryiii
Copy link
Member

Looks like the nesting issue was (at least partially) fixed on main, but this branch was based on an old commit. Rebased, added typing and made Ruff happy, and did some minor cleanup.

henryiii and others added 2 commits April 26, 2023 17:28
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
@henryiii henryiii force-pushed the elie/feat/update-textual branch from 4d0a347 to 752087e Compare April 26, 2023 21:46
Copy link
Member

@henryiii henryiii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks good to go!

@henryiii
Copy link
Member

I'm going to merge this. Future changes should be new PRs focus on single features / fixes (like tabs or keyboard navigation). We can discuss releasing when we meet tomorrow.

@henryiii henryiii merged commit d04f36b into main Apr 26, 2023
@henryiii henryiii deleted the elie/feat/update-textual branch April 26, 2023 22:13
@henryiii
Copy link
Member

We should probably update the gif and logo in the readme around release time, too. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants