open-sauced / hot

πŸ•The site that recommends the hottest projects on GitHub.
https://hot.opensauced.pizza
MIT License
426 stars 148 forks source link

style fix: Making the gradient text classes more intuitive would be nice: #266

Closed bdougie closed 1 month ago

bdougie commented 2 years ago

Making the gradient text classes more intuitive would be nice:

However, this is the wrong approach as we do not have the colors available as (actually) re-usable classes, we should create the gradient based on existing theme colors, rather than create duplicate color classes subsets for the gradients.

Visually, our start, middle and end color colors are not perceived by our users, it is unlikely the end and start color stops need additional hex colors other than the ones we have. I'm keen to stop introducing unwanted things we then have to open new fix PRs for.

_Originally posted by @0-vortex in https://github.com/open-sauced/hot/pull/242#discussion_r933760318_

rohit1101 commented 1 year ago

Can I work on this @bdougie ?

nickytonline commented 1 year ago

I'm currently discussing colour/typography etc. with @isabensusan in general, although we're more focused on the app atm, and not the hot repo.

rohit1101 commented 1 year ago

@nickytonline @bdougie Ok where can I find active issues to contribute ?

Any specific repos ?

nickytonline commented 12 months ago

When ininitially responded to this, I thought it was for the app repo. I edited my comment to reflect that.

This issue is a year old, so I don't really have much context on it.

For contributing @rohit1101, check out the contributing guide.