Skip to content

Conversation

@sman591
Copy link
Member

@sman591 sman591 commented Sep 8, 2016

  • Sets a standard distance for all icon spacing
  • Uses em so that it scales proportionally to font size

I use this in all my projects as I think icons look a ton better when they're not up right against the text. Adding a space only does so much, and spaces can't stack to increase the distance.

I couldn't get a local environment working so I couldn't fully test this locally, but applying my changes in the web inspector looks like this:

Before:
screenshot 2016-09-07 21 52 25

After:
screenshot 2016-09-07 21 59 05

* Sets a standard distance for all icon spacing
* Uses em so that it scales proportionally to font size
@mbillow
Copy link
Member

mbillow commented Sep 8, 2016

Hot. 🔥

@stevenmirabito
Copy link
Collaborator

stevenmirabito commented Sep 8, 2016

@sman591 Looks great, although I think this would be more appropriate as an addition to csh-material-bootstrap. I'm trying to keep all style changes like this in the library so everything looks consistent across apps. If you could make this change over there, preferably as default behavior for glyphicons in navbars, that would be very much appreciated.

@sman591
Copy link
Member Author

sman591 commented Sep 8, 2016

@stevenmirabito Sure thing. I'll update this PR once I've added it there.

@stevenmirabito
Copy link
Collaborator

Closing as this has been added to csh-material-bootstrap and will be added to this project pending the release of v1.0.1 of that library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants