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

More button styles #17

Merged
merged 6 commits into from Nov 8, 2017
Merged

Conversation

Fraham
Copy link
Contributor

@Fraham Fraham commented Nov 2, 2017

Added the standard colour options to button and also included click transition for when the user clicks on the button

capture

Copy link
Contributor

@TotomInc TotomInc left a comment

Choose a reason for hiding this comment

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

Looks really good!

@rhyneav
Copy link
Member

rhyneav commented Nov 3, 2017

Absolutely love it, it looks great! The click effect is a nice touch too.

Also, when I merged in #13 it created a conflict in the colors.less file. Could you please resolve that file, and I'll get this merged in shortly thereafter.

I also really like the functions you've been creating to generate colors. That's definitely something I need to work on more myself :)

@Fraham
Copy link
Contributor Author

Fraham commented Nov 3, 2017

Conflict resolved

@rhyneav
Copy link
Member

rhyneav commented Nov 4, 2017

Awesome, thank you! One very last thing: when you added the background color transition, it removed the shadow transition for hover so it's no longer smooth. If you could please add this back in, we should be good to rock and roll!

@Fraham
Copy link
Contributor Author

Fraham commented Nov 5, 2017

Issue fixed

@Fraham Fraham changed the base branch from master to develop November 7, 2017 15:24
@rhyneav
Copy link
Member

rhyneav commented Nov 8, 2017

Thank you again, these look great!

@rhyneav rhyneav merged commit c83c9b1 into papercss:develop Nov 8, 2017
@rhyneav rhyneav mentioned this pull request Nov 8, 2017
@Fraham Fraham deleted the Fraham-MoreButtonStyles branch November 22, 2017 21:48
@Fraham Fraham mentioned this pull request Nov 22, 2017
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.

None yet

3 participants