nm17/narodmon-app
1
1
Fork 0
Code Issues 8 Pull requests 1 Projects 1 Releases Packages Activity

Фикс #10 #15

Merged
nm17 merged 13 commits from issue-#10 into master 2023-06-22 11:18:11 +03:00
Conversation 0 Commits 13 Files changed 18 +467 -253
nm17 commented 2023-06-22 08:37:55 +03:00
Owner
Copy link
No description provided.
mezhendosina was assigned by nm17 2023-06-22 08:37:55 +03:00
nm17 added 12 commits 2023-06-22 08:37:56 +03:00
nm17 reviewed 2023-06-22 08:50:00 +03:00
nm17 left a comment
Author
Owner
Copy link

Блин, я не могу запросить изменения у самого себя.

Блин, я не могу запросить изменения у самого себя.
app/src/main/java/ru/nm17/narodmon/ui/navHost/MainScreenSealed.kt
@ -0,0 +4,4 @@
import androidx.annotation.StringRes
import ru.nm17.narodmon.R
sealed class MainScreenSealed(
nm17 commented 2023-06-22 08:40:22 +03:00
Author
Owner
Copy link

Интересное решение

Интересное решение
👍 1 ❤️ 1
DarkCat09 marked this conversation as resolved
app/src/main/java/ru/nm17/narodmon/ui/sensorsScreen/SensorsScreen.kt Outdated
@ -65,8 +54,7 @@ import ru.nm17.narodmon.ui.toChipTitle
@OptIn(ExperimentalMaterial3Api::class, ExperimentalAnimationApi::class)
@Composable
fun SensorsScreen(navController: NavController) {
nm17 commented 2023-06-22 08:45:23 +03:00
Author
Owner
Copy link

Также, надо понять нужно ли контроллер навигационный выносить за параметр. Возможно, это остатки моего кода, поэтому можешь это не исправлять.

Также, надо понять нужно ли контроллер навигационный выносить за параметр. Возможно, это остатки моего кода, поэтому можешь это не исправлять.
mezhendosina marked this conversation as resolved
app/src/main/java/ru/nm17/narodmon/ui/sensorsScreen/SensorsScreen.kt Outdated
@ -67,3 +56,2 @@
@Composable
fun SensorsScreen(navController: NavController) {
val coroutineScope = rememberCoroutineScope()
fun SensorsScreen(navController: NavController, onSettingsClick: () -> Unit) {
nm17 commented 2023-06-22 08:43:40 +03:00
Author
Owner
Copy link

Функция экрана датчиков становится оооочень большой. Я могу принять, но при условии что потом создашь отдельный ПР для рефакторинга этой функции, либо можешь исправить в этом пр.

Функция экрана датчиков становится оооочень большой. Я могу принять, но при условии что потом создашь отдельный ПР для рефакторинга этой функции, либо можешь исправить в этом пр.
mezhendosina commented 2023-06-22 09:10:58 +03:00
Collaborator
Copy link

Надо убрать onSettingsClick. Оно же тоже к навигатору обращаться в итоге будет

Надо убрать onSettingsClick. Оно же тоже к навигатору обращаться в итоге будет
mezhendosina marked this conversation as resolved
app/src/main/java/ru/nm17/narodmon/ui/webCamsScreen/WebCamsScreen.kt
@ -15,3 +14,3 @@
@Composable
fun WebCamsScreen() {
fun WebCamsScreen(navController: NavController) {
nm17 commented 2023-06-22 08:47:01 +03:00
Author
Owner
Copy link

Также тут под вопросом вынос контроллера.

Также тут под вопросом вынос контроллера.
DarkCat09 commented 2023-06-22 08:54:09 +03:00
Owner
Copy link

В студии подсвечивается как unused. Да, это в твоём коде экрана датчиков было, мы и копируем))

В студии подсвечивается как unused. Да, это в твоём коде экрана датчиков было, мы и копируем))
mezhendosina commented 2023-06-22 09:03:41 +03:00
Collaborator
Copy link

Он вынесен, чтобы можно было из веб-камер уйти в полноэкранное меню просмотра камеры

Он вынесен, чтобы можно было из веб-камер уйти в полноэкранное меню просмотра камеры
nm17 marked this conversation as resolved
app/src/main/res/drawable/ic_home.xml
@ -0,0 +1,5 @@
<vector android:height="24dp" android:tint="#000000"
android:viewportHeight="24" android:viewportWidth="24"
android:width="24dp" xmlns:android="http://schemas.android.com/apk/res/android">
<path android:fillColor="@android:color/white" android:pathData="M12,5.69l5,4.5V18h-2v-6H9v6H7v-7.81l5,-4.5M12,3L2,12h3v8h6v-6h2v6h6v-8h3L12,3z"/>
nm17 commented 2023-06-22 08:49:11 +03:00
Author
Owner
Copy link

А в Material Icon Library нет этих иконок? Было бы супер, если бы мы могли без своих обойтись

https://developer.android.com/jetpack/compose/graphics/images/material
https://developer.android.com/reference/kotlin/androidx/compose/material/icons/package-summary

А в Material Icon Library нет этих иконок? Было бы супер, если бы мы могли без своих обойтись https://developer.android.com/jetpack/compose/graphics/images/material https://developer.android.com/reference/kotlin/androidx/compose/material/icons/package-summary
DarkCat09 commented 2023-06-22 09:00:13 +03:00
Owner
Copy link

А в Material Icon Library нет этих иконок? Было бы супер, если бы мы могли без своих обойтись

Официальный список — https://fonts.google.com/icons
При выборе иконки там даже есть код для Compose.

Сейчас посмотрел, все нужные иконки имеются.

> А в Material Icon Library нет этих иконок? Было бы супер, если бы мы могли без своих обойтись Официальный список — https://fonts.google.com/icons При выборе иконки там даже есть код для Compose. Сейчас посмотрел, все нужные иконки имеются.
mezhendosina commented 2023-06-22 09:00:53 +03:00
Collaborator
Copy link

Дак я оттуда и брал))0)

Дак я оттуда и брал))0)
DarkCat09 commented 2023-06-22 09:22:23 +03:00
Owner
Copy link

Дак я оттуда и брал))0)

А, вижу. Раньше была вкладка с Jetpack Compose, убрали походу.

Вот так надо: https://developer.android.com/reference/kotlin/androidx/compose/material/icons/Icons.Outlined

Icons.Outlined.Home

вместо использования своей SVGшки.

Там нет как минимум иконки камеры

Прям во всех стилях (Filled, Outlined, Rounded) нету? Проверить сейчас негде.
Если так, то печально.
Можем посмотреть альтернативные библиотеки для Compose с теми же материал-диазайновыми иконками.

> Дак я оттуда и брал))0) А, вижу. Раньше была вкладка с Jetpack Compose, убрали походу. Вот так надо: https://developer.android.com/reference/kotlin/androidx/compose/material/icons/Icons.Outlined ```kotlin Icons.Outlined.Home ``` вместо использования своей SVGшки. > Там нет как минимум иконки камеры Прям во всех стилях (Filled, Outlined, Rounded) нету? Проверить сейчас негде. Если так, то печально. Можем посмотреть альтернативные библиотеки для Compose с теми же материал-диазайновыми иконками.
nm17 marked this conversation as resolved
nm17 changed title from WIP: Фикс #10 to Фикс #10 2023-06-22 09:05:05 +03:00
mezhendosina added 1 commit 2023-06-22 09:27:13 +03:00
nm17 merged commit 0a6c3e703e into master 2023-06-22 11:18:11 +03:00
Sign in to join this conversation.
Reviewers
No reviewers
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
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
mezhendosina
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: nm17/narodmon-app#15
No description provided.
Delete branch "issue-#10"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?