font-awesome <span> -> <li>

[Tetdoss]

Hello, your bundle works fine with font-awesome but font-awesome use < li > instead of < span > to display icon. It doesn't change anything but there are issues with others plugins.

For example, i'm using AdminLTE which uses font-awesome but when i want to use your {{icon...}} twig function, it is replaced with < span > and the css of AdminLTE doesn't handle < span > to display icon.

So can you normalize your bundle and use < li > instead of < span > for the font-awesome icons ?

Thank you ;)

[vrnagy]

You can change the icon_tag config parameter like:

braincrafted_bootstrap:
    icon_tag: i

[florianeckerstorfer]

We should totally change the default value to i in a feature release.

[Tetdoss]

Thanks for your answers, in fact I have this issue because i'm using bootstrap (useful for the carousel for example) AND font-awesome icon. But I'll try to use only font-awesome ;)