Skip to content
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

Removing from playlist doesn't update UI correctly #910

Open
vrih opened this issue Nov 13, 2018 · 1 comment · May be fixed by #1126
Open

Removing from playlist doesn't update UI correctly #910

vrih opened this issue Nov 13, 2018 · 1 comment · May be fixed by #1126

Comments

@vrih
Copy link
Contributor

vrih commented Nov 13, 2018

When removing tracks from the playlist, the correct tracks are removed but the UI is not updated correctly. Half the selected tracks remain in the UI.

I suspect there's an issue of deleting items from a list while iterating through that list, throwing the index off.

To reproduce, have a playlist of about 10 tracks. Select the middle 5 and remove from playlist.

@vrih
Copy link
Contributor Author

vrih commented Nov 18, 2018

vrih added a commit to vrih/XSub that referenced this issue Nov 18, 2018
vrih added a commit to vrih/XSub that referenced this issue Dec 9, 2018
MoritzMaxeiner pushed a commit to MoritzMaxeiner/DStream that referenced this issue Sep 20, 2020
MoritzMaxeiner pushed a commit to MoritzMaxeiner/DStream that referenced this issue Sep 20, 2020
MoritzMaxeiner pushed a commit to MoritzMaxeiner/DStream that referenced this issue Sep 21, 2020
MoritzMaxeiner pushed a commit to MoritzMaxeiner/DStream that referenced this issue Sep 21, 2020
MoritzMaxeiner pushed a commit to MoritzMaxeiner/DStream that referenced this issue Sep 22, 2020
MoritzMaxeiner pushed a commit to MoritzMaxeiner/DStream that referenced this issue Sep 25, 2020
MoritzMaxeiner pushed a commit to MoritzMaxeiner/DStream that referenced this issue Sep 26, 2020
paroj pushed a commit to paroj/DSub2000 that referenced this issue Sep 6, 2022
@paroj paroj linked a pull request Sep 6, 2022 that will close this issue
paroj pushed a commit to paroj/DSub2000 that referenced this issue Jun 8, 2023
paroj pushed a commit to paroj/DSub2000 that referenced this issue Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant