#10 login_page_ui

マージ済み
gegham が 6 個のコミットを login_page_ui から dev へマージ 4年前
musheghsahakyan984年前 にコメント
まだ内容がありません
geghammusheghsahakyan98 が担当者に指名 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;
gegham4年前 にコメント

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
gegham4年前 にコメント

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"
gegham4年前 にコメント

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 がプルリクエストをクローズ 4年前
gegham がブランチ login_page_ui を削除 4年前

レビューア

gegham が変更を承認 4年前
プルリクエストは 348619b205 でマージされています。
サインインしてこの会話に参加。
レビューアなし
ラベルなし
マイルストーンなし
担当者なし
2 人の参加者
通知
期日

期日は未設定です。

依存関係

このプルリクエストに依存関係はありません。

読み込み中…
まだ内容がありません