search

lawrencehook / remove-youtube-suggestions

A browser extension that removes YouTube suggestions, comments, shorts, and more
https://lawrencehook.com/rys/
Mozilla Public License 2.0
401 stars 33 forks source link

Hide all shorts #57

Closed borderline232 closed 1 year ago

borderline232 commented 2 years ago

If you could implement a feature in which youtube shorts videos are redirected to the normal youtube player that would be awesome.

The workflow for it is pretty simple:

Shorts video URL = youtube.com/shorts/<video-id>

Normal video URL = youtube.com/watch?v=<video-id>

You just need to mutate the shorts video URL to the normal one and it should work out. Thanks again.

remi-garcia commented 2 years ago

Hiding shorts would also be a nice feature

lawrencehook commented 2 years ago

image

@borderline232 Already implemented! There's an option called Redirect shorts in the "video player" section.

lawrencehook commented 2 years ago

@remi-garcia Good idea. I'll add it to my list. There's an existing option for hiding shorts from search results, so hiding them everywhere should be a straightforward addition.

0xatulpatil commented 1 year ago

Hey Lawrence, can I take up this issue? As most of the part is already implemented, this would be an easy first issue for me.

lawrencehook commented 1 year ago

@0xatulpatil Certainly!

0xatulpatil commented 1 year ago

Great! How do you want me to go about it?

All the short badges are marked using appropriate attributes on the "Results" and "Subscription" page. Would need to mark them on the homepage as well. Using CSS selectors we can hide them all at once.

Will this be a good starting point? Any other suggestions?

lawrencehook commented 1 year ago

That's a great starting point. I believe there's also shorts on the sidebar.

Sounds like you have a good understanding of how the code works.

There's a question of which section to put the new option under, but that can wait.

0xatulpatil commented 1 year ago

I think this feature does not belong to any specific section that currently exists on our options menu, so "Basic" would be the best place to put it.

Also, what should I name it? "Hide all shorts" , "Hide Shorts everywhere" or anything else?

lawrencehook commented 1 year ago

Good idea and "Hide Shorts everywhere" sounds good to me

0xatulpatil commented 1 year ago

73

@lawrencehook made a PR for the same, let me know if there are any changes to me made

lawrencehook commented 1 year ago

@0xatulpatil Awesome, I'll do some testing and get back to you. Will probably take a couple days.

lawrencehook commented 1 year ago

@0xatulpatil Merged, thanks!

I added some updates to handle shorts that appeared in the sidebar.

0c6cec355a84d9dcd6ed60d16e6898fb0922cc0b