Звучит как отличная идея — добавить туда плагин- outlet должно быть довольно просто.
Ах — я ещё не установил компонент темы, поэтому не мог увидеть назначенную вкладку. Теперь установил и вижу, что она там есть. Супер! ![]()
Я быстро посмотрел, и мне кажется, что это не связано с компонентом? Я также могу воспроизвести серую иконку на try:
Но в безопасном режиме иконка синяя и для meta, и для try… Странно.
Да, я полностью поддерживаю это предложение. От шестерёнки можно отказаться. Есть и другие способы попасть в настройки, и её наличие здесь не критично.
Я думаю, что с этим аспектом стоит подождать. Это уже довольно радикальное изменение; нам стоит некоторое время пожить с ним, прежде чем добавлять ещё больше изменений.
В целом мне это очень нравится ![]()
Я тоже заметил это на community.wanikani.com, так что, похоже, дело не во мне.
Это изменение пока действительно крутое. Мне нравится, что оно делает закладки более заметными, ведь раньше о них было так легко забыть.
Убрал шестерёнку и немного увеличил ширину:
Не уверен, нужно ли ещё больше (верхний вариант)?
На самом деле я уже внёс это изменение, но готов откатить его или оставить как отдельный компонент, не интегрируя в ядро позже.
Кстати, вот ещё один макет, который я рассматривал, но можем обсудить это позже, когда будем рассматривать такие варианты.
(Новая вкладка профиля зависит от этого PR).
У меня есть один комментарий. По крайней мере, в мобильной тёмной теме довольно сложно определить, какая вкладка выбрана. Если присмотреться, можно увидеть линию вокруг иконки выбранной вкладки, но она очень тонкая. Возможно, цвет этой линии должен иметь больший контраст с фоном? Кстати, цвет линии, которую я вижу вокруг поля ввода ответа, пока пишу это сообщение (по-прежнему в мобильной тёмной теме), мог бы подойти отлично.
Это ФАНТАСТИЧЕСКИ
… мои следующие замечания будут очень придирчивыми … так что, пожалуйста … потерпите меня. ![]()
- По какой-то причине … теперь у меня появляется полоса прокрутки, значит мы сбились на несколько пикселей. Я знаю, что @Johani и @awesomerobot немного боролись с динамическим размером здесь, так что это может быть не совсем новая проблема.
- Нам нужно перенести настройки анонимного режима в ваш новый замечательный дом (настройка сайта “разрешить анонимные публикации”):
- Вкладки любят ушки… Ладно, это самая придирчивая из всех, но 4 пикселя здесь помогут им выглядеть больше как вкладки
4. Стрелка вниз здесь не имеет смысла, она только добавляет шум:
Мое ощущение здесь таково, что это готово к выпуску
, давайте жить с этим еще неделю (с учетом супер-придирчивых замечаний, которые я только что высказал). Затем этот недолговечный компонент, возможно, придется удалить в пользу простого включения его в ядро.
Чем больше я это использую, тем больше мне это нравится. Это делает закладки и назначение гораздо более полезными для меня.
Мне нравится это изменение!
У меня есть одно предложение — не стоит ли расположить сообщения после уведомлений, поскольку они используются чаще, чем закладки в целом?
2 незначительных улучшения, которые, на мой взгляд, были бы полезны:
1) При переходе на страницу профиля в верхней части вкладки находятся:
Предлагаю добавить их (фактически, Уведомления, Приглашения и Значки пока отсутствуют) во вкладку профиля (то есть sam) в следующем порядке (при этом следует убрать стрелку вниз, так как здесь она не имеет смысла).
2) При наведении курсора на стрелку вниз во всех вкладках отображается подсказка «Посмотреть более старые уведомления», но в соответствующих вкладках было бы логичнее видеть «Посмотреть более старые закладки» и «Посмотреть более старые сообщения».
Этот компонент темы пока работает отлично. Спасибо за вашу работу!
Думаю, это зависит от длины заголовков тем в выпадающем списке, которая сильно варьируется. Один из способов «исправить» это — ограничить все заголовки так, чтобы они помещались в одну строку, и скрыть излишки, но это будет выглядеть довольно неприятно. Однако ограничение их ровно двумя строками кажется более приемлемым, исходя из того, что сейчас у меня в выпадающем списке?
+1. У меня тоже появляется полоса прокрутки, независимо от того, есть новый компонент или нет.
Это связано с тем, как мы сейчас оцениваем количество уведомлений для отображения. Если общая высота превышает высоту окна (в случае нескольких уведомлений в три строки), появится полоса прокрутки.
Отлично, давайте попробуем это в компоненте! Добавил стилизацию для этого.
![]()
Я выбрал нижний подход (проще и красивее?), чтобы немного сдвинуть элементы внутрь. Но я заметил, что иконка человека больше не идеально выровнена с иконками ниже:
Насколько это приемлемо для вас? Пока постараюсь найти чистое решение этой проблемы.
Убрал её для вкладки профиля!
Кроме того, стрелка всегда отображается для назначений, закладок и сообщений, независимо от количества элементов. Я беспокоился, что функция «второй клик для перехода на полную страницу» недостаточно очевидна.
Ах, спасибо, хорошее замечание! Или, может быть, подойдёт общее «просмотреть всё»? Отложу это (и ещё раз пересмотрю использование меток), пока мы не интегрируем это в основную часть.
Понятно, спасибо! Я немного скорректировал цвет для тёмного режима.
Мне это выглядит фантастически! Я считаю, что нам стоит оставить это как есть до следующей среды на meta. Затем… PR абсолютно приветствуется, чтобы внести это в ядро и удалить компонент. @codinghorror тоже это одобряет.
Меня не смущает небольшое смещение из-за уха — на мой взгляд, это выглядит нормально, хотя обычно такие вещи меня сильно раздражают.
Мне нравится скорость итераций здесь, качество и та проработка, которую вы в это вложили!
На данный момент единственная мелочь, которую я могу отметить, — это то, что при первоначальной загрузке элементы немного «прыгают», и на долю секунды появляется полоса прокрутки. Это очень незначительно, и я уверен, что немного CSS это исправит.
Ошибка: при нажатии «Выйти» вы перенаправляетесь на сводку профиля пользователя, и выход из системы не выполняется:
Я могу подтвердить сообщение об ошибке от @nexo здесь на meta.
На моих сайтах у меня пока нет этого меню person tobiaseigen — при нажатии на него меня просто перекидывает на страницу активности.
Спасибо! Я только что внес исправление! Обновление компонента темы должно помочь.
Не могли бы вы попробовать обновить Discourse? Для вкладки быстрого доступа к профилю/личному кабинету вам потребуется последняя версия Discourse, а именно этот коммит от двух дней назад.
Это была быстрая работа — исправление сработало!
Ах, я должен был догадаться попробовать это в первую очередь.
Но да, обновление Discourse помогло, и теперь эта функция работает на моём сайте.
Спасибо! ![]()
А как насчёт настройки пользовательских ссылок в компоненте темы? Я вижу, что это может быть невероятно полезно для любого сайта с SSO, чтобы предоставлять ссылки на панель управления, актуальные только для авторизованных пользователей. На моём сайте я просто заменил пользовательский заголовок на Brand Header, что работает отлично и при этом не требует кастомизации в будущем! Теперь также хотелось бы иметь возможность добавить ссылку «Редактировать профиль», ведущую на WordPress, которая тоже была бы устойчива к будущим изменениям.
Вы можете сделать все уведомления во вкладках прокручиваемыми. Даже при быстром просмотре происходит перенаправление на другую страницу.













