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

[Unified player] Selecting another video from streams tab doesn't update playlist dialog if kept opened #4253

Closed
vkay94 opened this issue Sep 8, 2020 · 2 comments · Fixed by #4272
Labels
bug Issue is related to a bug player Issues related to any player (main, popup and background)

Comments

@vkay94
Copy link
Contributor

vkay94 commented Sep 8, 2020

Version

  • Unified player test apk: a801d09 (2020-09-06)

Steps to reproduce the bug

  1. Open a video stream and turn on Autoplay switch
  2. Open the playqueue/playlist within the player (next to Playback Speed Controls)
  3. Start another stream in the "Next streams"-tab

Expected behavior

One of these depending on the head dev's preference (I guess):

  • In the dialog the chosen (and next stream if autoplay/autoqueue is still enabled) should change accordingly to the current played stream
  • When the other stream has been chosen the dialog should be dismissed and not opened again

Actual behaviour

When the other stream has been shosen the playlist dialog is dismissed but opened again with the same videos (currently playing video =/= selected video in the playlist/queue + they can't be selected). When you close it and open in again the items are correct again.

Does it appear on v0.19.8?

No. In v0.19.8 you can't choose another stream from the tab while the playlist dialog is opened.

Video preview

https://streamable.com/kcm6p5

@vkay94 vkay94 added the bug Issue is related to a bug label Sep 8, 2020
@opusforlife2 opusforlife2 added the player Issues related to any player (main, popup and background) label Sep 8, 2020
@opusforlife2 opusforlife2 added this to To Do in Unified Player regressions and bugs via automation Sep 8, 2020
@avently
Copy link
Contributor

avently commented Sep 8, 2020

Video costs a thousand of words

@avently
Copy link
Contributor

avently commented Sep 11, 2020

see #4272

@B0pol B0pol moved this from To Do to In Progress in Unified Player regressions and bugs Sep 19, 2020
@opusforlife2 opusforlife2 moved this from In Progress to To Do in Unified Player regressions and bugs Sep 19, 2020
@opusforlife2 opusforlife2 moved this from To Do to In Progress in Unified Player regressions and bugs Sep 19, 2020
Unified Player regressions and bugs automation moved this from In Progress to Done Sep 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is related to a bug player Issues related to any player (main, popup and background)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants