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

Make the mute button faster accessible for pop-up and background player #3275

Closed
Poolitzer opened this issue Mar 25, 2020 · 14 comments
Closed
Labels
feature request Issue is related to a feature in the app GUI Issue is related to the graphical user interface

Comments

@Poolitzer
Copy link
Member

Poolitzer commented Mar 25, 2020

this is related to #3165.

Pop-up Player:

Currently, to mute a video playing in pop-up, I need to scroll down my notifications, press on the NewPipe one, then it's in the upper corner. If the pop-up window is big enough though, there is space after the captions string. Someone could use that to implement the mute button.

screenshot of the pop-up player

I hope the red arrow makes clear what I mean.

Background Player:

Currently, to mute a video which is playing in the Background, I need to scroll down my notifications, press on the NewPipe one, then it's in the upper right corner. This process can be simplified if we add the mute button between the repeat and the back control directly in the notification.

screenshot of the pop-up player

I hope the red arrow makes clear what I mean.

@Poolitzer Poolitzer added the feature request Issue is related to a feature in the app label Mar 25, 2020
@Poolitzer Poolitzer changed the title Add the mute button after the caption text in the pop-up player if its big enough Make the mute button faster accessible for pop-up and background player Mar 25, 2020
@Stypox
Copy link
Member

Stypox commented Mar 26, 2020

I don't think this should be done. In the popup player there are already too many icons, and when it is not full-sized it looks strange. In the background notification there is almost not enough space for the repeat button on phones with lower dpis. Also, the background notification will be replaced soon.
@TobiGr what is your opinion? I don't think this should be implemented, also because the mute button is not used by so many people

@Poolitzer
Copy link
Member Author

Both of your issues can be countered with "only display the button if enough place is there". At least for the pop up window this works, not sure about the notification.

@Stypox
Copy link
Member

Stypox commented Mar 26, 2020

Yeah but there are more important buttons that could be fit instead

@wb9688
Copy link
Contributor

wb9688 commented Mar 26, 2020

About putting that in the notification: not possible after #3178

@B0pol
Copy link
Member

B0pol commented Mar 26, 2020

About putting that in the notification: not possible after #3178

no, that's false. It will even be better, because you can choose in settings which one to choose. Now there are more important buttons that could be in notifications, but if user can choose, there is no problem to add it via settings.

@opusforlife2
Copy link
Collaborator

What's the use of having a mute button for background audio, anyway? The PR linked issue just asked for MX Player like functionality, which I can understand as it's for video. For example, you could watch a math problem being solved while another app is playing music. What about audio, though?

@Poolitzer
Copy link
Member Author

I am not sure, I just wanted to provide feedback for a way to easier access it :)

@opusforlife2
Copy link
Collaborator

Oh, I am asking everyone for their opinion. Just seems like a pointless addition to the app. I thought for a moment that it might be useful to mute when sponsored sections start, but then you have the much better option of directly skipping them. Can anyone think of a use for the mute button in the background player?

@Poolitzer
Copy link
Member Author

The idea of adding it in general? I guess you are a bit too late for that?

@opusforlife2
Copy link
Collaborator

The idea of adding it in general? I guess you are a bit too late for that?

NUKE THE CODE, I SAY! MWAHAHAHAHAH!

@Poolitzer
Copy link
Member Author

Oh okay. Uhm sure. Let me just see where I put the code nukes...

Steps away

@opusforlife2
Copy link
Collaborator

sinister giggles

@Stypox Stypox added the GUI Issue is related to the graphical user interface label Mar 30, 2020
@opusforlife2
Copy link
Collaborator

Yes or no?

@Stypox
Copy link
Member

Stypox commented Sep 2, 2020

This was chosen this way since the mute button is a niche feature. So it does not deserve a exclusive space

@Stypox Stypox closed this as completed Sep 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue is related to a feature in the app GUI Issue is related to the graphical user interface
Projects
None yet
Development

No branches or pull requests

5 participants