search

GabovAleksander / practicum-android-diploma

0 stars 3 forks source link

Iteration 2 review #57

Closed GabovAleksander closed 1 month ago

GabovAleksander commented 1 month ago

Ссылка на таск-трекер: https://github.com/users/GabovAleksander/projects/1

Список выполненных задач:

  1. Добавить на экране поиска отображение нужных плейсхолдеров, в зависимости от типа ошибки

  2. Продолжить реализацию экрана "Поиск вакансий"

  3. Верстка экрана "Поиск вакансий"

  4. Реализовать логику экрана "Поиск вакансий"

  5. Реализовать логику экрана "Детали вакансии" - ViewModel, Fragment

  6. Верстка экрана "Детали вакансии"

  7. Реализовать логику экрана "Детали вакансии" - Data слой

  8. Реализовать возможность поделится вакансией

  9. Реализовать взаимодействие экрана деталей вакансии с model слоем.

  10. Реализовать правильное получение информации о деталях избранной вакансии в зависимости от наличия интернета.

  11. Верстка экрана "Избранные вакансии"

  12. Реализовать логику экрана "Избранные вакансии"

  13. разместить некоторые классы по своим местам

  14. Реализовать логику экрана "Избранные вакансии"- domain, data

Количество человек в команде: 5 Время на задачи: план 54ч, факт 82ч

AleksandrIlinskii commented 1 month ago

4 UI проблемы при любом запросе:

  1. Самая верхняя вакансия скрыта за плашкой с общим количеством вакансий
  2. Под названием вакансии должна отображаться компания, а не дублироваться название компании
  3. У иконки компании нет обводки
  4. У вас очень большие отступы справа и слева у элементов списка telegram-cloud-photo-size-4-5987631444161316013-y
AleksandrIlinskii commented 1 month ago

При пагинации индикатор загрузки должен быть под последней вакансией, а не на ней

https://github.com/user-attachments/assets/226a43dd-aac9-460c-a65e-74381bfc38e3

AleksandrIlinskii commented 1 month ago

Заголовок ошибки не отцентрован (он по левому краю)

telegram-cloud-photo-size-4-5987631444161316020-y

AleksandrIlinskii commented 1 month ago

Если в процессе загрузки новой страницы контента отсутствовал интернет или произошла ошибка, то индикатор загрузки должен исчезнуть и пользователь должен увидеть стандартный Toast с сообщением "Проверьте подключение к интернету" или "Произошла ошибка".

https://github.com/user-attachments/assets/8476f7c0-f490-4aae-b703-ba9edb205441

AleksandrIlinskii commented 1 month ago

Проверьте пожалуйста, какой используется цвет для белой плашки. Мне кажется она должна быть не такой белой, а серее telegram-cloud-photo-size-4-5987631444161316022-y

AleksandrIlinskii commented 1 month ago

При нажатии на иконку появляется Ripple effect, это стандартное поведение для андроида. Оно должно покрывать всю иконку, на которую нажимают telegram-cloud-photo-size-4-5987631444161316026-y