Practice-Looper / practice-looper

The Practice Looper app puts an end to the constant rewinding to loop only parts of your favourite songs from different audio sources - including songs from your Spotify Premium account.
https://www.practice-looper.com
0 stars 0 forks source link

Issue#147 pull to refresh #150

Closed simonsymhoven closed 3 years ago

simonsymhoven commented 3 years ago

Hab zuerst mal den fehlenden Dialog noch eingebaut, passt das so vom Text her?

Xamarin Forms liefert selbst bereits eine RefreshView, habe also diese verwendet und das Einbinden eines extra Nuggets vermieden. Nach dem ich das implementiert hatte, ist mir aufgefallen, dass Zurück/Vorwärts auch irgendwie geil wäre, falls man sich verklickt hat. Denke das macht Sinn?

IMG_8701

Da die Refresh View im Xaml Code Behind erzeugt wird und dem Grid hinzugefügt wird und nur hier Zugriff auf die spotifyWebView existiert, war ich mir nicht ganz sicher wie man das in Xamarin/C# am dümmsten umsetzt. Am Anfang habe ich den entsprechenden Command einfach im Code Behind definiert und da an die RefreshView gehängt, was irgendwie nicht so ganz im Sinne von MVVM ist.. Habe also in den beiden Commands der neuen Navigation-Buttons und dem PullToRefresh Command in dem MainViewModel über das MessagingCenter einfach eine Notification rausgehauen und fange die im Code Behind ab um dann Reload(), GoBack() oder GoForward() auf der spotifyWebView aufzurufen. Haut das hin oder gibt's da einen schöneren Weg?