Skip to content

Удаление выбранных сообщений и очистка чата для себя#95

Merged
Kibnet merged 13 commits intomasterfrom
feature/#12-Clearing-chat-history-for-yourself
Oct 13, 2021
Merged

Удаление выбранных сообщений и очистка чата для себя#95
Kibnet merged 13 commits intomasterfrom
feature/#12-Clearing-chat-history-for-yourself

Conversation

@Mileeena
Copy link
Copy Markdown
Collaborator

No description provided.

@Mileeena Mileeena linked an issue Sep 19, 2021 that may be closed by this pull request
4 tasks
@Mileeena Mileeena force-pushed the feature/#12-Clearing-chat-history-for-yourself branch from 791421b to 0184b05 Compare October 1, 2021 14:44
@EvgeniyLyapunov EvgeniyLyapunov linked an issue Oct 1, 2021 that may be closed by this pull request
3 tasks
EvgeniyLyapunov and others added 5 commits October 3, 2021 07:23
 - написал логику "удаления сообщений для себя" на клиенте и сервере;
 - отредактировал контекст меню по кнопке "...";
 - добавил кнопку "удалить выбранные сообщения" в панель режима выбора сообщений;
- пока не закрывается автоматически окно поддтверждения после нажатия кнопки удалить, не успел
@Mileeena Mileeena changed the title Очистка истории чата для себя Удаление выбранных сообщений и очистка чата для себя Oct 4, 2021
UnluckyD
UnluckyD previously approved these changes Oct 4, 2021
@KlinskiyDD
Copy link
Copy Markdown
Collaborator

У вас вот эта штука не работает при нажатии на корзину
https://puu.sh/Igcsf/40670f642b.png

Посмотрите в кнопке "копировать". Я в ней реализовал точно такое же поведение.
https://puu.sh/IgctT/58bddd17f6.png

https://puu.sh/IgcuP/3769d1fba0.png

KlinskiyDD
KlinskiyDD previously approved these changes Oct 5, 2021
Copy link
Copy Markdown
Collaborator

@KlinskiyDD KlinskiyDD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вы молодцы парни!

@EvgeniyLyapunov
Copy link
Copy Markdown
Collaborator

У вас вот эта штука не работает при нажатии на корзину https://puu.sh/Igcsf/40670f642b.png

Посмотрите в кнопке "копировать". Я в ней реализовал точно такое же поведение. https://puu.sh/IgctT/58bddd17f6.png

https://puu.sh/IgcuP/3769d1fba0.png

Сделал, теперь корзина становится полностью серой в момент нажатия на неё, как и иконка "Копировать в буфер обмена"

- Добавил эффект на нажатие на корзину;
- Дописал логику кнопки "отмена" в режиме выборочного удаления сообщений;
- Переименовал метод
@EvgeniyLyapunov EvgeniyLyapunov dismissed stale reviews from KlinskiyDD and UnluckyD via f976894 October 5, 2021 18:34
UnluckyD
UnluckyD previously approved these changes Oct 6, 2021
KlinskiyDD
KlinskiyDD previously approved these changes Oct 6, 2021
Copy link
Copy Markdown
Contributor

@Kibnet Kibnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Выглядит и работает офигенно! Но в коде есть ошибки, лучше исправить.

@Mileeena Mileeena requested a review from Kibnet October 8, 2021 09:08
Kibnet
Kibnet previously approved these changes Oct 10, 2021
Copy link
Copy Markdown
Contributor

@Kibnet Kibnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Очень достойно! Осталось пару некритичных замечаний.

Copy link
Copy Markdown
Contributor

@Kibnet Kibnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Идеально! Работает как надо, код аккуратный.

@Kibnet Kibnet merged commit 3ac98a0 into master Oct 13, 2021
@Kibnet Kibnet deleted the feature/#12-Clearing-chat-history-for-yourself branch March 27, 2026 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Выборочная очистка истории чата для себя Очистка истории чата для себя

5 participants