You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The CLI is meant to be a thin wrapper around the rest of the project. It currently has way too many non-reusable presentation concerns factored into it.
Refactor as many of them as possible to interact only with a presenter, model, and view instances. They generally should not interact with services directly.
The presenters are to use flows to support modern reactive UIs.
The text was updated successfully, but these errors were encountered:
The render and render-text modules are becoming really Presentation layer concerned. Time for a rename.
The PropertyRenderer and ViewRenderer can be renamed for simplicity to Adapter and View, respectively.
carltonwhitehead
changed the title
Consolidate the CLI view package into the render-text package
Refactor the CLI to use a presentation layer
Aug 4, 2023
The surface area of this work is huge. It affects practically every command already written. That's going to be a lot of repetitive, boring work to do, hard to gauge what is needed for other UIs, creating uncertainty around the goalposts for this ticket. This project is supposed to be fun to work on, not a soul-suck. Therefore I'm not going to do this all at once. Instead, I will do these one-by-one as opportunities for code reuse arise when developing UIs.
If anyone wants to dip a toe into this project and help out in a meaningful way, add a comment here and/or reach me through another channel, and I'll be happy to go over the steps to do a small chunk of this.
The CLI is meant to be a thin wrapper around the rest of the project. It currently has way too many non-reusable presentation concerns factored into it.
Refactor as many of them as possible to interact only with a presenter, model, and view instances. They generally should not interact with services directly.
The presenters are to use flows to support modern reactive UIs.
The text was updated successfully, but these errors were encountered: