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

please update to latest font-awesome #706

Closed
cornernote opened this issue Oct 28, 2013 · 7 comments
Closed

please update to latest font-awesome #706

cornernote opened this issue Oct 28, 2013 · 7 comments
Assignees

Comments

@cornernote
Copy link

No description provided.

@digitalcrab
Copy link
Contributor

Hello @cornernote !

Font awesome was upgraded to 4.0.3 version and committed to master branch and will be included to next release.
New icons for example: http://yii-demo.yukki.name/bootstrap/issue706

Please read carefully next information:
https://github.com/FortAwesome/Font-Awesome/wiki/Upgrading-from-3.2.1-to-4
http://fontawesome.io/whats-new/

Thank you

@kvpt
Copy link
Contributor

kvpt commented Dec 14, 2013

Hello @fromYukki
Several widgets hardcode icon- prefix in icon property, the new naming convention breaks those widgets.

@digitalcrab
Copy link
Contributor

Hello! @Kev360 can you give me an example? As far as i know hardcoded only bootstrap icons. Font Awesome icons can be disabled at all, and it does not affect any Yii Booster functionality.

@kvpt
Copy link
Contributor

kvpt commented Dec 14, 2013

@digitalcrab
Copy link
Contributor

Hello @Kev360
As I said above, this applies only to icons from Bootstrap
http://getbootstrap.com/2.3.2/base-css.html#icons

If you run YiiBooster without any special settings font-awesome will be turned off, and TbButton still will be worked properly with icons from bootstrap.
Maybe you want to suggest to add the same code for fa- icon class?

digitalcrab pushed a commit that referenced this issue Dec 15, 2013
…cons" css for Bootstrap, changed logic for icons in widgets
@digitalcrab
Copy link
Contributor

Hello @Kev360
After my last commit you can use icons as before, and in this case icons will be used from Bootstrap. Example:

        <?php $this->widget('bootstrap.widgets.TbButton', array(
            'buttonType' => 'link',
            'icon' => 'music',
            'label' => 'Bootstrap Icon',
        )); ?>

And you can use Font-Awesome icons too (in the same time). Example:

        <?php $this->widget('bootstrap.widgets.TbButton', array(
            'buttonType' => 'link',
            'icon' => 'fa fa-anchor',
            'label' => 'Font-Awesome Icon',
        )); ?>

Result: http://yii-demo.yukki.name/bootstrap/issue706

Maybe you have any other useful suggestions?
Thank you so much!

@kvpt
Copy link
Contributor

kvpt commented Dec 29, 2013

Thanks !!!

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

No branches or pull requests

3 participants