-
-
Notifications
You must be signed in to change notification settings - Fork 40
Fix UI performance #269
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
base: development
Are you sure you want to change the base?
Fix UI performance #269
Conversation
7775d13 to
d868f70
Compare
app/src/main/java/com/cappielloantonio/tempo/util/DownloadUtil.java
Outdated
Show resolved
Hide resolved
d868f70 to
efb3d63
Compare
|
Something that I've noticed switching from tempo to tempus, is that the track will start playing well before the UI updates. Is this related? It sounds like a small thing (and I suppose it is), but what happens is that a new song is playing, I go "what the hell is this rubbish" look at the song and it's one of my favourite artists. Then the screen updates and I realise it's a different song. I'd say it's only a 2ish second delay, but it's irritating. |
I don't recall if I seen that happen however that is pretty annoying. I am not sure if it is related but it could be. It would be good to see if we could recreate that scenario but is that something you can consistently recreate ? |
|
Unfortunately broke my phone, but when I get a replacement I'll see if it's easily recreated. |
SearchFragment, which removes the need forNestedScrollView.RecyclerViewforArtistPageFragment, which removes the need forNestedScrollView.For 3, my server does not have fancy features such as related artists. So need someone else to test that.