Skip to content
This repository has been archived by the owner on Dec 28, 2022. It is now read-only.

fontawesome icons #43

Closed
clojens opened this issue Nov 10, 2014 · 9 comments
Closed

fontawesome icons #43

clojens opened this issue Nov 10, 2014 · 9 comments

Comments

@clojens
Copy link

clojens commented Nov 10, 2014

I noticed quite a few icons were missing between the version used here and current font-awesome release. I hacked my local asciidoc-preview but I could do a pull request if you like. I just replaced all fonts/ and less/ contents of the folders - not knowing or caring about any tweaking you guys did and it works fine, just checking to make sure you didn't have any patched magic going on I wasn't aware of or if there was any other reason for not wanting to do a bump on fontawesome.

Cheers,
Rob

@clojens clojens changed the title fontawesome fontawesome icons Nov 10, 2014
@mojavelinux
Copy link
Member

No reason other than we just haven't upgraded yet. So let's do it!

We need to be sure to update it in core as well so it remains consistent. Could you file an issue there too? I'll schedule it for 1.5.2.

Are you aware of any incompatibilities between Font Awesome 4.2.0 and 4.1.0 (the current version we're using). If not, 👍

@clojens
Copy link
Author

clojens commented Nov 20, 2014

Nah seems all they did was add 40 new icons this time and no renames or anything, we should be fine. I'll take a closer look again and lookup the core repo to pull request there as well.

@mojavelinux
Copy link
Member

This answers my question I asked in pull request in core, so it will make
it into 1.5.2!

@anthonny
Copy link
Member

Wip
capture du 2014-11-27 01 42 10

@clojens
Copy link
Author

clojens commented Nov 27, 2014

Oh right one more thing: may I suggest default non-italic icons?

@mojavelinux
Copy link
Member

That seems like it's caused by a misconfiguration of the bundled stylesheet
since it should be handled by the default stylesheet.

@mojavelinux
Copy link
Member

I'll have a look to see what's going on.

@anthonny
Copy link
Member

Fixed
capture du 2014-11-27 07 18 05

But i need to refactor and use the stylesheet factory, i will do it when i'll deal with #39

@mojavelinux
Copy link
Member

Most excellent!

But i need to refactor and use the stylesheet factory, i will do it when i'll deal with #39

This is becoming a high priority for me, so let's put our heads together and get it done soon. I'm confident it will be a big win for the community.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants