Предварительный просмотр списков тем (TLP)

Ага, вы запускаете это поверх плагина «Новости»? Боюсь, результаты этого непредсказуемы, и это может быть причиной проблемы. Плагин «Новости» официально не поддерживается, так как у него есть свои собственные шаблоны.

Если удалить плагин «Новости», исчезнет ли проблема?

3 лайка

Улучшения плиток и изменения во внутренней структуре плиток

@nathank и я работаем над оптимизацией макета внутри плитки:

  • Перераспределение элементов для более эффективного использования пространства, в частности:
    • Перемещение авторов в подвал плитки, что позволяет увеличить ширину для заголовка, категории и тегов.
    • Обеспечение того, чтобы авторы занимали только одну строку; добавление числа, обозначающего количество дополнительных авторов, если список большой.
    • Перенос мета-информации для освобождения места для большего количества авторов.
    • Размещение тегов рядом с категорией с сохранением возможности их переноса.
    • Оптимизация отступов.
  • Добавление новой кнопки «Поделиться» :chains:, чтобы вы могли делиться темой прямо из списка тем :rocket:
  • Различные исправления поведения и форматирования кнопок действий.
  • Удаление серой полосы в верхней части списка тем.
  • Уменьшение избыточности в CSS (особенно между десктопной и мобильной версиями).

NB: Рекомендуется сначала протестировать это на тестовом сайте, чтобы убедиться, что ваши переопределения CSS работают корректно.

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

NB: Я немного кастомизировал свою плитку, расположив авторов справа по умолчанию (в ядре они расположены слева):

.topic-list.tiles-style {
  .topic-users {
    justify-content: flex-end;
  }
}

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

7 лайков

Мы не можем выбрать миниатюру, верно? Иногда первое изображение не подходит для миниатюры!

1 лайк

Да, это возможно. Установите плагин sidecar, и в редакторе метаданных темы появится интерфейс для выбора миниатюры.

Если вы не можете устанавливать плагины, миниатюру можно выбрать и стандартным способом.

В теме, если вы хотите выбрать изображение, отличное от изображения в первом сообщении, добавьте |thumbnail к разметке. Например:

![alttext|100x100](upload://aaa)
![alttext|100x100|thumbnail](upload://bbb) << это изображение станет миниатюрой

Решение через плагин UI лучше, так как вы можете выбрать любое изображение из любого сообщения. Решение через Markdown ограничено первым сообщением.

4 лайка

@merefield Поскольку мастер запроса функций по ссылке больше не активен, я просто размещу его здесь.

Для плагина-сайдкара есть ли способ разрешить сотрудникам (модераторам, администраторам) изменять миниатюру пользователя?

1 лайк

Это уже должно быть возможно, у вас возникла проблема?

1 лайк

Да. Как администратор, я не могу изменить миниатюру тем других пользователей. Уже пробовал обновить HTML поста. Или, возможно, мне нужно перепечь (rebake)?

Мы работаем на последней версии, прошедшей тесты.

1 лайк

Получаете ли вы сообщение об ошибке в логах? На каком этапе происходит сбой? Надеюсь, вы используете предоставленный интерфейс?

1 лайк

В логах ничего нет
Изображения в подсказке не отображаются
Не совсем понимаю, что вы имеете в виду под «предоставленным интерфейсом», но вот быстрая демонстрация проблемы:

CleanShot 2022-09-08 at 14.25.04

1 лайк

Кстати, есть ещё одна небольшая проблема:

Даже если TLP не активирован для списка тем, в код незаметно проникает следующий CSS:

.mobile-view .topic-list .right {
  margin-left: 95px;
}

Смотрите видео:

CleanShot 2022-09-08 at 14.32.24

3 лайка

Моя вина, сейчас нужно быть пользователем или администратором:

С радостью рассмотрю это как запрос на добавление функции для модераторов.

3 лайка

Моя проблема в том, что она возникает, когда я администратор.

1 лайк

Загружаются изображения? У них должен быть идентификатор загрузки.

1 лайк

Привет, Роберт

Мне удалось выявить проблему! Как администратор, проблема возникает только для тем, созданных до установки плагина sidecar.

:heart: Наши модераторы, безусловно, будут этому очень рады :slight_smile:

1 лайк

Это происходило из-за того, как мы настроили тестовый сайт для форума jrgongs: изображения хранились в бакетах S3 на исходном форуме, на который указывает тестовый сайт. У тестового сайта другое имя current_db, поэтому Post::each_upload_url не может найти загруженные файлы.
Сбой произошёл не из-за того, что посты были созданы до установки плагина sidecar; он возник на постах, которые были созданы до того, как производственный сайт был скопирован на тестовый.

5 лайков

А … нужно ли от меня какое-то действие?

После обновления экземпляра Discourse я заметил, что избранные темы (те, которым присвоен тег featured) больше не отображаются при первоначальной загрузке страницы и появляются только после ручного выбора одного из списков тем: категории, последние и т. д. В моём случае в настройках интерфейса параметр страница по умолчанию установлен на «Категории», что ранее работало как ожидалось. Если в качестве страницы по умолчанию установлены «Последние», «Новые» или «Непрочитанные», избранные темы отображаются на странице, как и предполагалось, без необходимости ручного выбора списка тем. Если же выбраны «Категории» или «Лучшие», они не появляются, пока пользователь вручную не изменит список тем. Не уверен, упустил ли я что-то или, возможно, изменение в ядре Discourse повлияло на это поведение.

3 лайка

Готово.

Теперь вы должны иметь возможность просматривать миниатюры в качестве модератора.

Это также исправляет очевидную логическую ошибку здесь, где было присваивание вместо проверки на равенство! :open_mouth: :+1:

На самом деле логика интерфейса и безопасности здесь требует некоторого дальнейшего улучшения. Кнопка действительно должна быть скрыта, если вы не соответствуете критериям. Я вернусь к этому в TC в ближайшее время… Также API не является полностью безопасным с точки зрения этой логики, так как в настоящее время он просто предотвращает отображение кнопок. Для полной защиты требуются дополнительные изменения. Но учитывая, что это действие сопряжено с очень низким риском, я не думаю, что это является приоритетом.

5 лайков

Нет, это был в основном ложный сигнал, спасибо!

3 лайка

Спасибо за исправление! Протестируем его после развертывания на боевом сервере :slight_smile:

Есть ли шанс сделать так, чтобы кнопка «Выбрать миниатюру» отображалась только для тем, которые находятся в категории или имеют тег с включёнными миниатюрами в TLP tc? Или это потребует значительных усилий для реализации?

3 лайка