Custom Header Links (icons)

Я обновил это и получил следующее сообщение об ошибке на странице /admin/customize/themes:

Ошибка: Неопределенная переменная: "$add-whitespace". в строке 14 файла common.scss >> @if $add_whitespace == "true" { ----^

Обновление: Я установил новый флажок «whitespace» и снял его (сохраняя каждый раз), и ошибка исчезла.

2 лайка

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

Тем не менее, я кратко обсудил это с Дэвидом, и он предложил в любом случае добавлять rel="noopener" к внешним ссылкам. Я добавил это здесь. Если ссылка настроена на открытие в новой вкладке, этот атрибут добавляется автоматически.

Что касается noreferrer,

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

Это всё ещё актуально? Можете ли вы поделиться скриншотом настроек вашего компонента?

Да, я тоже заметил это при обновлении, но это не связано с новыми изменениями. Как-то так сложилось, что компилятор не сразу подхватывает новое Setting. Поэтому, возможно, вам придётся поставить галочку в этом поле и снять её.

Но не волнуйтесь, по моим тестам, эта ошибка никак не влияет на работу вашего сайта.

5 лайков

Сегодня наш сайт сообщил мне, что нужно обновить компонент темы, что я и сделал, нажав кнопку «Обновить». Но теперь появляется ошибка:
Ошибка: Неопределённая переменная: "$add-whitespace". в строке 14 файла common.scss >> @if $add_whitespace == "true" { ----^

У меня была эта проблема, и она решилась сама собой, когда я поставил и снял галочку: Custom Header Links (icons) - #129 by Jonathan5

2 лайка

Спасибо за отчет @GuidoD

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

Сообщите, если возникнут какие-либо проблемы на вашем сайте — их быть не должно.

2 лайка

Верно, я не заметил никаких дальнейших проблем.
В настоящее время я выполняю пересборку сайта на Discourse и проверю всё снова после этого. Если ошибка останется, я обращу внимание на подсказку Джонатана.

После пересборки ошибка исчезла :slight_smile:

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

2 лайка

Привет! Я работаю на версии Discourse 2.8.0.beta1, и при обновлении до последней версии этого компонента темы я получаю следующее сообщение об ошибке:

Ошибка: Неопределённая переменная "$add-whitespace". на строке 14 файла common.scss >> @if $add_whitespace == "true" { ----^
2 лайка

У меня такая же ошибка. Я на версии 2.7 stable.

Понимаю. Как я уже упоминал выше, это никак не повлияет на ваш сайт.

4 лайка

Могу подтвердить, что это работает.

Отметьте эту настройку, затем снимите отметку и перезагрузите страницу.

screenshot-vapingcommunity.co.uk-2021.06.07-13_48_21

Привет, спасибо за этот плагин.
Я заметил несколько моментов, касающихся иконок FA, которыми хотел бы поделиться:
Интеграция иконок far- (regular) кажется рабочей, а fas- (solid) — нет.

Попробуйте, например, это:

fas-building
против
far-building

Что вы думаете по этому поводу? Иконок far- значительно меньше, чем fas- (не говоря уже о дизайнерской стороне), поэтому это проблема…

Кажется, вам нужно поискать в коде, какие иконки можно использовать и какие имена для них применять (если это не изменилось с момента написания комментария выше).

Что ж, описание компонента темы предполагает, что можно использовать любой бесплатный значок FA5 — независимо от его класса (brand, regular, solid).

Посмотрите на код, на который вы ссылаетесь:

Здесь нет условия if для solid. Значит, должно быть написано так:

when "solid"
  key = "fas-"

?

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

Ладно — в данном случае это должно быть исправлено в описании компонента темы?
Но на самом деле это работает со всеми иконками far-, так что, по моему мнению, иконки fas- просто «забыли».

1 лайк

Мои познания ограничиваются тем, что это не работает, как описано, и как понять, какие иконки будут работать, просто посмотрев на код :slight_smile:

Кажется, имена переменных для сплошных иконок обычно реализуются без префикса. Вы пробовали просто building?

Да, конечно. Безрезультатно.
Кажется, с Discourse поставляется лишь небольшое подмножество fas-.

У меня работает.. по умолчанию включён поднабор (вы можете увидеть его в /styleguide/atoms/icons). Дополнительные иконки нужно сначала добавить — либо в настройках администратора, либо непосредственно в компоненте, как в этом случае:

2 лайка

У меня тоже получилось (используя «building» в SVG-иконках и в компоненте темы) — отображается сплошная версия иконки :+1: Возможно, мне придётся съесть свои слова (я всё ещё почти уверен, что некоторые вещи не работали, хотя должны были…)

2 лайка