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

Icon only mode + making Blue icon less noticeable #16

Closed
wants to merge 1 commit into from

Conversation

martin-bloksa
Copy link

I changed the code of script.js to:

  • Remove the text and border next to the badges
  • Make both badges approx. the same size
  • Reduce the visual importance of Blue badge and make it less noticeable than the Verified badge

I changed the code of script.js to:

- Remove the text and border next to the badges
- Make both badges the same size
- Reduce the visual importance of Blue badge and make it less noticeable than the Verified badge
@martin-bloksa
Copy link
Author

Tested on 2 Windows computers, Chrome, Dark mode only.

@CJStyles123
Copy link

I love this, is there any way to have this affect the notifications tab too?

@martin-bloksa
Copy link
Author

I love this, is there any way to have this affect the notifications tab too?

I only did the "reskin", it seems like the extension code itself doesn't work in the "Verified" notifications tab. So that would be a question for @wseagar

@wseagar
Copy link
Owner

wseagar commented Nov 11, 2022

Hey thanks for this but we're going to make these options configurable. Can you have a look at master I've put in the plumbing for an options panel and have added my own version of disabling the text. Feel free to work this change into the new options system if you can find a place!

@wseagar
Copy link
Owner

wseagar commented Nov 13, 2022

These changes have been included with the options panel and the PR is now out of date. Thanks for contributing though!

@wseagar wseagar closed this Nov 13, 2022
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.

None yet

3 participants