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

Notification and New Post Buttons wont change #192

Open
Pixielart opened this issue Jun 5, 2024 · 5 comments
Open

Notification and New Post Buttons wont change #192

Pixielart opened this issue Jun 5, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@Pixielart
Copy link

Platform

Win 11

Browser

Chrome

Addon version

1.1.0

Details

Tumblr has changed the code for the Notifs and Post buttons, palettes doesn't change their colour anymore

@Pixielart Pixielart added the bug Something isn't working label Jun 5, 2024
@morphinpink
Copy link

I've noticed this too :( I made this style for stylus to fix it:


.jGgIg .ML6ef { 
 background-color: var(--color-primary-link);
} 

.jGgIg .ML6ef:hover { 
 background-color: var(--color-primary-link);
} 

.jpu7k {
 background: var(--color-primary-link); 
}

I don't know if sharing it would help the bug get fixed faster but in the meantime that should work!

@marcustyphoon
Copy link
Collaborator

This will work in more places/be more resilient to changes:

:root {
  --chrome-ui: rgb(var(--deprecated-accent));
}

There are a bunch of colors this technique won't work on, though, such as the color applied when hovering over the post button. So it's not a complete solution and to make one we'll probably have to rethink some things.

@morphinpink
Copy link

This will work in more places/be more resilient to changes:

:root {
  --chrome-ui: rgb(var(--deprecated-accent));
}

There are a bunch of colors this technique won't work on, though, such as the color applied when hovering over the post button. So it's not a complete solution and to make one we'll probably have to rethink some things.

Would this work on Firefox?

@AprilSylph
Copy link
Owner

chrome in that code snippet does not refer to Google Chrome. It's just part of a variable name that Tumblr uses. So yes, it will.

@morphinpink
Copy link

chrome in that code snippet does not refer to Google Chrome. It's just part of a variable name that Tumblr uses. So yes, it will.

thank you! i'll use that then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants