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.

Sign up for GitHub

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

Jump to bottom

add material icons #86

Merged
merged 8 commits into from Mar 23, 2016
Merged

add material icons #86

merged 8 commits into from Mar 23, 2016

Conversation

thatkookooguy
Copy link
Member

Change Summary

  • add icons to the menu-bar component.

    Basically, added the icons that already present in the demo (while replacing things for their kibibit uses).
    I used the angular-material font icon set so now we have those usable as well.

This change is Reviewable

@thatkookooguy
add material icons
099d84d 
will allow us to use some default angular material icons which gives us extra points 😉
@thatkookooguy thatkookooguy deployed to kibibit-demo-pr-86 March 23, 2016 08:49 Active
@thatkookooguy
Copy link
Member Author

Well.... build kept failing on Heroku and Travis because of left-pad/left-pad#4. But now everything works again because NPM un-un-published it 😉

thanks for the fun open-source experience everybody 😄

@thatkookooguy thatkookooguy deployed to kibibit-demo-pr-86 March 23, 2016 09:31 Active
@thatkookooguy thatkookooguy merged commit 74771fc into master Mar 23, 2016
@thatkookooguy thatkookooguy deleted the add-material-icons branch March 23, 2016 09:36
neilkalman-redkix pushed a commit that referenced this pull request Jul 4, 2016
@thatkookooguy
Merge pull request #86 from Kibibit/add-material-icons
8ea4b52 
add material icons
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ortichon ortichon

Labels
Front End
Projects
None yet
Milestone
01. Create a kickass **code editor** ...
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@thatkookooguy @ortichon