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

added angular font-awesome and removed assets #573

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mohaseebtariq
Copy link

@mohaseebtariq mohaseebtariq commented Nov 5, 2024

This pull request addresses an issue with the font icon support in the angular-editor package after its upgrade to Angular 13. The project was previously using Font Awesome version 4.7, which is no longer compatible due to support and dependency issues.

To resolve this, I have forked the project and integrated @fortawesome/angular-fontawesome, which is specifically designed for Angular applications. This update ensures that font icons are fully functional and provides a more modern and maintainable approach to icon management within the editor.

Changes Made:

Removed the existing Font Awesome 4.7 dependency.
Added @fortawesome/angular-fontawesome as a dependency.
Updated relevant components to utilize the new icon library, ensuring compatibility with Angular 13 and beyond.
Benefits:

Improved compatibility with the latest Angular versions.
Enhanced performance and maintainability of the icon management.
Access to the latest Font Awesome features and icons.
I believe this enhancement will greatly benefit users of the angular-editor by providing a more robust and modern icon solution.

@qqaimh
Copy link

qqaimh commented Nov 5, 2024 via email

@kolkov
Copy link
Owner

kolkov commented Dec 9, 2024

Этот запрос на включение изменений решает проблему с поддержкой значков шрифтов в пакете angular-editor после его обновления до Angular 13. Ранее в проекте использовался Font Awesome версии 4.7, который больше несовместим из-за проблем с поддержкой и зависимостями.

Чтобы решить эту проблему, я разветвил проект и интегрировал @fortawesome/angular-fontawesome, который специально разработан для приложений Angular. Это обновление гарантирует, что иконки шрифтов полностью функциональны, и обеспечивает более современный и поддерживаемый подход к управлению иконками в редакторе.

Внесенные изменения:

Удалена существующая зависимость Font Awesome 4.7. Добавлена ​​@fortawesome/angular-fontawesome в качестве зависимости. Обновлены соответствующие компоненты для использования новой библиотеки иконок, что обеспечивает совместимость с Angular 13 и выше. Преимущества:

Улучшенная совместимость с последними версиями Angular. Улучшенная производительность и удобство обслуживания управления иконками. Доступ к последним функциям и иконкам Font Awesome. Я считаю, что это улучшение принесет большую пользу пользователям angular-editor, предоставив более надежное и современное решение для иконок.

Thank you very much! I will look at and evaluate the request as soon as possible. Before, I wanted to make completely independent icons of the project, but maybe it will be better this way.

@kolkov
Copy link
Owner

kolkov commented Dec 26, 2024

Thanks for your contribution, it made me think that it was high time to get rid of the FontsAwesome dependency completely. Check the latest version, is everything working?

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