Проверка ДЗ 1

[qwert21ant]

Чистота кода и общая структура проекта

• README (1/1): всё ок, но лучше скрины как-нибудь сгруппировать в меньшее количество изображений, а то они слишком большие
• Форматирование кода (1/1)
• Код разбит на фичи и слои (2/2) Ок, только вроде папка assets должна лежать выше

Вёрстка + навигация

• Полностью свёрстаны 2 экрана по макетам (5/10): Ну тут дофига)
  • Начнём с AppBar'а: Он должен раскрываться только когда пролистали в самый вверх, скорее всего это лишнее: https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/screens/main/widgets/app_bar.dart#L74-L76
  • Иконка глаза должна быть одного размера, и должна находиться на одной и той же вертикальной оси при сворачивании
  • У списка слишком большой горизонтальный padding: https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/screens/main/widgets/tasks_list.dart#L18
  • У Card почему-то отсутствует тень. Вероятно она где-то в main.dart в темах потерялась https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/screens/main/widgets/tasks_list.dart#L19
  • Вероятно лучше было использовать ListView, а ещё лучше SliverList https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/screens/main/widgets/tasks_list.dart#L23
  • Про task items:
  • Для важности "Высокий" должно отображаться 2 восклицательных знака Можно использовать TextSpan для этого
  • Последний айтем логичнее было бы сделать полностью кликабельным, вместо двух отдельных кнопок
  • Второй экран
  • Тоже следует уменьшить внешние отступы: https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/screens/save_task/form/text_form.dart#L19
  • И увеличить внутренние горизонтальные) https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/screens/save_task/form/text_form.dart#L23
  • Выбор важности сделан немного не так как в макете, можно было использовать параметр selectedItemBuilder для задания того, как будет выглядеть выбранный айтем и там уже склеить строку "Важность" и выбранную важность
  • На счёт интуитивности: кому как) https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/screens/save_task/form/priority_form.dart#L64
  • Тут тоже логичнее сделать весь тайл кликабельным
• Свайп по айтему Выполнено/Удалить с логикой (3/3) Логика есть, ошибки по вёрстке:
  • При dismiss'е содержимое не обрезается: Можно обернуть Dismissible в ClipRect Также стоит добавить отступы по краям у иконок в background'е Dismissible'а
  • При dismiss'е первого элемента скругление пропадает: Можно обернуть первый айтем в ClipRRect
• Показ выполненных дел с логикой (3/3) Тут всё ок

Разное

• Реализовано нормальное логирование (2/2) Ну оно и вправду есть)
• Есть человеческая иконка у приложения (4/4) Имеется, однако почему-то в asset'ах лежит одно, а в нативе android'а другое

Доп

• Навигацию переделай с pushReplacementNamed на обычные пуши и попы, т. к. если нажать кнопку назад на втором экране, то приложение закроется
• TextButton какие-то узкие все

• Не думаю что игнорировать @immutable и @Deprecated хорошая идея

  Swedish? https://github.com/LilyKras/toDoApp/blob/bbe1c1b3dc11515bf9ed54ecdde149186806e8ce/lib/main.dart#L37-L38

  P.S. на мобилке эти все ошибки тоже есть

[MeTpali]

К предыдущему комментарию даже добавить нечего. Отмечу только пару моментов, что вместо Card на мой взгляд лучше использовать Material, потому что у Card другие задачи, и вместо DropdownButtonFormField использовать PopupMenuButton, чтобы полностью макету соответствовало.

[LilyKras]

Спасибо за проверку! А почему вы снижаете за критерий свайпа за верстку, если за верстку вы уже снизили в предыдущем критерии?

[qwert21ant]

Ну я спросил у своего ментора, он сказал отнести эти проблемы к пункту со свайпом

[MeTpali]

Кстати в критериях про свайп реально написано только про логику и ничего про ui