hot icon indicating copy to clipboard operation
hot copied to clipboard

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

Open bdougie opened this issue 1 year ago β€’ 4 comments

Making the gradient text classes more intuitive would be nice:

  • headerTextGradientStart/headerTextGradientMiddle/headerTextGradientEnd - πŸ‘Ž
  • headerTextGradient0/headerTextGradient50/headerTextGradient100 - πŸ‘

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

bdougie avatar Jul 30 '22 11:07 bdougie

Can I work on this @bdougie ?

rohit1101 avatar Nov 01 '23 06:11 rohit1101

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.

nickytonline avatar Nov 03 '23 00:11 nickytonline

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

Any specific repos ?

rohit1101 avatar Nov 08 '23 09:11 rohit1101

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.

nickytonline avatar Nov 08 '23 11:11 nickytonline