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

Подключен phosphor вместо fa #35

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Lootjs
Copy link
Contributor

@Lootjs Lootjs commented Jun 10, 2022

phosphor-vue имеет ограничение peer dependency, из за этого придется делать install с флагом --force : npm install --force

@Lootjs
Copy link
Contributor Author

Lootjs commented Jun 14, 2022

разрабы все еще не приняли мой PR, убирающий peer dependency. Пока думаю использовать обычный phosphor font

Comment on lines +9 to +13
## Иконки
Мы используем phosphor - https://phosphoricons.com/. Нажав на иконку, отобразится код для копирования.

Если надо использовать иконки в компонентах, то используем слоты.

Copy link
Collaborator

Choose a reason for hiding this comment

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

praise: Хорошая документация 👍

@KirillSolovyev
Copy link
Collaborator

question: Посмотрел примеры иконок: все они с закгрулениями. По дизайну у нас все углы прямые. Можно ли стилями убрать закгруление или есть иконки без закгрулений?

@Lootjs
Copy link
Contributor Author

Lootjs commented Jun 14, 2022

question: Посмотрел примеры иконок: все они с закгрулениями. По дизайну у нас все углы прямые. Можно ли стилями убрать закгруление или есть иконки без закгрулений?

@KirIIISolovyov ,
хмм, пожалуй, тут нам надо синхронизироваться с дизайнерами

@Lootjs
Copy link
Contributor Author

Lootjs commented Jun 14, 2022

Пока не понятно как решить проблему с этим пакетом - phosphor-vue. Разрабы игнорят мой PR phosphor-icons/vue#25

Есть вариант подключить иконки через пакет https://github.com/phosphor-icons/phosphor-icons, это по сути подключение css с иконками, весом в 2-3МБ.

Пока что форкнул проект phosphor-vue, убрал peer dependency, залил пакет в npmjs registry, но не работает почему то rollap build.

@KirIIISolovyov
Copy link
Collaborator

Пока не понятно как решить проблему с этим пакетом - phosphor-vue. Разрабы игнорят мой PR phosphor-icons/phosphor-vue#25

Есть вариант подключить иконки через пакет https://github.com/phosphor-icons/phosphor-icons, это по сути подключение css с иконками, весом в 2-3МБ.

Пока что форкнул проект phosphor-vue, убрал peer dependency, залил пакет в npmjs registry, но не работает почему то rollap build.

А у нас purge css настроен? Если нет, то нужно добавить issue. Если да, то он должен удалить неиспользуемые стили. Если кроме размера css файла нет минусов, то можно выбрать этот подход

@Lootjs
Copy link
Contributor Author

Lootjs commented Jun 15, 2022

Пока не понятно как решить проблему с этим пакетом - phosphor-vue. Разрабы игнорят мой PR phosphor-icons/phosphor-vue#25
Есть вариант подключить иконки через пакет https://github.com/phosphor-icons/phosphor-icons, это по сути подключение css с иконками, весом в 2-3МБ.
Пока что форкнул проект phosphor-vue, убрал peer dependency, залил пакет в npmjs registry, но не работает почему то rollap build.

А у нас purge css настроен? Если нет, то нужно добавить issue. Если да, то он должен удалить неиспользуемые стили. Если кроме размера css файла нет минусов, то можно выбрать этот подход

image

@Lootjs
Copy link
Contributor Author

Lootjs commented Jun 15, 2022

замерджили фикс, теперь все должно быть ок. без всяких --force флагов

@Lootjs
Copy link
Contributor Author

Lootjs commented Jun 15, 2022

жду релиз новой версии в npm

@Lootjs
Copy link
Contributor Author

Lootjs commented Jun 22, 2022

image
шел седьмой день

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