Skip to content

Added stacked_icon helper#71

Open
jtrost wants to merge 1 commit intoFortAwesome:mainfrom
jtrost:master
Open

Added stacked_icon helper#71
jtrost wants to merge 1 commit intoFortAwesome:mainfrom
jtrost:master

Conversation

@jtrost
Copy link

@jtrost jtrost commented Jan 31, 2015

This feature is from issue #59

Usage

<%= stacked_icon(
  icon("camera", "", class: "fa-stack-1x"), # The first icon to stack
  icon("ban", "", class: "fa-stack-2x text-danger"), # The second icon to stack
  class: "fa-lg" # Additional classes for fa-stack
) %>

Output

<span class="fa-stack fa-lg"><i class="fa fa-camera fa-stack-1x"></i><i class="fa fa-ban fa-stack-2x text-danger"></i></span>

Choose a reason for hiding this comment

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

Line is too long. [82/80]

@jtrost jtrost mentioned this pull request Jan 31, 2015
@jvanbaarsen
Copy link

@supercodepoet Is this something you would be interested in? If so what do you think need to be changed for this to be merged? I can take over this PR if needed.

@ignisf
Copy link

ignisf commented Jul 27, 2015

@supercodepoet, apart from the stylistic issues @houndci found in the PR, is there another reason you believe it should not be merged?

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.

4 participants