Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: Task 5 completed #5

Merged
merged 21 commits into from
Jul 22, 2024
Merged

feature: Task 5 completed #5

merged 21 commits into from
Jul 22, 2024

Conversation

Gribbirg
Copy link
Owner

@Gribbirg Gribbirg commented Jul 19, 2024

Пятое задание выполнено.

Сделано по ТЗ

Compose+

  • поддержка светлой и тёмной темы уже давно есть
  • цвета уже давно в кастомной AppTheme
  • для переопределения основных цветов дополнительно обернул в MaterialTheme
  • ссылки на типофграфию уже давно есть
  • ripple эффект на кнопки и checkbox идёт по умолчанию, примера ради добавил его на кнопку выбора важности
  • экран настроек добавлен (навигация через кнопку меню на главном экране со списком)

  • анимации переходов уже давно есть
  • вынес стили и цвета в отдельный модуль core/theme
  • шторка важности дела реализована (+ анимация на кнопки высокой важности)
  • снекбар при удалении дела также добавлен (кастомный с таймером)

DivKit

  • экран "о приложении" также доступен через меню
  • state и div-action используются для изменения состояния логотипа (изображения) приложения при нажатии на него

  • через кастомный DivActionHandler работает кнопка "Вернуться назад"

Дополнительно

  • добалены README на двух языках с информацией о приложении (в том числе ссылка на телегу и информация о сборке)
  • авторизация вынесена на экран настроек, но пока не до конца доделано (работает, но информации о текущем пользователе пока нет)
  • DivKit:
    • кастомный DivActionHandler обрабатывает нажатия на ссылки
    • поддержка цветовых тем и языков

@katrinSergeevna22
Copy link

Привет) Работа выполнена отлично)
Задание Compose+: Все выполнено, единственное заметила, что снекбар появляется не всегда при удалении.
По DivKit все отлично!

Copy link

@MAXBAF1 MAXBAF1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Привет, посмотрел проект, очень хорошо.

Единственный минус - при удалении дела и перехода после этого на экран добавления дела, snackbar исчезает, а при возращении обратно больше не появляется и возможности вернуть дело нет.

А ещё я думаю, пользователю лучше оставить возможность снять выполнение свайпом влево.

@Gribbirg Gribbirg merged commit 64ab2e7 into master Jul 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants