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

Clicking timestamp shows Toast "Playing in popup mode", even if the player is already in popup mode. #6662

Closed
4 tasks done
skyGtm opened this issue Jul 14, 2021 · 6 comments · Fixed by #7202
Closed
4 tasks done
Labels
bug Issue is related to a bug GUI Issue is related to the graphical user interface

Comments

@skyGtm
Copy link

skyGtm commented Jul 14, 2021

Checklist

Steps to reproduce the bug

Play any Youtube video that has timestamps in description
(I used this: https://www.youtube.com/watch?v=aleEl74qkXU)

  1. Go to video description
  2. Click on time-stamp.

Note: This issue was first mentioned in #5523 (comment)

Actual behavior

Here is a screen record.

abc.mp4

Expected behavior

It should show "Playing in popup mode" toast message only if video was not playing in Popup mode.

Screenshots/Screen recordings

Logs

Device info

  • Android version/Custom ROM version:
  • Device model:
@skyGtm skyGtm added the bug Issue is related to a bug label Jul 14, 2021
@TobiGr TobiGr added the good first issue Easy/simple issues perfect for newcomers to get involved in the project label Sep 4, 2021
@laksh-21
Copy link
Contributor

laksh-21 commented Oct 2, 2021

Hi @TobiGr I looked through the code and:

  1. To know what mode the player is in, PlayerType of the Player is queried using getPlayerType() here.
  2. NavigationHelper.playOnPopupPlayer() is the source of the Toast message here
    So, this function needs access to PlayerType of the Player to decide to show the Toast message or not, so I was wondering if passing the PlayerType through the hierarchy in which this function is called would be the best option for solving this issue.

Thanks :)

@triallax triallax added the GUI Issue is related to the graphical user interface label Oct 2, 2021
@vhouriet
Copy link
Contributor

vhouriet commented Oct 5, 2021

Hey @laksh-21, you can get the PlayerHolder instance with its getInstance() method. You then have access to the Player methods.

If you're not working on this issue anymore, I'd like to give it a try.

@laksh-21
Copy link
Contributor

laksh-21 commented Oct 5, 2021

@vhouriet I'm not working on it, I was just looking into the issue to see if I could solve it. You can work on it if you want but get it assigned to you first :)

@vhouriet
Copy link
Contributor

vhouriet commented Oct 5, 2021

@laksh-21 Ok thanks, how do I get it assigned to me ? Sorry, I'm pretty new to Github.

@laksh-21
Copy link
Contributor

laksh-21 commented Oct 5, 2021

@vhouriet just leave a comment here that you want to work on this issue. Maybe mention the author.
Also read the contribution guidelines.

@vhouriet
Copy link
Contributor

vhouriet commented Oct 5, 2021

I'd like to work on this issue.

vhouriet pushed a commit to vhouriet/NewPipe that referenced this issue Oct 5, 2021
Atemu pushed a commit to Atemu/NewPipe that referenced this issue Nov 1, 2021
@ShareASmile ShareASmile removed the good first issue Easy/simple issues perfect for newcomers to get involved in the project label Oct 19, 2024
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 GUI Issue is related to the graphical user interface
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants