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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Icon refactor #4049

Merged
merged 4 commits into from Oct 21, 2015
Merged

Icon refactor #4049

merged 4 commits into from Oct 21, 2015

Conversation

BigBlueHat
Copy link
Contributor

While working on https://github.com/hypothesis/developers.hypothes.is I wanted to use the GitHub icons inline in some content. This is essentially the same refactoring I did there, but on the default Jekyll theme--in hopes that it helps others.

The footer has been refactored to use these new icon includes, and about.md shows an example of using them inline.

Example:

{% include icon-github.html username="bigbluehat" %}

These could also be handy on author pages and the like.

Hope it's useful!
馃帺

Made one for each to avoid over paramaterizing
the includes. Also allows for various icon formats.
Had to remove whitespace from icon includes
because Markdown...
@envygeeks
Copy link
Contributor

I am absolutely 馃憤 on this because it demonstrates to a further extend how some of our software works and it also makes it cleaner to read the source. 鉂わ笍

You can find the source code for Jekyll at [github.com/jekyll/jekyll](https://github.com/jekyll/jekyll)
You can find the source code for Jekyll at
{% include icon-github.html username="jekyll" %} /
[jekyll](https://github.com/jekyll/jekyll)
Copy link
Member

Choose a reason for hiding this comment

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

What is this addition about? Can you explain this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's meant to be an example of in using the icons inline.

It looks like this:

2015-10-20 20_07_43-store

@parkr
Copy link
Member

parkr commented Oct 20, 2015

Where are the SVG's used?

@envygeeks
Copy link
Contributor

I believe the only place we had them previously was in the footer? /cc @jglovier who would know more if I recall right yeah?

@BigBlueHat
Copy link
Contributor Author

@parkr @envygeeks previously they were only used in the footer. This allows them to also be used within page content--which can be pretty handy when generating author or code project pages.

See the screenshot here: #4049 (comment)

@jglovier
Copy link
Member

I believe the only place we had them previously was in the footer?

Yup, that's correct.

Also, 馃憤 馃憤 !

@jglovier
Copy link
Member

Sorry, I lied. There is also a hamburger menu bar which is used for collapsing the navigation on mobile.

@gynter
Copy link
Contributor

gynter commented Oct 21, 2015

馃憤 Why did You create separate SVG files for SVG content only? I'd probably recommend putting SVG data directly to the icon includes. One include less to make could mean smaller generation time for webs with hundreds of pages.

parkr added a commit that referenced this pull request Oct 21, 2015
@parkr parkr merged commit 36f35bf into jekyll:master Oct 21, 2015
parkr added a commit that referenced this pull request Oct 21, 2015
@parkr
Copy link
Member

parkr commented Oct 21, 2015

Thanks!

@envygeeks
Copy link
Contributor

鉂わ笍

@BigBlueHat
Copy link
Contributor Author

@gynter the reason for the separate .svg files is that they're actually .svg files now. 馃槃 They can be edited directly in Inkscape or whatever if needed. I like my files raw. 馃嵈 馃槈

Thanks for the merge @parkr!

@jglovier
Copy link
Member

shaggy-guy-thumbsup

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants