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

Add option to hide youtube logo #56

Closed
wants to merge 1 commit into from
Closed

Conversation

DavG
Copy link

@DavG DavG commented Aug 10, 2022

Hi.

My LG C7 is starting to have some burning, because of this (stupid) YouTube logo in pure red, so I've searched a solution.
I've found this ticket on the parent repository : FriedChickenButt#30

I followed the instructions and basically copy/paste the code.
Thanks @ksala and @Informatic :)

I also added the checkbox in the option modal, and tagged the version as 0.4.0

I've tested the feature and it works.
When you check/uncheck, the logo don't instanly appear/disappear, you have for example to go to the left menu for the change to apply. That's all :)

Hope it will helps other.

Regards.

Copy link
Member

@Informatic Informatic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I'll merge it in before next release.

@nurikk
Copy link

nurikk commented Jan 2, 2023

Looks good to me. I'll merge it in before next release.

Hi, @Informatic! There’s not much development happening lately in this repo. Maybe it’s worth releasing a small release with this feature?

@Kneit0r
Copy link

Kneit0r commented Mar 15, 2023

Any news on this?

@throwaway96
Copy link
Member

Has this been tested?

If the 0.4.0 stuff (all changes to *.json basically) are removed, I'll merge it. The changelog entry can stay, but it should just be under "unreleased".

@tomikaka22
Copy link

In case anyone is interested, me and the creator of youtube-webos-cobalt modified the code a while ago to work without the DOMNodeInserted event.

tomikaka22@c2a7438

I can open a pull request if needed.

@throwaway96
Copy link
Member

@tomikaka22

I'd appreciate a PR for that version, and then this one can be closed.

@tomikaka22
Copy link

@throwaway96

Done! Here it is: #135

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants