Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Highlight comments that is last, in the user comments list / Подсветка в списке комментариев тех, которые являются последними #590

Open
paul-k-pastvu opened this issue May 2, 2023 · 8 comments · Fixed by #660
Labels
Comments All issues related with a user comments Feature Request All feature requests good first issue All tasks for good start by new contributors Moderation Issues and tools related with a moderators and moderation Priority: Low UI All features with changing of User Interface UX/Design Everything related to UX or Design

Comments

@paul-k-pastvu
Copy link

paul-k-pastvu commented May 2, 2023

EN: Highlight comments that is last, in the user comments list. Useful for moderators for meaning, comment was replied or not.
RU: Подсветка в списке комментариев пользователя тех, которые являются последними по времени (в данной новости или под данным изображением). Это нужно модераторам, чтобы, глядя на список, было легче понять, отвечали ли пользователю, или нет.

(По мотивам обработки ленты комментариев участницы https://pastvu.com/u/prababushka/comments/11)

Лента (список) комментариев пользователя

Под "последним" комментарием имеется в виду не последний от этого пользователя, а в принципе последний в соответствующем обсуждении.

Т.е. хочется глядя на ленту комментариев пользователя уметь понимать, писал ли после него кто-то ещё что-нибудь в каждом затронутом обсуждении (под каждым снимком, в каждой новости) - не важно, в ответ пользователю, или в других ветках, - или вот на этих репликах пользователя всё и остановилось.

При этом, если комментарий не последний, именно ответов на комментарий пользователя не было, можно дополнительно делать текстовую пометку "(нет ответов)"

Отдельные обсуждения (под изображениями, новостями)

То же цветовое выделение для самого последнего комментария в этом обсуждении (перенесено в #630)

Режим доступа

модераторы, админы

@paul-k-pastvu paul-k-pastvu added Feature Request All feature requests UI All features with changing of User Interface Priority: Low Moderation Issues and tools related with a moderators and moderation Comments All issues related with a user comments labels May 2, 2023
@kabalin
Copy link
Member

kabalin commented Aug 30, 2023

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

@paul-k-pastvu
Copy link
Author

Да, всё ровно так, никаких уточнений ;)

@kabalin
Copy link
Member

kabalin commented Nov 4, 2023

@paul-k-pastvu такая цветовая схема подойдет? Если нет, жду предложений как выделить комментарий.

image

@paul-k-pastvu
Copy link
Author

по цвету - ок. По сути задачи я дополнил постановку, раз оказалось, что было немного невнятно.

@kabalin
Copy link
Member

kabalin commented Nov 5, 2023

Подсветка в дереве вынесена в отдельную задачу #630

@kabalin
Copy link
Member

kabalin commented Dec 1, 2023

По оригинальной задаче (подсветка в списке комментариев пользователя): проблема в том, что запрос будет слишком затратный (он и так медленно обрабатывается для пользователей с большим числом комментариев) - в обычной ситуации мы выбираем комментарии по пользователю, если нам нужно знать последний ли это комментарий в данной сущности (фото или новость), нужно будет еще подтягивать все комментарии по всей сущности и вычислять.

При этом, если комментарий не последний, именно ответов на комментарий пользователя не было, можно дополнительно делать текстовую пометку "(нет ответов)"

Это единственное, что можно сделать не очень затратно (агрегировать запрос, есть ли хотя бы один дочерний комментарий). Смысла ограничивать модератору нет (это еще один запрос - является ли пользователь модератором снимка где оставлен комментарий), ограничить можно по глобальному свойству (администратор или зарегистрированный в системе).

Пример:
image

@paul-k-pastvu
Copy link
Author

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

  • Текст "Ждёт ответа"
  • Режим доступа: зарегистрированные пользователи

@kabalin
Copy link
Member

kabalin commented Jan 8, 2024

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

@kabalin kabalin reopened this Jan 8, 2024
@kabalin kabalin added the good first issue All tasks for good start by new contributors label Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Comments All issues related with a user comments Feature Request All feature requests good first issue All tasks for good start by new contributors Moderation Issues and tools related with a moderators and moderation Priority: Low UI All features with changing of User Interface UX/Design Everything related to UX or Design
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants