Значок часов в закладках сломан после смены набора иконок

При переключении иконок FontAwesome Pro с Solid на Light иконка, отображаемая при установке закладки с напоминанием на сообщение, перестает показываться.

Похоже, что плагин меняет <use xlink:href="#discourse-bookmark-clock"></use> на <use xlink:href="#fal-discourse-bookmark-clock"></use>, но в результате отображается пустое место, так как версии этой иконки с префиксом fal не существует.

3 лайка

У меня точно такая же проблема, в 2.9beta9 тесты пройдены

1 лайк

Выкапываю этот вопрос, так как он всё ещё актуален в версии 3.2

1 лайк

У нас всё ещё есть эта проблема в версии 3.3 :smiley:

1 лайк

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

К сожалению, поскольку это пользовательская иконка, созданная нами для Discourse на основе набора открытых иконок Font Awesome, я не думаю, что мы можем создать и распространить версию этой иконки в стиле Pro, так как это нарушило бы лицензирование Font Awesome Pro.

2 лайка

Эта тема была автоматически закрыта через 6 дней после последнего ответа. Новые ответы больше не принимаются.

Я вижу, что иконка была добавлена обратно в eb39fb6, но даже когда я перехожу на ваше исправление без более новых коммитов в ветке 3.3.1 stable, иконка закладки с часами возвращается, но, к сожалению, исчезают кнопка «Ещё» (d-icon-ellipsis-h) и кнопка «Редактировать» (d-icon-pencil-alt). У вас есть какая-то идея, что происходит? Я пытался очистить кэш иконок, но безрезультатно.

3 лайка

Интересно, связано ли это: Some icons not showing up after upgrade :thinking:

1 лайк

Кажется, согласно файлу .discourse-compatibility, вы привязаны к более старой версии плагина, верно? Возможно, это проблема совместимости со старыми версиями Discourse? discourse-fontawesome-pro/.discourse-compatibility at eb39fb6bdc567b77a54ce50366b804c9971467e2 · discourse/discourse-fontawesome-pro · GitHub

Я бы не ожидал, что удаление ссылки на одну иконку сломает другие, не связанные с ней иконки :thinking:

1 лайк

Хорошее замечание, но нет, я проигнорировал это :sunglasses:

2 лайка

Хм, а у вас нет более позднего коммита, в котором иконки были переименованы? DEV: update plugin to be compatible with core upgrade to fontawesome … · discourse/discourse-fontawesome-pro@eb39fb6 · GitHub

Когда вы заглядываете в инспектор, ссылка на иконку — ellipsis или ellipsis-h?

3 лайка

Итак, я нахожусь на cc8e59d7

image

1 лайк

Не совсем понятно, нужно ли нам держать это открытым @RGJ?

Что ж… это всё ещё нерешённая проблема… Я понятия не имею, что её вызывает.

Поэтому было бы здорово, если бы любой, кто может предложить что-то полезное, мог это добавить :wink:

1 лайк