vas3k.club icon indicating copy to clipboard operation
vas3k.club copied to clipboard

Фича: редактировать и удалить свои комментарии

Open hudochenkov opened this issue 4 years ago • 8 comments

Добавить возможность удалить свой комментарий.

Добавить редактирование своих комментариев. Клуб может же быть лучше, чем твиттер :)

hudochenkov avatar Jun 13 '20 17:06 hudochenkov

А вот такого у тебя нет?

Screenshot 2020-06-13 at 19 12 19

vas3k avatar Jun 13 '20 17:06 vas3k

Нету :(

Screen Shot 2020-06-13 at 19 15 37

https://vas3k.club/post/1494/#comment-6e5338f5-2cd0-4a14-947a-acd93be4e46f

hudochenkov avatar Jun 13 '20 17:06 hudochenkov

Комментарии можно удалять или редактировать только первые 3 часа. Потом они становятся вечными. Этот создан 3 дня назад.

vas3k avatar Jun 13 '20 17:06 vas3k

Аааа! Это объясняет. Заметил, что в батле я вижу эти иконки (коммент старше трех часов). Они, наверное, показываются только в комментах первого уровня, а в реплаях их нет.

hudochenkov avatar Jun 13 '20 17:06 hudochenkov

В реплаях тоже есть, по ховеру: image

Konfuze avatar Jun 13 '20 21:06 Konfuze

Из кода теперь ясно.

Обычный коммент:

https://github.com/vas3k/vas3k.club/blob/f2707099359ac53d989e178f832b416e0420bcf9/frontend/html/comments/types/normal.html#L45-L51

Реплай проверяет is_editable:

https://github.com/vas3k/vas3k.club/blob/f2707099359ac53d989e178f832b416e0420bcf9/frontend/html/comments/types/reply.html#L32-L37

is_editable как раз то, про что Вастрик говорил выше.

https://github.com/vas3k/vas3k.club/blob/f2707099359ac53d989e178f832b416e0420bcf9/comments/models.py#L97-L98

Я бы сделал поведение для всех комментариев одинаковым, или использовать везде is_editable или нет. Чтобы UI был предсказуемым и консистентным.

hudochenkov avatar Jun 13 '20 22:06 hudochenkov

Да, эта функция была придумана позже, так что хорошо было бы везде на нее переехать. Можешь сделать :)

vas3k avatar Jun 14 '20 10:06 vas3k

Имхо это слишком грубая обратная связь, я даже на секунду поверил, что это я виноват и что-то делаю не так)) image Достаточно просто глушить кнопку и при наведении показывать тултип с объяснением, почему нельзя редактировать коммент

Takheer avatar Mar 10 '21 10:03 Takheer