SUAI-TaskPlanner-Contest / TaskPlanner

Client application for working with todos and syncing with CalDAV servers
MIT License
3 stars 2 forks source link

66 create warning window #74

Closed pavelparfishov closed 1 year ago

pavelparfishov commented 1 year ago

Task merge request

Related Issue | [Создать окно предупреждений #66](https://github.com/SUAI-TaskPlanner-Contest/TaskPlanner/issues/66) -- | -- Квалификация ревьюера | Знание требований заказчика Исполнители: @pavelparfishov

Code changes

Был написан QML-код для универсального окна ошибок. Также были написаны конструкции на Python для вывода текста ошибки в окно. Информация об ошибках хранится в словаре. Еще были добавлены обработчики нажатия на кнопки и изменение цвета кнопки при нажатии для визуального удобства.

UI Demonstration

![image](https://github.com/SUAI-TaskPlanner-Contest/TaskPlanner/assets/78814540/1adadb27-5b0f-4fa3-a2b3-01caa83a00fc)

How to test

Проверить наличие необходимых полей и визуальную привлекательность окна. Также проверить корректность конструкций на Python

pavelparfishov commented 1 year ago

@astronik00 Привет. Добавил обработку нажатий на кнопки и изменение цвета кнопок при нажатии и наведении курсора на кнопки.

astronik00 commented 1 year ago

Убрать системное окно: https://evileg.com/ru/post/280/

astronik00 commented 1 year ago

Вы удаляете задачу, у которой есть подзадачи!

В принципе по действию это и так понятно. Смысл в том, что необходимо предупредить об удалении всех подзадач. Например: "У этой задачи есть подзадачи. В случае ее удаления все подзадачи будут также удалены". Выбор: "продолжить" и "отмена"

Вы выходите из окна редактирования при наличии несохраненных данных!

Я думаю лучше: "В случае закрытия этого окна, все несохраненные данные будут утеряны" и тоже поменять на "продолжить" и "отмена"

Вы удаляете сервер, у которого есть задачи!

"У этого сервера есть задачи. В случае удаления сервера все задачи будут также удалены". Выбор: "продолжить" и "отмена"

в общем поменьше экспрессии

pavelparfishov commented 1 year ago

@astronik00 Убрал системное окно. Изменил тексты.

astronik00 commented 1 year ago

Отлично. Давай теперь шрифты одинаковые сделаем. Попробуй шрифт Inter, я тебе скину позже, как он подключается. Сейчас узнаю это у Ярослава как раз.

pavelparfishov commented 1 year ago

@astronik00 Поменял шрифты, цвета. Добавил больше закруглений:

image

astronik00 commented 1 year ago

@astronik00 Поменял шрифты, цвета. Добавил больше закруглений:

image

Отлично выглядит. Думаю кнопки надо побольше в высоту сделать и можно мерджить

pavelparfishov commented 1 year ago

@astronik00 Увеличил кнопки

image