Conversation
4 tasks
bb737f1 to
0a0b86b
Compare
7e4a237 to
7c20609
Compare
7c20609 to
90612b2
Compare
UnluckyD
previously approved these changes
Sep 23, 2021
90612b2 to
6e521f4
Compare
Kibnet
requested changes
Sep 24, 2021
Contributor
Kibnet
left a comment
There was a problem hiding this comment.
- Во время выбора сообщений остаются доступны редактирование и ответ на сообщение, из-за чего панель может быть увеличена по высоте. Эти команды должны быть недоступны пока включен режим выбора сообщений
- Порядок выбора сообщений влияет на порядок скопированных сообщений, порядок должен быть как в окне сообщений
Collaborator
|
Kibnet
requested changes
Sep 27, 2021
Contributor
Kibnet
left a comment
There was a problem hiding this comment.
Уже круто, осталось исправить 3 замечания и можем вливать
Логику для выхода из режима выбора не успел написать
Написал метод, который убирет флаги у сообщений, если мы нажимаем кнопку отмены. (надо переименовать или переписать его) Добавил метод выбора сообщений в SettingsViewModel
492a8d3 to
0675f76
Compare
Теперь при уменьшении размера окна, кнопки ведут себя адекватно Обновил дизайн
0675f76 to
275c3ac
Compare
Случайно обнаружил, что в результате каких-то других правок в главном окне, сместились надписи над полями ввода на странице Регистрации
EvgeniyLyapunov
previously approved these changes
Sep 30, 2021
Collaborator
EvgeniyLyapunov
left a comment
There was a problem hiding this comment.
Работа над замечаниями произведена.
Kibnet
previously approved these changes
Sep 30, 2021
| <WrapPanel Grid.Column="0"> | ||
| <Button HorizontalAlignment="Left" | ||
| Command="{Binding SelectMessagesMode.CopyToClipboardCommand}" | ||
| Classes="SelectMessageBorder CopyToClipboard" ToolTip.Tip="Скопировать в буфер обмена"> |
Contributor
There was a problem hiding this comment.
Тултип работает нестабильно. Или это только у меня?
d3d9e59
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Сверстал начальный дизайн для фичи "Выбор сообщений"