Conversation
|
Рекомендация: |
| android:orientation="vertical" | ||
| android:padding="@dimen/fab_margin"> | ||
|
|
||
| <TextView |
There was a problem hiding this comment.
Рекомендация:
В качестве заголовка экрана лучше использовать MaterialToolbar. Это стандартный компонент. В него можно передать текст заголовка с помощью атрибута title. Характеристики текста можно задать через стиль и атрибута titleTextAppearance.
| android:textColor="@color/white" | ||
| android:textSize="@dimen/heading_text" /> | ||
|
|
||
| <LinearLayout |
There was a problem hiding this comment.
Рекомендация:
На мой взгляд этот LinearLayout лишний, попробуй переверстать экран только с одним корневым LinearLayout.
| android:id="@+id/main" | ||
| android:paddingTop="10dp"> | ||
|
|
||
| <LinearLayout |
There was a problem hiding this comment.
Рекомендация:
В качестве заголовка экрана лучше использовать MaterialToolbar. Это стандартный компонент. В него можно передать текст заголовка с помощью атрибута title, иконку(стрелочка влево) с помощью атрибута app:navigationIcon и обработать нажатие с помощью setNavigationOnClickListener {}. Характеристики текста можно задать через стиль и атрибута titleTextAppearance.
|
|
||
| </LinearLayout> | ||
|
|
||
| <LinearLayout |
There was a problem hiding this comment.
Рекомендация:
На мой взгляд этот LinearLayout лишний, попробуй переверстать экран только с одним корневым LinearLayout.
| android:layout_marginTop="40dp" | ||
| android:orientation="vertical"> | ||
|
|
||
| <LinearLayout |
There was a problem hiding this comment.
Рекомендация к конструкции
<LinearLayout>
<TextView/>
<SwitchCompat/>
</LinearLayout>
Во-первых, это излишняя вложенность, которая негативно сказывается при отрисовке фрейма.
Во-вторых, она вызывает трудности с понимание того как свёрстан экран;
В-третьих, проще формировать и применять стили.
<com.google.android.material.switchmaterial.SwitchMaterial
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:paddingHorizontal="16dp"
android:paddingVertical="20dp"
android:text="@string/button_settings_dark_mode"
app:thumbTint="..." // Нужен для стилизации свичера
app:trackTint="..." // Нужен для стилизации свичера
android:textSize="16sp" />
Итого иерархия всего layout-а будет вот такой:
<LinearLayout>
<MaterialToolbar/>
< SwitchCompat или SwitchMaterial/>
<MaterialButton/>
<MaterialButton/>
<MaterialButton/>
<LinearLayout/>
|
Исправил критические ошибки |
|
И так, теперь точно всё должно быть верно. Я изменил вертикальные отступы экрана "Настройки" согласно макетам в фигме, перекрасил переключатель на тёмную тему согласно фигме, ну и там помимо этого 9 спринт лежит. |
qu1eres
changed the title
2 participants
No description provided.