Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use gray on hover in button groups instead of primary #60

Merged
merged 2 commits into from
Nov 19, 2020

Conversation

nickdunn2
Copy link

@nickdunn2 nickdunn2 commented Nov 18, 2020

This PR just changes the hover color on button groups from primary to darken($gray-100, 5%) (the same as our btn-secondary hover state).

image

FLO-12070

@kalebdf kalebdf temporarily deployed to flo-scss-flo-12070-btn--xyeem4 November 18, 2020 23:39 Inactive
Comment on lines +51 to +60
css: 'sha384-9/reFTGAW83EW2RDu2S0VKaIzap3H66lZH81PoYlFhbGU+6BZp6G7niu735Sk7lN'
css_hash: sha384-hnvsmW4CEqIE9xpwdmvw+oyi2zalIdkqZJKT8ovyObj2fiPNSLt7xdsda4JGUfSf
js: 'sha384-bTuOyWFzvtzatBsqmXxShF1CAHyTzY4LiDCiZ5xdOi/Uq4W2ss8wllsHVHbNOd5P'
js: 'sha384-9/reFTGAW83EW2RDu2S0VKaIzap3H66lZH81PoYlFhbGU+6BZp6G7niu735Sk7lN'
js_hash: sha384-1RkA5wjfeG5u+6ycZ90q+PMlafLBaMCHvGk6PlVrQmIuTvpdA9qupxwbnPf9QbHH
js_bundle: >-
https://stackpath.bootstrapcdn.com/bootstrap/4.4.1/js/bootstrap.bundle.min.js
js_bundle_hash: sha384-gEp0IQktG6BLXHMpPT1pxMSC8EKUA159uPI9c/AmIhedq/EvAEfBroQ5tsSOS1/A
jquery: 'sha384-bTuOyWFzvtzatBsqmXxShF1CAHyTzY4LiDCiZ5xdOi/Uq4W2ss8wllsHVHbNOd5P'
jquery: 'sha384-9/reFTGAW83EW2RDu2S0VKaIzap3H66lZH81PoYlFhbGU+6BZp6G7niu735Sk7lN'
jquery_hash: sha384-J6qa4849blE2+poT4WnyKhv5vZF5SrPo0iEjwBvKU7imGFAV0wwj1yYfoRSJoZ+n
popper: 'sha384-bTuOyWFzvtzatBsqmXxShF1CAHyTzY4LiDCiZ5xdOi/Uq4W2ss8wllsHVHbNOd5P'
popper: 'sha384-9/reFTGAW83EW2RDu2S0VKaIzap3H66lZH81PoYlFhbGU+6BZp6G7niu735Sk7lN'
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no idea what these changes are.

@kalebdf kalebdf temporarily deployed to flo-scss-flo-12070-btn--jycprg November 19, 2020 19:43 Inactive
Copy link

@thatcatcancode thatcatcancode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK - seems like a pretty safe change, the build process is what confuses me

@kalebdf
Copy link
Contributor

kalebdf commented Nov 19, 2020

Feel free to merge away.

The build process can definitely be improved/demystified.

@nickdunn2 nickdunn2 merged commit c9869cd into master Nov 19, 2020
@nickdunn2 nickdunn2 deleted the FLO-12070/btn-group-hover-state branch November 19, 2020 22:00
matttelliott-flosports pushed a commit that referenced this pull request Nov 19, 2020
## [0.7.2](v0.7.1...v0.7.2) (2020-11-19)

### Bug Fixes

* use gray on hover in button groups instead of primary ([#60](#60)) ([c9869cd](c9869cd))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants