-
-
Notifications
You must be signed in to change notification settings - Fork 32
fix: Settings button moved to YouTube Chat menu #196
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
base: mv2
Are you sure you want to change the base?
Conversation
r2dev2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionality is great and works!
However, I think there might be some rendering issues. On Chrome, it looked fine, but at least in my Firefox, the text looks weird (as seen below). Would you be able to fix this?
Also, would you be able to add a hover effect for when the mouse hovers over the button? See the comment I had for the :hover rule, I think the color just needs to be tweaked and then it'll work!
r2dev2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionality good, just some quick css + logging changes!
r2dev2
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.

No description provided.