-
Notifications
You must be signed in to change notification settings - Fork 65
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
Audio Player Improvements #294
Conversation
Great job! My quick feedback about the UI:
It takes a full ~9 seconds from the moment of tapping on a song to it actually audio playing. The entire app is frozen. I am in LAN mode for sure (I enforced it in the settings), so it's not a Remote access issue. Any ideas why this is? This appears to fix the crashing in audio in #289 but not the video crashing. We can fix that separately under another issue/PR but I just wanted to note it here. |
Two more things:
It's clear we need to focus and go step by step. |
Okay I'll check. Also can you provide me some more information about the "name going too far in the slider" issue? Can you share a screenshot? |
Where are we on this? |
I'm working on this. |
@cpg I've updated this PR and resolved the conflicts. Changes in this PR :
|
reverted spacing changes.
Removing apiconfig from the pull request
It's not quite there:
|
This reverts commit c60e330.
@cpg we also support repeat all and going to previous song functionality in the player. If we're removing the songs from the queue once they're played, then everytime user clicks previous button we'll have to add the exact previous song to queue again, this is a heavy task as compared to other approaches since we'll have to reload the table view everytime previous is clicked. Now the issue that you mentiontioned is a genuine one as not removing will make the user to scroll way down to next song. What we can do is automatically scroll the table view to the next song when the queue is presented. If the user wishes to go to previous we don't even have to update/reload the data, we just need to scroll to the next song. Also, if a user wants to re-listen to a track he can just drag down that song to appropriate order or just tap on it. |
@cpg you can test the changes now. The iPad rotation issue will be resolved in Gestures PR. |
This PR includes following features: