Notifications - "unread only" filter

Хорошо, я сосредоточусь на этом в первую очередь и скоро пришлю вам макеты этой страницы :slightly_smiling_face:

5 лайков


Что думаете? @Johani, @jomaxro

4 лайка

О, мне нравится! Молодец.

4 лайка

Итак, это окончательно? Могу ли я начать над этим работать?

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

3 лайка

Конечно, я буду использовать select kit :slightly_smiling_face:

1 лайк

Я использовал select kit и создал PR.

11 лайков

Я завершил эту задачу, и мой pull request был слит.

7 лайков

@Ahmed_Gagan

Заметил одну вещь — сейчас он не подтягивает все непрочитанные уведомления. Фильтруется только первая загруженная «страница» уведомлений, но если у вас есть непрочитанные за более ранний период, нужно прокрутить страницу вниз во вкладке «Все» и только после этого переключиться на непрочитанные.

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

РЕДАКТИРОВАНИЕ.
Я только что обнаружил, возможно, ещё одну ошибку.
Когда вы нажимаете колесико мыши в этом представлении:

Ожидается, что тема откроется в новой вкладке, но вместо этого уведомление очищается, и ни одна новая вкладка не открывается.

4 лайка

Я проверю это ещё раз и вернусь к вам :slightly_smiling_face:

3 лайка

Очень рады, что это наконец запущено!

Спасибо всем, кто работал над этим с января 2016 года :heart_eyes:

4 лайка

Начал работу над этими изменениями, они будут отправлены в ближайшее время.

8 лайков

Я только что объединил PR от @Ahmed_Gagan для этого, и у меня всё отлично работает на Meta :heart_eyes:

В ближайшие пару часов я разверну это изменение на вашем сайте, @loginerror

9 лайков