android-proxy-toggle icon indicating copy to clipboard operation
android-proxy-toggle copied to clipboard

Move compose state to viewmodel

Open christos-christidis opened this issue 3 years ago • 1 comments

This PR moves the whole app to Compose without changing the logic (viewModel) and breaks up the app into modules, in the 1st commit. In the 2nd commit, composables are made state-less with all logic moved to the viewmodel.

Besides that, there's a lot of refactoring, updating of libs, fixing of errors, modernization of architecture etc.

christos-christidis avatar Oct 07 '22 08:10 christos-christidis

Screenshot are great but have you considered reducing their number a bit? E.g. your main screen screenshots include icons such as info, and there's a separate screenshot just for the info icon - does that bring value on its own?

Yeah I think we could remove stuff that is already included in main screen. And only let those stuff which is not, like error states of textfields etc..

christos-christidis avatar Oct 10 '22 11:10 christos-christidis