Norfeldt / github-issue-reactions-browser-extension

List a link of reactions on a github issue page
MIT License
50 stars 10 forks source link

[Discussion] Color theme #22

Closed Hacksore closed 1 year ago

Hacksore commented 1 year ago

I love this extension very much as it helps me sift through big issues.

One thing I'd like to discuss is the use of @wesbos yellow which does not really fit well on github. Would you be opposed to using a dark/light theme color that matches github or allow the user to control them?

Quick mock up with slightly more on theme colors. image

Norfeldt commented 1 year ago

What about the bright colours from the reactions (👍, ❤️)?

Hacksore commented 1 year ago

Personally I've accepted that as the github/emoji standard style but the controls on the other hand should be consistent with github overall style.

purplespider commented 1 year ago

I agree with this, actually. I know it's a small thing, but it would be nice if the extension could better fit in with GitHub's styles. e.g. by inheriting and/or using their class names. Would make it feel less "clunky". e.g.

Before & After: Screenshot 2023-03-02 at 10 08 44@2xScreenshot 2023-03-02 at 10 25 20@2x Note:

wesbos commented 1 year ago

I agree (though I am flattered by Cobalt2) - it should use the github CSS variables.

var(--color-fg-default) --color-social-reaction-bg-hover

Norfeldt commented 1 year ago

You all (@Hacksore @purplespider @wesbos) have persuaded me. I just got a child so don't have much time to make this feature, but PRs are welcome 🤗

Hacksore commented 1 year ago

PR is up! Thanks @wesbos for getting the colors, true MVP.