Skip to content

Add options to hide pinned hype chat list and hype chat button#709

Merged
AnatoleAM merged 2 commits into
SevenTV:masterfrom
GODrums:master
Jun 23, 2023
Merged

Add options to hide pinned hype chat list and hype chat button#709
AnatoleAM merged 2 commits into
SevenTV:masterfrom
GODrums:master

Conversation

@GODrums
Copy link
Copy Markdown
Contributor

@GODrums GODrums commented Jun 23, 2023

Feature: 2 new options to hide elements of the new Twitch feature called "Hype Chats"

Purpose

Twitch just introduced the new feature "hype chats" (see: https://help.twitch.tv/s/article/hype-chat-faq).
This feature can be really annoying for casual viewers considering the bright colors and constant pin above the chat.
Therefore this PR aims to introduce options to remove most of the hype chat features.
However, this change does not affect the solid boxes of "hype chat"-messages in the chat.

Implementation

Both options disable the respective site elements via their unique class or attribute.
For the "Hype Chat"-button, the attribute "aria-label" is used as unique identification.
For the "Pinned list of hype chats", the element's class name proves unique.

Both new options have been tested in various combinations with other existing options and work flawlessly.

The unchanged features on Twitch

The middle button (lightning icon) is the new "Hype Chat" button:
Screenshot_486
An example of a pinned "paid" hype chat:
Screenshot_485

With the proposed options enabled (elements hidden)

"Hide Hype Chat Button"-toggle enabled:
Screenshot_488
"Hide Pinned Hype Chats"-toggle enabled:
Screenshot_487

The options in the settings popup

Screenshot_489
Screenshot_490

Note

As this is my first time contributing to this extension/repo, please pardon any contributing errors of mine. I have used this extension for quite some time and would now love to help in its development.
Feel free to adapt my changes to your desires or decline my PR completely.

Copy link
Copy Markdown
Contributor

@AnatoleAM AnatoleAM left a comment

Choose a reason for hiding this comment

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

LGTM

@AnatoleAM AnatoleAM merged commit 1cdf48b into SevenTV:master Jun 23, 2023
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.

2 participants