Отчёт составлен верно, но оценка серьёзности проблемы немного завышена. Эти ссылки не являются пользовательским контентом. Добавлять их могут только администраторы через панель управления. Поэтому можно с уверенностью предположить, что администратор не будет добавлять ссылки на ненадёжные сайты.
Тем не менее, я кратко обсудил это с Дэвидом, и он предложил в любом случае добавлять rel="noopener" к внешним ссылкам. Я добавил это здесь. Если ссылка настроена на открытие в новой вкладке, этот атрибут добавляется автоматически.
Что касается noreferrer,
я считаю, что добавлять его не стоит, так как вы, вероятно, захотите ссылаться на свой основной сайт и получать аналитику о том, сколько пользователей перешло на основной сайт с вашего сообщества.
Это всё ещё актуально? Можете ли вы поделиться скриншотом настроек вашего компонента?
Да, я тоже заметил это при обновлении, но это не связано с новыми изменениями. Как-то так сложилось, что компилятор не сразу подхватывает новое Setting. Поэтому, возможно, вам придётся поставить галочку в этом поле и снять её.
Но не волнуйтесь, по моим тестам, эта ошибка никак не влияет на работу вашего сайта.
Сегодня наш сайт сообщил мне, что нужно обновить компонент темы, что я и сделал, нажав кнопку «Обновить». Но теперь появляется ошибка: Ошибка: Неопределённая переменная: "$add-whitespace". в строке 14 файла common.scss >> @if $add_whitespace == "true" { ----^
Верно, я не заметил никаких дальнейших проблем.
В настоящее время я выполняю пересборку сайта на Discourse и проверю всё снова после этого. Если ошибка останется, я обращу внимание на подсказку Джонатана.
После пересборки ошибка исчезла
Спасибо за отличный компонент темы, который мы используем уже два года.
Привет, спасибо за этот плагин.
Я заметил несколько моментов, касающихся иконок FA, которыми хотел бы поделиться:
Интеграция иконок far- (regular) кажется рабочей, а fas- (solid) — нет.
Попробуйте, например, это:
fas-building
против
far-building
Что вы думаете по этому поводу? Иконок far- значительно меньше, чем fas- (не говоря уже о дизайнерской стороне), поэтому это проблема…
Кажется, вам нужно поискать в коде, какие иконки можно использовать и какие имена для них применять (если это не изменилось с момента написания комментария выше).
Ладно — в данном случае это должно быть исправлено в описании компонента темы?
Но на самом деле это работает со всеми иконками far-, так что, по моему мнению, иконки fas- просто «забыли».
У меня работает.. по умолчанию включён поднабор (вы можете увидеть его в /styleguide/atoms/icons). Дополнительные иконки нужно сначала добавить — либо в настройках администратора, либо непосредственно в компоненте, как в этом случае:
У меня тоже получилось (используя «building» в SVG-иконках и в компоненте темы) — отображается сплошная версия иконки Возможно, мне придётся съесть свои слова (я всё ещё почти уверен, что некоторые вещи не работали, хотя должны были…)