TUI with separation of input and output #34
Conversation
the-Jamz
left a comment
the-Jamz
left a comment
There was a problem hiding this comment.
Generally looks good, when I have a minute I'll check this across a few platforms to make sure everything is still working given the dependency updates. Thanks a lot for all the work you've done here!
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
Could improve readability if we reduce the nesting here somehow
There was a problem hiding this comment.
Agreed. I combined some conditions from nested if statements and moved event handling into a separate function to help with this.
|
Feel free to let me know if there is anything I can do to help move things forward. I understand that the dependency updates may require additional testing so if there is a way that I can help with that, I am happy to contribute to make the process smoother. |
2 participants
Resolves #10