feat: Добавил фичу "Редактирование выбранного сообщения"#61
Conversation
d7a4cd1 to
6985c9f
Compare
Kozlov-AE
left a comment
There was a problem hiding this comment.
Не нашел изменений в модели сообщения в БД.
Присутствует неиспользуемый код.
В остальном вроде все ок)
Kozlov-AE
left a comment
There was a problem hiding this comment.
Все ок, комменты отправил в ЛС в телегу.
Kozlov-AE
left a comment
There was a problem hiding this comment.
Вроде все ок. Ток забыл удалить лишние методы из мэйнвьюмодел. Ну и у павлв по поводу ревизий таблицы сообщений поинтересуйся (я думаю можно убрать из БД поле "редактировано"
Kibnet
left a comment
There was a problem hiding this comment.
По работе есть замечания:
- В контекстном меню на сообщении есть кнопка "цитировать", она не работает, поэтому она там не нужна.
- Контекстное меню вызывается только при попадании в центр сообщения, а зона в несколько пикселей по границе не реагирует, видимо нужно переместить контекстное меню выше на уровень или сколько из там.
- При изменении сообщения, меняется и время, а это неочевидно. Время это мы конечно должны фиксировать и показывать потом даже будем при наведении на карандаш. То есть отображаемое время должно остаться исходное.
- При переходе в режим редактирования должен отображаться карандаш слева от текста в поле ввода, чтобы понять что мы редактируем, а не пишем новое сообщение.
В целом работает круто!
67c37a5 to
698c6a3
Compare
Kibnet
left a comment
There was a problem hiding this comment.
Уже очень хорошо. Есть несколько мелких замечаний.
| { | ||
| idEditMessage = message.Id; | ||
| MessageText = message.Text; | ||
| message.Selected = false; |
There was a problem hiding this comment.
Кажется логичным делать это поле false только при выходе из режима редактирования, а не при входе в режим. Тогда на основе этого поля можно будет визуальное выделение сделать в списке сообщений.
SkillChat.Server/Hubs/ChatHub.cs
Outdated
| Id = hubEditedMessage.Id, | ||
| Message = mes.Text, | ||
| EditedPostTime = mes.EditedPostTime, | ||
| Edited = true, |
There was a problem hiding this comment.
Тут тоже напрашивается автомаппер
698c6a3 to
630cf9c
Compare
8669e14 to
8478450
Compare
8478450 to
f12add5
Compare
Можно также редактировать последнее сообщение нажав на клавишу «вверх» при выделенном окне ввода.
Единственное, что у меня совершенно не получился дизайн меню. Не могу скруглить его, а также добавить цвет выделения MenuItem.
Еще одна проблема связанна с выделением своего сообщения. В авалони существует стандартное ContextMenu для TextBox.
Я скрыл его, чтобы оно не вылезало, но тогда меню которое я вызываю из Stackpanel можно вызвать только нажатием правой кнопки мышки на любом краю сообщения, а по центру не работает.