Closed joshka closed 5 months ago
I think it is worth merging this at this point and continuing a refactor if needed in a separate PR. It'll be easier to track changes that way.
I think it is worth merging this at this point and continuing a refactor if needed in a separate PR. It'll be easier to track changes that way.
Sounds good. I'll start a round 2 PR on it later though as I think this can still be simplified a lot.
We should leave the command ignore in there so users can override defaults
The purpose of this PR is to gradually refactor the app module to split the search and summary functionality better. Right now app is a module that has a bit of everything.
Don't merge this, I'll update this as I'm working through the refactor.
This PR refactors app.rs and adds a permanent statusbar.