Conversation
|
@girishpanchal30 Tested this feature:
Also how the filter provided should be used? I tried to test it but i couldn't make it work. |
|
@irinelenache Fixed display icon issue when updating from the latest version of the plugin. For filter:
Thanks |
|
@girishpanchal30 Now the icons work fine in frontend after updating, but there is still a problem in dashboard. Encountered it with 2 icons (Safari, Bitcoin) Followed your steps for the filter and it works fine 👍 |
|
@irinelenache Fixed the above issue please retest again. Thanks |
|
@girishpanchal30 Now the issue is fixed, thanks 🚀 |
|
@girishpanchal30 I don't think this method of adding icons would work. As an end-user, it would be hard for me to connect to FTP, find the file and add something to an array every time I want another icon. What I would do here is to create another category named Font Awesome 5 and load the new icons as we do with fa4 We can avoid conflicts by not allowing users to use both fa5 and fa4 icon sets at the same time ... or at least display a notice somewhere in the dashboard that the icons might not work well together. |
|
@cristian-ungureanu do you think it would be good if we register the fa5 icon as a new category? Because we already have 300+ icons added in our plugin. Thanks |
|
@girishpanchal30 if adding a new category for FA5 is a bad idea, at least let's provide a way for the users to add fa5 classes in a field somewhere in the plugin. We can have a field where the customer adds all the classes that are needed separated with ",", then those icons will be available as the rest of the icons. |
|
@cristian-ungureanu Yes, we can add a textarea field in the plugin settings panel. there are no hooks to add a field in you have mentioned location. Ref - https://tinyurl.com/y3hsav6r Thanks |
|
@irinelenache Can you retest again? I have added a new field to add more FA5 icons cc @cristian-ungureanu |
|
@girishpanchal30 Tested again and it works fine for me 👍 If @cristian-ungureanu approves it, it can be moved to Ready to merge |
|
@girishpanchal30 Tested again and everything works fine 👍 |
|
Hi @girishpanchal30, I've seen that you've requested edit acces on one of Menu Icon figma files. Have a great day. |
|
🎉 This PR is included in version 0.12.11 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |

Close #130