From 60f626a45fa3ec1ecb5567a19534b4267e6b331e Mon Sep 17 00:00:00 2001
From: M M Arif <mmarif@swatian.com>
Date: Wed, 4 Dec 2019 16:29:39 +0000
Subject: [PATCH]

---
 Code-Standards.md | 47 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 47 insertions(+)

diff --git a/Code-Standards.md b/Code-Standards.md
index 2399fc4..96bfe41 100644
--- a/Code-Standards.md
+++ b/Code-Standards.md
@@ -100,4 +100,51 @@ popupMenu.setOnMenuItemClickListener(new PopupMenu.OnMenuItemClickListener() {
         return false;
     }
 });
+```
+
+#### Variable names and scope
+Variable names has to be clear for what it is used. Scope should be defined properly for the variable. Example,
+
+```
+private TextView issueNumber;
+private ImageView issueAssigneeAvatar;
+```
+
+#### Layout ID's name
+Every layout must have components with defined ID's. Example,
+
+```
+<TextView
+    android:id="@+id/branchName"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    android:textColor="@color/white"
+    android:textIsSelectable="true"
+    android:textSize="18sp" />
+
+<TextView
+    android:id="@+id/branchCommitAuthor"
+    android:layout_marginTop="10dp"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    android:textIsSelectable="true"
+    android:textColor="@color/colorWhite"
+    android:textSize="16sp" />
+
+<TextView
+    android:id="@+id/branchCommitHash"
+    android:layout_marginTop="10dp"
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    android:textColor="@color/colorWhite"
+    android:textSize="16sp"
+    android:textColorLink="@color/lightBlue" />
+```
+
+#### String names
+All the string names in strings.xml must be meaningful. Also check for the string if it is already defined.
+
+```
+<string name="mergePRSuccessMsg">Pull Request was merged successfully</string>
+<string name="mergePR404ErrorMsg">Pull Request is not available for merge</string>
 ```
\ No newline at end of file