-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor App Architecture #30
Comments
@imrofayel was the app meant to be ran in the browser on any device? |
@kwe92 it's a desktop app primarily for Windows. |
It is a great app, refactoring could be step-by-step
|
@amrgetment Thank you for these suggestions. I wanna refactor this app since long but kinda busy in studies and job. But I'm willing to support any contribution. If you wanna collaborate on it, let me know. 🕊️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Would you be willing to let me refactor your apps architecture?
I would:
reduce the size of widgets (some are 1000 lines long)
remove state from views (turn all views stateless) and place presentation state in view models (using Provider package)
I can open a draft PR with some ideas I have and you can let me know if you like them.
The text was updated successfully, but these errors were encountered: