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(core/button): add space between leading icon and text #149

Merged
merged 5 commits into from
Nov 20, 2022

Conversation

goncalosard
Copy link
Contributor

Summary

Added a space between the icon and the text on a button.
Used gap to create this space.

Fix #130

How did you test this change?

Tested locally

@danielleroux danielleroux changed the title Fix/toolbar toogle button styling fix(core/button): add space between leading icon and text Nov 17, 2022
Copy link
Collaborator

@nuke-ellington nuke-ellington left a comment

Choose a reason for hiding this comment

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

I think we can just put the gap into the main .btn class. Adding an extra class definition just adds unneccessary clutter IMO.

@danielleroux danielleroux added the pull request affects patch version The pull request affects only patch version label Nov 18, 2022
@danielleroux danielleroux added this to the 1.1.0 milestone Nov 18, 2022
@danielleroux danielleroux merged commit 8392ef6 into main Nov 20, 2022
@danielleroux danielleroux deleted the fix/toolbar-toogle-button-styling branch November 20, 2022 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pull request affects patch version The pull request affects only patch version
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Toolbar toggle button (button w/ icon and text) styling
4 participants