feature: Task 6 completed

[Gribbirg]

Шестое задание выполнено.

Сделано по ТЗ

Accessibility

• теоретически всё можно сделать даже без глаз)

• дела я объединял в один элемент, действия (выполнить, удалить и т.д.) доступны с помощью Actions
• обновление списка также через Actions на любом элементе

Автотесты

• unit тесты я писал на всё подряд, так что они есть практически для всего
• также есть пара скромных ui тестов (из примера и удаление дела)
• код не самый красивый местами, но работает и слава богу)

[arekalov]

Привет, посмотрел работу, вот некоторые замечания

Asseccebility:

1. Что то пошло не так и все действия с делом не работают(
2. nit: Было бы круто переопределить описание click'a у кнопок toolBara, чтобы действие кнопок были понятны для пользователя (например для кнопки меню воспроизводить не коснитесь дважды, чтобы активировать, а коснитесь дважды чтобы перейти в меню приложения; также с глазиком, только там еще стейт меняется).
3. nit: считаю что фокус на кнопку добавления нового дела должен попадать раньше, чем на список дел, чтобы пользоваться было удобнее.
4. nit: кнопку добавить дело (которая в конце дела) стоит сделать невидимой для talkBack.
5. На экране редактирования не очень понятные contentDescription (просто текст) для полей выбора дела, также стоит переопределить описание click'a.
6. Свитчер озвучивается неправильно.
7. Описание кнопки удалить озвучивается дважды.

Testing:

1. UI тесты не запускаются вместе, проходит только addItem. При запуске отдельно - проходят.
2. UI тесты есть, но покрытие неполное: не протыкивается выбор дедлайна, приоритета.

Итог:

По доступности замечания есть, но не такие значительные, тяжело учесть все случаи. В тестах большой респект, за большой coverage. В целом работа очень достойная.

[arekalov]

задержка избыточна, тк следующей строкой дожидаешься окончания всех корутин. ниже также

[arekalov]

CoroutineExceptionHandler не покрыт тестами

[Gribbirg]

Привет, посмотрел работу, вот некоторые замечания

Asseccebility:

1. Что то пошло не так и все действия с делом не работают(
2. nit: Было бы круто переопределить описание click'a у кнопок toolBara, чтобы действие кнопок были понятны для пользователя (например для кнопки меню воспроизводить не коснитесь дважды, чтобы активировать, а коснитесь дважды чтобы перейти в меню приложения; также с глазиком, только там еще стейт меняется).
3. nit: считаю что фокус на кнопку добавления нового дела должен попадать раньше, чем на список дел, чтобы пользоваться было удобнее.
4. nit: кнопку добавить дело (которая в конце дела) стоит сделать невидимой для talkBack.
5. На экране редактирования не очень понятные contentDescription (просто текст) для полей выбора дела, также стоит переопределить описание click'a.
6. Свитчер озвучивается неправильно.
7. Описание кнопки удалить озвучивается дважды.

Testing:

1. UI тесты не запускаются вместе, проходит только addItem. При запуске отдельно - проходят.
2. UI тесты есть, но покрытие неполное: не протыкивается выбор дедлайна, приоритета.

Итог:

По доступности замечания есть, но не такие значительные, тяжело учесть все случаи. В тестах большой респект, за большой coverage. В целом работа очень достойная.

Привет!

Спасибо за ревью!

По поводу действий с делом: не знаю, почему у тебя не заработало, у меня всё нормально, вот видео:

VID_20240728_203114.mp4

Напиши пожалуйста, на чём у тебя не заработало, я постараюсь разобраться)

[arekalov]

Привет, посмотрел работу, вот некоторые замечания

Asseccebility:

1. Что то пошло не так и все действия с делом не работают(
2. nit: Было бы круто переопределить описание click'a у кнопок toolBara, чтобы действие кнопок были понятны для пользователя (например для кнопки меню воспроизводить не коснитесь дважды, чтобы активировать, а коснитесь дважды чтобы перейти в меню приложения; также с глазиком, только там еще стейт меняется).
3. nit: считаю что фокус на кнопку добавления нового дела должен попадать раньше, чем на список дел, чтобы пользоваться было удобнее.
4. nit: кнопку добавить дело (которая в конце дела) стоит сделать невидимой для talkBack.
5. На экране редактирования не очень понятные contentDescription (просто текст) для полей выбора дела, также стоит переопределить описание click'a.
6. Свитчер озвучивается неправильно.
7. Описание кнопки удалить озвучивается дважды.

Testing:

1. UI тесты не запускаются вместе, проходит только addItem. При запуске отдельно - проходят.
2. UI тесты есть, но покрытие неполное: не протыкивается выбор дедлайна, приоритета.

Итог:

По доступности замечания есть, но не такие значительные, тяжело учесть все случаи. В тестах большой респект, за большой coverage. В целом работа очень достойная.

Привет!

Спасибо за ревью!

По поводу действий с делом: не знаю, почему у тебя не заработало, у меня всё нормально, вот видео:

VID_20240728_203114.mp4

Напиши пожалуйста, на чём у тебя не заработало, я постараюсь разобраться)

Запускал на 11 Андроиде, miui 14.
Но я не снижал за это, потому что по коду там все гладко)