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 duotone style #250

Closed
wants to merge 3 commits into from
Closed

add duotone style #250

wants to merge 3 commits into from

Conversation

MoElnaggar14
Copy link
Contributor

@MoElnaggar14 MoElnaggar14 commented Mar 27, 2020

add duotone style #223, #230 , all you need is just drag and drop font-awesome pro

@ghowen
Copy link

ghowen commented Mar 27, 2020

Duotone fontName needs to be FontAwesome5Duotone-Solid, otherwise loading the font will crash the app.

@MoElnaggar14
Copy link
Contributor Author

it's related to font files in FontAwesome but it's okay

chriszielinski added a commit to chriszielinski/FontAwesome.swift that referenced this pull request Mar 29, 2020
@MatrixSenpai
Copy link
Collaborator

This adds the capability to use a duotone font, but does nothing about actually implementing. If you were to actually put this in a label, image, or other FontAwesomeUIControl subclass, it wouldn't produce the desired result. More needs to be added to the actual UIControl subclasses

@MatrixSenpai
Copy link
Collaborator

I'm closing this without merging for now. I want to come back and review this issue later. This solution is incomplete, and I want to implement parts of it, just not now

@MoElnaggar14 MoElnaggar14 deleted the patch-1 branch February 6, 2021 13:33
christophertrudel pushed a commit to christophertrudel/FontAwesome.swift that referenced this pull request Mar 10, 2021
christophertrudel pushed a commit to christophertrudel/FontAwesome.swift that referenced this pull request Mar 11, 2021
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.

3 participants