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

Add new icons for compilation and monitoring #9

Merged
merged 1 commit into from
Sep 26, 2016

Conversation

jbergstroem
Copy link
Contributor

Not very proud of the sizing here. Did you (or someone else) perhaps adjust icon height to better fit the text-size in the svg? Other than that, I created two issues (#7, #8) I ran into while creating this PR.

@jbergstroem jbergstroem force-pushed the feature/additional-icons branch from fa799b3 to f5fab96 Compare September 22, 2016 18:28
@dfrankland
Copy link
Owner

Could you separate these into 2 pull-requests? It's hard to review otherwise. Thank you for your contribution! 😀

@jbergstroem jbergstroem force-pushed the feature/additional-icons branch from f5fab96 to 2c4fc0d Compare September 22, 2016 19:16
@jbergstroem
Copy link
Contributor Author

jbergstroem commented Sep 22, 2016

Will do. I'll rebase on the git stuff.

Edit: done!

This makes things like using `clang`, `gcc` or `top`
look better.
@jbergstroem jbergstroem force-pushed the feature/additional-icons branch from 2c4fc0d to 93ba876 Compare September 22, 2016 19:19
@jbergstroem jbergstroem changed the title Add new icons and introduce a configuration option for disabling colors Add new icons for compilation and monitoring Sep 22, 2016
@jbergstroem
Copy link
Contributor Author

ping!

@dfrankland dfrankland merged commit 6ff1534 into dfrankland:master Sep 26, 2016
@dfrankland
Copy link
Owner

pong! 👍

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.

2 participants