#10 login_page_ui

Слито
gegham слито 6 коммит(ов) из login_page_ui в dev 4 лет назад
musheghsahakyan98 прокомментировал 4 лет назад
Пока нет содержимого.
gegham был назначен musheghsahakyan98 4 лет назад
gegham запросил(а) изменения 4 лет назад
ui/src/main/java/com/wireguard/android/activity/LoginActivity.java
@@ -37,3 +34,4 @@
private static final String SEPARATOR = "\\.";
private static final int REQUEST_CODE = 1555;
private static final String CERTIFICATE_NAME = "Bubble Certificate";
private boolean bubbleNameStateFlag = false;
gegham прокомментировал 4 лет назад

These 3 variables should be saved in the onSavedInstanceSave bundle, not to be loss in case of state loss.
Make the app portrait only.

These 3 variables should be saved in the onSavedInstanceSave bundle, not to be loss in case of state loss. Make the app portrait only.
ui/src/main/res/layout/activity_login.xml
@@ -8,2 +8,4 @@
tools:context="com.wireguard.android.activity.LoginActivity">


<ImageView
gegham прокомментировал 4 лет назад

Keep the dimensions and colors in the corresponding resource files. All across the app XMLs.

Keep the dimensions and colors in the corresponding resource files. All across the app XMLs.
ui/src/main/res/layout/activity_login.xml
@@ -31,2 +81,4 @@
app:layout_constraintVertical_bias="0.424" />

<TextView
android:id="@+id/textView4"
gegham прокомментировал 4 лет назад

id names should be logical, textView4 is a bad naming. Change such all across the app XMLs.

id names should be logical, textView4 is a bad naming. Change such all across the app XMLs.
gegham одобрил(а) эти изменения 4 лет назад
gegham closed this pull request 4 лет назад
gegham удалена ветка login_page_ui 4 лет назад

Рецензенты

gegham одобрил(а) эти изменения 4 лет назад
Pull Request был объединен как %!с(string=348619b205).
Войдите, чтобы присоединиться к обсуждению.
Нет рецензентов
Нет меток
Нет этапа
Нет назначенных лиц
2 участников
Уведомления
Срок выполнения

Срок выполнения не установлен.

Зависимости

Этот запрос на слияние в настоящее время не имеет никаких зависимостей.

Загрузка…
Пока нет содержимого.