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

"Preferred 'Open' Action" setting should be applicable for videos only #6371

Closed
4 tasks done
SameenAhnaf opened this issue May 25, 2021 · 0 comments · Fixed by #6385
Closed
4 tasks done

"Preferred 'Open' Action" setting should be applicable for videos only #6371

SameenAhnaf opened this issue May 25, 2021 · 0 comments · Fixed by #6385
Labels
bug Issue is related to a bug

Comments

@SameenAhnaf
Copy link
Collaborator

SameenAhnaf commented May 25, 2021

Checklist

Steps to reproduce the bug

It seems like that shared URL action doesn't always work the way as expected. In reality, people share videos, channels and playlists for different reasons. However, playback options are actually quite relevant for videos.

But that's not always the case with channels or playlists. Many users share directly by long press on link without opening the link. Checkback is a necessary feature and that's one of the key reasons why long press menu was not added for unsubscribed channels and playlists. That's why, info should always be shown.

One more thing, "Video Player" and "Show Info" works exactly the same for videos. So, having both options makes no real sense.

Actual behaviour

"Preferred 'Open' Action" setting should be applicable for videos only. "Download" option ends up giving us no output for channels and playlists.

Expected behavior

Info should always be shown for channels and playlists.

Screenshots/Screen recordings

Logs

Device info

  • Android version/Custom ROM version:
  • Device model:
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant