search

rob729 / Quiet_Hours

No more trouble to make silent mode ON/OFF manually every time.
3 stars 9 forks source link

Add Swipe Refresh Layout #71

Closed coder2699 closed 4 years ago

coder2699 commented 4 years ago

Describe the feature you'd like In the MainFragment.kt when a profile becomes active it does not appears unless user navigates to other screen or reopen it again. To counter this problem add a swipe refresh layout so that there's no need for the user to navigate somewhere and come back.

Screenshots

Would you like to work on the issue?

rob729 commented 4 years ago

IMO the Swipe to refresh approach is not good. As a developer, it should be our responsibility to show the latest data on the screen. That's why MVVM architecture with Livedata is being used so that the UI reacts to data changes. You should think about how you can use the same to fix the Current active profile issue.

coder2699 commented 4 years ago

@rob729 Yeah, I get your point. But right now I can't think of an optimal solution. If you feel adding swipe refresh layout is not good, then no problem.