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

feat(YouTube): Hide automatic search highlights/links #3764

Closed
3 tasks done
SodaWithoutSparkles opened this issue Oct 13, 2024 · 6 comments
Closed
3 tasks done

feat(YouTube): Hide automatic search highlights/links #3764

SodaWithoutSparkles opened this issue Oct 13, 2024 · 6 comments
Labels
Feature request Requesting a new feature that's not implemented yet

Comments

@SodaWithoutSparkles
Copy link
Contributor

Feature description

Removes the automatic search keyword detection (blue words) and the ability to tap on it.

Image

Motivation

  1. The highlight is arbitrary, some noun changes and others don't w/o obvious reasons.
  2. The term isnt always accurate. It may break the sentence in unexpected ways similar to a random punctuation inserted incorrectly
  3. It adds unnecessary lengths to the sentence with the 🔍 icon. This may break carefully formatted text. For example, people deliberately put spoilers at line 4 to make it appear only when you tap "read all", and the phrase "spoiler alert" at line 3. but this "feature" might break the spoiler alert by shifting it to line 4 and you now have an unexpected spoiler.
  4. It might interfere with the "double-tap to like comment" behavior

There are also posts on reddit on this issue:

Acknowledgements

  • I have checked all open and closed feature requests and this is not a duplicate
  • I have chosen an appropriate title.
  • All requested information has been provided properly.
@SodaWithoutSparkles SodaWithoutSparkles added the Feature request Requesting a new feature that's not implemented yet label Oct 13, 2024
@oSumAtrIX
Copy link
Member

oSumAtrIX commented Oct 13, 2024

Not sure what this issue is supposed to be, I already argued with you on Discord about the exact same points.

The highlight is not arbitrary. I don't know what you mean by noun changing. The sentence is unchanged. The length is hardly an issue. It sums up to the length of a word, and not a book. Such formatting is unintended at best. YouTube does not have a spoiler feature. I'd like to see how it "breaks" according to you. You can double tap everywhere else on the comment, it hardly prevents you from doing so.

The feature seems useful to look up keywords. If you got a good reason besides the ones we already talked about or if you got a good follow up, I'll reopen

@oSumAtrIX oSumAtrIX closed this as not planned Won't fix, can't repro, duplicate, stale Oct 13, 2024
@LisoUseInAIKyrios
Copy link
Contributor

To hide all comments where these appears, this custom filter works: comment_thread.eml$search_icon_

But it likely will be very difficult to show the comment and only removing the search links and search icons.

@oSumAtrIX
Copy link
Member

It is likely parsed to some model where you'd probably have to null some field that has that metadata. Hiding comments just for that would be arbitrary.

@KobeW50
Copy link
Contributor

KobeW50 commented Oct 14, 2024

But it likely will be very difficult to show the comment and only removing the search links and search icons.

inotia00 is working on a patch to remove the highlighted link while keeping the comment. He explains his implementation in this GitHub issue: inotia00/ReVanced_Extended#2435

@kasperbjerby
Copy link

kasperbjerby commented Nov 13, 2024

Not sure what this issue is supposed to be, I already argued with you on Discord about the exact same points.

The highlight is not arbitrary. I don't know what you mean by noun changing. The sentence is unchanged. The length is hardly an issue. It sums up to the length of a word, and not a book. Such formatting is unintended at best. YouTube does not have a spoiler feature. I'd like to see how it "breaks" according to you. You can double tap everywhere else on the comment, it hardly prevents you from doing so.

The feature seems useful to look up keywords. If you got a good reason besides the ones we already talked about or if you got a good follow up, I'll reopen

What a useless respons.. We are many that just find it annoying, we dont have to convince you it's a bad feature and all we want is a option to turn it off, who made you the gatekeeper of feature request that are and are not allowed? Can fully understand why people are choosing alternatives when this is the mindset you guys have in here

@oSumAtrIX
Copy link
Member

oSumAtrIX commented Nov 13, 2024

Since you ignored what I wrote without even discussing any of the points made, I'll cut this short.

we dont have to convince you

Image
https://github.com/ReVanced/revanced-patches/blob/main/CONTRIBUTING.md

Feel free to PR it.

@ReVanced ReVanced locked as resolved and limited conversation to collaborators Nov 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Feature request Requesting a new feature that's not implemented yet
Projects
None yet
Development

No branches or pull requests

5 participants