github-dark icon indicating copy to clipboard operation
github-dark copied to clipboard

Fix read notifications

Open seanysean opened this issue 4 years ago • 5 comments

What's the issue:

When clicking on a notification or viewing read notifications (https://github.com/notifications/read), they appear bright.

What's fixed:

Fixes the above issue by changing the background of the notification items to be darker. I used #999 for the text color because secondary-text-color was too dark.

Screenshots:

Before: image After: image

seanysean avatar Oct 17 '19 19:10 seanysean

Hmm, not completely a fan of the colors, but I'm unable to find any more suitable ones right now. You're right that the --secondary-text-color is too dark. But maybe we should instead change the background color of the items? Maybe use --depth-color instead?

snowe2010 avatar Oct 23 '19 21:10 snowe2010

I updated the background color as suggested, let me know if there is anything more you would like for me to change.

seanysean avatar Oct 23 '19 22:10 seanysean

Well I was actually asking for feedback 😅 Do you think that's a better color?

snowe2010 avatar Oct 23 '19 23:10 snowe2010

Ah, my mistake. I originally chose --accent-color because it is the same color as the bar that displays the repo name (which is how the github default style does it as well as shown in screenshot), but I think either color would work perfectly fine. image

seanysean avatar Oct 23 '19 23:10 seanysean

ah, that's a good point about the default... hmm.... Let me think about this for a bit.

snowe2010 avatar Oct 24 '19 15:10 snowe2010