Запрос: добавить иконки бренда Zulip в локальную спрайт-таблицу Font Awesome

Внести вклад Функция
1

Привет от сообщества Julia Discourse!

Zulip теперь доступен в качестве иконки бренда Font Awesome (версия 7.2+). Пакет @fortawesome/fontawesome-free в ядре уже обновлён до версии 7.3.1, но предоставляемый спрайт всё ещё собирается из версии v6.6.0, поэтому, похоже, пока ещё нельзя включить fab-zulip через настройку подмножества SVG-иконок.

Мы хотим добавить fab-zulip и fab-circle-zulip в этот встроенный спрайт, аналогично тому, как это было сделано в UX: update twitter icon for X, add threads & mastodon icons - Pull Request #26203 - discourse/discourse - GitHub (в соответствии с запросом в Add temporary Threads logo to current Font Awesome version). Мы готовы создать PR по этой схеме, но сначала хотел уточнить здесь, является ли это по-прежнему правильным подходом.

Заранее благодарю за ваши рекомендации!

4 лайка
2

Раз уж ты об этом заговорил, нам стоит выстроить более удобный процесс для актуализации этих данных, чтобы такие дополнения не требовали особых запросов, а мы не отставали слишком сильно.

Я посмотрю, что можно сделать. Если хочешь, можешь в это время создать PR для добавления иконки — я проверю и объединю его за тебя… Следование предыдущему образцу должно быть в порядке.

3 лайка
3

Спасибо! Открыл здесь: https://github.com/discourse/discourse/pull/43564

1 лайк