expressjs / expressjs.com

https://expressjs.com
Other
5.25k stars 1.46k forks source link

dark theme updated #1527

Closed iryanraushan closed 3 months ago

iryanraushan commented 3 months ago

update the dark theme color

netlify[bot] commented 3 months ago

Deploy Preview for expressjscom-preview ready!

Name Link
Latest commit 11eba849dd1911a70741149a7b641f0f0bbe8601
Latest deploy log https://app.netlify.com/sites/expressjscom-preview/deploys/6667c5e5d3c257000853db01
Deploy Preview https://deploy-preview-1527--expressjscom-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

chrisdel101 commented 3 months ago

Hi @iryanraushan. Our dark mode is far from perfect! But I'm not seeing a real need to "update the dark theme color." Using GitHub dark mode as a model, the range between colors should be very small, blending together almost like it's one color. Obviously there's no single "right way," but that's what we're going for.

I'm going to close this (@crandmck can re-open if you disagree :) )