Quick access to bookmarks and messages on user menu

Using the new profile menu on mobile, I notice that clicking any of these buttons

shows basically the same result - the top of the user profile page, which only has a navigation menu. You have to scroll down a lot to actually see what you came for.

Instead of that, why not show a short list of what you clicked on, right in the menu?

Bookmarks:

Messages:

16 лайков

I agree it’s odd to arrive at basically the profile menu on mobile when clicking these links. the idea of displaying results in the menu panel, like notifications, is interesting! or at least jump down to an anchor in the results?

2 лайка

This problem exists for a long time. It makes it quite difficult to use the profile on mobile.

https://meta.discourse.org/t/profile-page-on-mobile-selecting-menu-item-should-jump-to-the-selected-content/19896?u=gerhard

6 лайков

I like this idea! Although you’d need a “tab” (since that’s basically how these would function now) for notifications as well, to return to it.

Seems like 80+% of the time you’re only looking for recent bookmarks, messages, etc., so this might be useful.

You might also need a divider between the tabs and the settings icon though, since they now have different functions.

2 лайка

Basically profile page needs fixing. As @sam indicated it might be better if, when you view your own profile only, we compress the top part. It’s especially large on mobile too…

2 лайка

Mobile looks so much better these days :slight_smile: user page is much improved.

I still sort of like the idea of having a “quick access” somehow, cause despite the improvements I still avoid the user page quite a lot.

2 лайка

Hello everyone,

I started working on this one. Here is a part of my progress so far.

6 лайков

I think this change looks good, well done @nbianca, provided “view older {x}…” is updated to be conditional and link to the right place in each view:

  • view older bookmarks…
  • view older messages…
  • view older notifications…

technically assignments are included here as well @sam

This should be reviewed closely as it is a reasonably large change…

8 лайков

Это всё ещё pr-welcome? Я бы с радостью взялся за это!

Я прочитал тему и последний PR, но не до конца уверен в следующем:

  • После закрытия и повторного открытия меню пользователя (без обновления страницы) оно должно всегда показывать все уведомления?
  • Если нет, то когда приходят новые уведомления, я предполагаю, что открытие меню пользователя должно всегда показывать все уведомления?
5 лайков

Я думаю, что сначала стоит реализовать это в компоненте темы. Улучшения API, которые сделают такой компонент темы возможным (и при необходимости очистят ядро), будут только приветствоваться. Но я считаю, что тема — идеальное место для проверки этой концепции!

Затем мы можем установить это на meta, «пожить» с ним какое-то время и посмотреть, понравится ли нам это. Кроме того, наша инфраструктура ядра API и компонентов темы будет постепенно улучшаться, что является огромным плюсом.

9 лайков

Компонент темы находится здесь: GitHub - xrav3nz/discourse-quick-access · GitHub. Приветствуются предложения!

Я позаботился о том, чтобы дизайн был удобен для других плагинов и тем, позволяя им предоставлять собственную панель быстрого доступа. Я подробно опишу это (или добавлю пример в виде PR в discourse-assign), если и когда возникнет интерес.

Краткая демонстрация:

Цвет выбранной панели

Пока используется цвет для состояния наведения.

Переход на полную страницу

  • Вы всё ещё можете нажать правой кнопкой мыши или Ctrl+кликнуть, чтобы открыть полную страницу в новой вкладке, или
  • Нажать на ссылку внизу (она всегда отображается для закладок и сообщений), или
  • Дважды щёлкнуть по той же панели быстрого доступа.

Пустое состояние

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

21 лайк

@xrav3nz, выглядит отлично! Это уже работает? Я добавил компонент, но всё ещё наблюдаю старое поведение?

1 лайк

Отличная работа, всё очень качественно. Мне нравится, что второй клик переводит на страницу.

Это установлено здесь, на Meta. Просто выберите Sam's Simple Theme, чтобы попробовать его в меню-гамбургере.

Некоторые замечания:

  1. Это должно работать с discourse-assign. Попробуйте установить его и назначить что-нибудь себе.

  1. Мне кажется, желтый цвет здесь слишком навязчив с точки зрения стиля. @Johani / @awesomerobot, есть ли у кого-нибудь идеи для менее «шумного» оформления?

  2. Не совсем понятно, что делать с этим, но на мобильных устройствах области клика слишком малы. Возможно, компоненту стоит сделать элементы немного крупнее, чтобы было проще что-то изменить?

  3. Раздел «Настройки» выглядит «иначе». Может, стоит сделать его единым и добавить список с пунктами Summary (Сводка), Activity (Активность) или, например, список с Account (Аккаунт), Profile (Профиль), Emails (Почта) и так далее.

    a. На самом деле… подумав, возможно, стоит вообще убрать шестеренку и использовать иконку «человек Сэм», чтобы развернуть панель с пунктами Summary, Activity и т. д. Это даст вам большую область для клика по вкладкам.

В целом… потрясающая работа!

13 лайков

Отличная работа, @xrav3nz :clap: Выглядит потрясающе!

Мы можем использовать тот же паттерн для иконок заголовка и меню, например, так:

А вот как это будет выглядеть с непрочитанным уведомлением сверху:

15 лайков

Здесь автор оригинального поста. Рад видеть, что идея может сохраняться четыре года и всё ещё оказаться полезным дополнением!

В этом и заключается сила Discourse, если говорить кратко.

16 лайков

Эта функция просто отличная для закладок! :heart_eyes: Также хотелось бы видеть её для назначений.

Мне кажется, что рамка, которую нарисовал @johani, может быть немного слишком незаметной, но она мне нравится… Так что давайте остановимся на ней.

14 лайков

Спасибо за обратную связь! Только что выпустил обновление, чтобы устранить первые два пункта. Остальные два разберём позже на этой неделе (#4 потребует небольшого PR в ядро в первую очередь).

Здорово и спасибо за макет!

:wink:

Кроме того, этот коммит показывает, насколько просто другим темам/плагинам внедрить новую панель быстрого доступа!

12 лайков

Это просто фантастика! Насколько фантастика, спросите вы? Настолько, что я включил это на всех темах в Meta.

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

Шестерёнка :gear: — это не вкладка. Она кажется совершенно неуместной. Если её убрать, можно сделать все вкладки немного шире, что поможет увеличить области клика на мобильных устройствах! Я думаю, стоит попробовать её удалить.

Отображение «Выйти» на каждой вкладке, на мой взгляд, излишне. Зачем мы приглашаем людей выйти только потому, что они просматривают уведомления? Это выглядит как смешение ответственности.

Моя рекомендация следующая:

  1. Удалите «Выйти» со всех подвкладок.
  2. Добавьте новую вкладку при нажатии на person sam слева. На ней могут быть ссылки: «Обзор», «Активность», «Сообщения», «Настройки», «Выйти».

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

21 лайк

Это потрясающе!

Я заметил одну очень небольшую проблему — этот компонент, кажется, влияет на цвет иконок в других частях приложения (например, статус «просмотр»). Обычно иконка здесь синяя:

Мне нравится эта идея — как насчет того, чтобы добавить быстрый доступ к выбору темы и размеру шрифта прямо в «вкладку sam»? Меню-гамбургер здесь, на Meta, становится довольно загроможденным опциями тем, поэтому было бы здорово переместить их в другое место.

10 лайков

Мне это очень нравится. Отличная работа, всем участникам!

Огромное «за» — я думаю, это сделает плагин assign гораздо более полезным и практичным.

Мне тоже нравятся эти изменения, и я с нетерпением жду возможности их опробовать. Не уверен, что хорошая идея прятать «Выйти» за дополнительным кликом, но согласен, что сейчас оно выглядит неуместно, поэтому готов попробовать. Я собирался предложить разместить его там, где сейчас находится значок настроек, но согласен, что они там тоже выглядят неуместно, и нам не нужно поощрять людей выходить из системы каждый раз при переходе в это меню.

Также было бы здорово, если бы можно было добавить одну или две пользовательские ссылки внутри person sam. Мы используем SSO с WordPress, поэтому нам приходится перенаправлять пользователей в WordPress для редактирования профиля и настроек электронной почты.

6 лайков