agzam/vyatsu-schedule
2
2
Fork 1
Code Issues 3 Pull Requests 1 Actions Packages Projects 1 Releases 15 Wiki Activity

New materail3 UI #12

Merged
Agzamus merged 4 commits from parzival-2077/vyatsu-schedule:feature/material3-ui into material3-ui 2026-10-22 02:24:58 +03:00
Conversation 0 Commits 4 Files Changed 32
+2188 -103
parzival-2077 commented 2026-10-20 21:06:45 +03:00
Contributor
Copy Link
Copy Source

ура

ура
Agzamus reviewed 2026-10-22 02:04:32 +03:00
app/src/main/java/agzam/vyatsu/ui/home/HomeScreen.kt
@@ -63,0 +79,4 @@
HomeTab.SETTINGS -> {
if (tab == t) painterResource(R.drawable.settings_24dp_fill)
else painterResource(R.drawable.settings_24dp_outline)
}
Agzamus commented 2026-10-22 02:04:32 +03:00
Owner
Copy Link
Copy Source

[WARN] Спам if-ами, лучше вынести в enum

[WARN] Спам `if`-ами, лучше вынести в `enum`
Agzamus reviewed 2026-10-22 02:09:21 +03:00
app/src/main/java/agzam/vyatsu/ui/picker/PickerScreen.kt
@@ -57,0 +121,4 @@
val onRetry: () -> Unit = when (segment) {
PickerSegment.GROUPS -> viewModel::retryGroups
PickerSegment.TEACHERS -> viewModel::retryTeachers
}
Agzamus commented 2026-10-22 02:09:21 +03:00
Owner
Copy Link
Copy Source

[WARN] Спам условиями, PickerSegment является enum-ом - лучше часть впихнуть в него (далее по коду тоже есть подобные спамы)

[WARN] Спам условиями, `PickerSegment` является `enum`-ом - лучше часть впихнуть в него (далее по коду тоже есть подобные спамы)
Agzamus reviewed 2026-10-22 02:13:15 +03:00
app/src/main/java/agzam/vyatsu/ui/schedule/components/ScheduleDateHeader.kt
@@ -0,0 +63,4 @@
// }?.weekStart
// },
// onSelect = onSelectWeek
// )
Agzamus commented 2026-10-22 02:13:15 +03:00
Owner
Copy Link
Copy Source

[WARN] Закоменченый код

[WARN] Закоменченый код
Agzamus reviewed 2026-10-22 02:19:24 +03:00
app/src/main/java/agzam/vyatsu/ui/teacher/TeacherScheduleScreen.kt
@@ -43,0 +256,4 @@
@Composable
fun ScheduleEmptyDay() {
TODO("Not yet implemented")
}
Agzamus commented 2026-10-22 02:19:24 +03:00
Owner
Copy Link
Copy Source

[WARN] Для TODO надо оставить комментарий вида "TODO: xxx"

[WARN] Для TODO надо оставить комментарий вида "TODO: xxx"
Agzamus approved these changes 2026-10-22 02:21:08 +03:00
Agzamus left a comment
Owner
Copy Link
Copy Source

Добавил в комментариях варнинги чего лучше исправить и послать отдельный PR

Добавил в комментариях варнинги чего лучше исправить и послать отдельный PR
Agzamus changed target branch from main to material3-ui 2026-10-22 02:24:39 +03:00
Agzamus added 4 commits 2026-10-22 02:24:39 +03:00
New materail3 UI 4fb1e3edca
some update e1019b18d3
Some update UI 2 2a4e010b21
Обновить app/build.gradle.kts 8f117c2fbe
Agzamus merged commit f01cab2826 into material3-ui 2026-10-22 02:24:58 +03:00
Sign in to join this conversation.
Reviewers
No Reviewers
Agzamus
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Agzamus (Арасланов Агзам) vantablackTea
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: agzam/vyatsu-schedule#12