remove-youtube-suggestions icon indicating copy to clipboard operation
remove-youtube-suggestions copied to clipboard

Hide all shorts

Open borderline232 opened this issue 2 years ago • 3 comments

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.

borderline232 avatar Jun 17 '22 22:06 borderline232

Hiding shorts would also be a nice feature

remi-garcia avatar Jun 18 '22 06:06 remi-garcia

image

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

lawrencehook avatar Jun 18 '22 13:06 lawrencehook

@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.

lawrencehook avatar Jun 18 '22 13:06 lawrencehook

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.

0xatulpatil avatar Jan 12 '23 16:01 0xatulpatil

@0xatulpatil Certainly!

lawrencehook avatar Jan 12 '23 17:01 lawrencehook

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?

0xatulpatil avatar Jan 12 '23 18:01 0xatulpatil

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.

lawrencehook avatar Jan 12 '23 19:01 lawrencehook

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?

0xatulpatil avatar Jan 12 '23 19:01 0xatulpatil

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

lawrencehook avatar Jan 12 '23 20:01 lawrencehook

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

0xatulpatil avatar Jan 13 '23 17:01 0xatulpatil

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

lawrencehook avatar Jan 13 '23 19:01 lawrencehook

@0xatulpatil Merged, thanks!

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

0c6cec355a84d9dcd6ed60d16e6898fb0922cc0b

lawrencehook avatar Jan 16 '23 04:01 lawrencehook