ondrej
03.Ноябрь.2024 21:50:39
1
Когда вы создаете тему из личного сообщения, иконка отсутствует, как видно на изображении ниже.
Это в теме действия администратора в личном сообщении
3 лайка
Lilly
(Lillian )
03.Ноябрь.2024 21:57:09
2
О, оно там точно есть, просто не отображается из-за каких-то проделок, возможно, Font Awesome 6 не совсем в порядке… lol
Спасибо за предупреждение!
2 лайка
Я только что преобразовал это в личное сообщение и обратно в тему, и кажется, что иконка отображается правильно?
Lilly:
Похоже, что по какой-то причине здесь ожидается другая иконка…
edit: посмотрев код для небольших действий, я вижу, что он использует «exclamation» в качестве запасного варианта, если не удаётся найти подходящую иконку… но у public_topic иконка определена… интрига нарастает
3 лайка
Firepup650
(Firepup Sixfifty)
04.Ноябрь.2024 18:42:39
7
Kris, пост:6, тема:334309:
Просматривая код для мелких действий, я заметил, что при отсутствии подходящей иконки используется «восклицательный знак»…
Не уверен, что это когда-либо работало на нашем форуме. У нас много шуточных постов, которые были превращены в действия в шутку, и ни у одного из них не отображаются иконки.
1 лайк
Moin
29.Декабрь.2024 13:53:03
8
Kris, пост:6, тема:334309:
Я только что преобразовал это в ЛС и обратно в тему, и, кажется, иконка отображается правильно?
Это происходит потому, что Meta — это публичный сайт. В таком случае код действия отличается.
https://github.com/discourse/discourse/blob/e9d4afec534349e9934cbd608f57ed010a61fd79/app/serializers/post_serializer.rb#L477-L480
Текст отличается, когда включена опция «требуется вход».
Думаю, проблема в том, что хотя для «public_topic» есть иконка, для «open_topic» иконка не определена.
const icons = {
"closed.enabled": "lock",
"closed.disabled": "unlock-keyhole",
"autoclosed.enabled": "lock",
"autoclosed.disabled": "unlock-keyhole",
"archived.enabled": "folder",
"archived.disabled": "folder-open",
"pinned.enabled": "thumbtack",
"pinned.disabled": "thumbtack unpinned",
"pinned_globally.enabled": "thumbtack",
"pinned_globally.disabled": "thumbtack unpinned",
"banner.enabled": "thumbtack",
"banner.disabled": "thumbtack unpinned",
"visible.enabled": "far-eye",
"visible.disabled": "far-eye-slash",
split_topic: "right-from-bracket",
invited_user: "circle-plus",
invited_group: "circle-plus",
user_left: "circle-minus",
removed_user: "circle-minus",
This file has been truncated. show original
4 лайка
Ах, теперь всё ясно! Спасибо, что разобрались.
В этом PR будет добавлена иконка:
main ← ux-open-topic-action
merged 06:50PM - 06 Jan 25 UTC
Investigated here: https://meta.discourse.org/t/convert-to-topic-icon-missing/33… 4309/8?u=awesomerobot
When a site is set to `login_required` the small action for converting a PM to a topic is `open_topic` rather than `public_topic` — and there's no icon defined for it. This adds one (the same as `public_topic`).
Before:

After:

5 лайков