diff --git a/app/src/main/java/luyao/box/about/AboutActivity.kt b/app/src/main/java/luyao/box/about/AboutActivity.kt
index 9b40828..6824aaa 100644
--- a/app/src/main/java/luyao/box/about/AboutActivity.kt
+++ b/app/src/main/java/luyao/box/about/AboutActivity.kt
@@ -54,7 +54,7 @@ class AboutActivity : BaseActivity() {
     }
 
     private fun showFeedBackMenu() {
-        val feedbackMenu = PopupMenu(this, feedback, Gravity.BOTTOM)
+        val feedbackMenu = PopupMenu(this, feedback, Gravity.RIGHT)
         feedbackMenu.menuInflater.inflate(R.menu.menu_feedback, feedbackMenu.menu)
         feedbackMenu.setOnMenuItemClickListener { item: MenuItem ->
             when (item.itemId) {
diff --git a/app/src/main/java/luyao/box/ui/MainActivity.kt b/app/src/main/java/luyao/box/ui/MainActivity.kt
index db14946..1c51102 100644
--- a/app/src/main/java/luyao/box/ui/MainActivity.kt
+++ b/app/src/main/java/luyao/box/ui/MainActivity.kt
@@ -110,11 +110,6 @@ class MainActivity : BaseActivity(), NavigationView.OnNavigationItemSelectedList
             R.id.nav_about -> {
                 startActivity(AboutActivity::class.java)
             }
-
-            R.id.nav_feedback -> {
-
-            }
-
         }
 
         drawer_layout.closeDrawer(GravityCompat.START)
diff --git a/app/src/main/res/layout/activity_about.xml b/app/src/main/res/layout/activity_about.xml
index a8c9e27..994f29c 100644
--- a/app/src/main/res/layout/activity_about.xml
+++ b/app/src/main/res/layout/activity_about.xml
@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
               xmlns:tools="http://schemas.android.com/tools"
+              android:id="@+id/about_root"
               android:orientation="vertical"
               android:layout_width="match_parent"
               android:layout_height="match_parent">
diff --git a/app/src/main/res/layout/activity_device_info.xml b/app/src/main/res/layout/activity_device_info.xml
index 0efc148..d49f7ca 100644
--- a/app/src/main/res/layout/activity_device_info.xml
+++ b/app/src/main/res/layout/activity_device_info.xml
@@ -2,13 +2,15 @@
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
               android:layout_width="match_parent"
               android:layout_height="match_parent"
+              xmlns:app="http://schemas.android.com/apk/res-auto"
               android:orientation="vertical">
 
     <include layout="@layout/title_layout"/>
 
     <ScrollView
             android:layout_width="match_parent"
-            android:layout_height="match_parent">
+            android:layout_height="match_parent"
+            android:background="@color/gray_color">
 
         <LinearLayout
                 android:layout_width="match_parent"
@@ -17,212 +19,260 @@
                 android:orientation="vertical"
                 android:padding="10dp">
 
+            <androidx.cardview.widget.CardView
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    app:cardCornerRadius="6dp"
+                    app:contentPadding="8dp">
 
-            <LinearLayout style="@style/Device_Lin">
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/brand"/>
+                <LinearLayout
+                        android:layout_width="match_parent"
+                        android:layout_height="wrap_content"
+                        android:orientation="vertical">
 
-                <TextView
-                        android:id="@+id/brand"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/brand"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/brand"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/display"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/display"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/display"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/display"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/product"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/product"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/product"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/product"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/device"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/device"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/device"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/device"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/board"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/board"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/board"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/board"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/model"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/model"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/model"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/model"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/manufacturer"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/manufacturer"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/manufacturer"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/manufacturer"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/bootloader"/>
+                    </LinearLayout>
+                </LinearLayout>
 
-                <TextView
-                        android:id="@+id/bootloader"
-                        style="@style/Device_Text"/>
+            </androidx.cardview.widget.CardView>
 
-            </LinearLayout>
 
-            <LinearLayout style="@style/Device_Lin">
+            <androidx.cardview.widget.CardView
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:layout_marginTop="10dp"
+                    app:cardCornerRadius="6dp"
+                    app:contentPadding="8dp">
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/hardware"/>
+                <LinearLayout
+                        android:layout_width="match_parent"
+                        android:layout_height="wrap_content"
+                        android:orientation="vertical">
 
-                <TextView
-                        android:id="@+id/hardware"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/hardware"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/hardware"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/sdk"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/api"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/sdk"/>
 
+                        <TextView
+                                android:id="@+id/api"
+                                style="@style/Device_Text"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                    </LinearLayout>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/android_version"/>
 
-                <TextView
-                        android:id="@+id/version"
-                        style="@style/Device_Text"/>
+                    <LinearLayout style="@style/Device_Lin">
 
-            </LinearLayout>
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/android_version"/>
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                android:id="@+id/version"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/kernel_version"/>
+                    </LinearLayout>
 
-                <TextView
-                        android:id="@+id/os"
-                        style="@style/Device_Text"/>
 
-            </LinearLayout>
+                </LinearLayout>
+            </androidx.cardview.widget.CardView>
 
-            <LinearLayout style="@style/Device_Lin">
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/imei"/>
+            <androidx.cardview.widget.CardView
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:layout_marginTop="10dp"
+                    app:cardCornerRadius="6dp"
+                    app:contentPadding="8dp">
 
-                <TextView
-                        android:id="@+id/imei"
-                        style="@style/Device_Text"/>
+                <LinearLayout
+                        android:layout_width="match_parent"
+                        android:layout_height="wrap_content"
+                        android:orientation="vertical">
 
-            </LinearLayout>
+                    <LinearLayout style="@style/Device_Lin">
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/imei"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/meid"/>
+                        <TextView
+                                android:id="@+id/imei"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        android:id="@+id/meid"
-                        style="@style/Device_Text"/>
+                    </LinearLayout>
 
-            </LinearLayout>
+                    <LinearLayout style="@style/Device_Lin">
 
-            <LinearLayout style="@style/Device_Lin">
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/meid"/>
 
-                <TextView
-                        style="@style/Device_Text"
-                        android:layout_width="100dp"
-                        android:text="@string/sn"/>
+                        <TextView
+                                android:id="@+id/meid"
+                                style="@style/Device_Text"/>
 
-                <TextView
-                        android:id="@+id/sn"
-                        style="@style/Device_Text"/>
+                    </LinearLayout>
 
-            </LinearLayout>
+                    <LinearLayout style="@style/Device_Lin">
+
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/sn"/>
+
+                        <TextView
+                                android:id="@+id/sn"
+                                style="@style/Device_Text"/>
+
+                    </LinearLayout>
+
+                    <LinearLayout style="@style/Device_Lin">
+
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/mac_address"/>
+
+                        <TextView
+                                android:id="@+id/macAddress"
+                                style="@style/Device_Text"/>
+
+                    </LinearLayout>
+
+                    <LinearLayout style="@style/Device_Lin">
+
+                        <TextView
+                                style="@style/Device_Text"
+                                android:layout_width="100dp"
+                                android:text="@string/radio_version"/>
+
+                        <TextView
+                                android:id="@+id/radioVersion"
+                                style="@style/Device_Text"/>
+
+                    </LinearLayout>
+
+
+                </LinearLayout>
+            </androidx.cardview.widget.CardView>
 
             <LinearLayout style="@style/Device_Lin">
 
                 <TextView
                         style="@style/Device_Text"
                         android:layout_width="100dp"
-                        android:text="@string/mac_address"/>
+                        android:text="@string/bootloader"/>
 
                 <TextView
-                        android:id="@+id/macAddress"
+                        android:id="@+id/bootloader"
                         style="@style/Device_Text"/>
 
             </LinearLayout>
@@ -233,27 +283,29 @@
                 <TextView
                         style="@style/Device_Text"
                         android:layout_width="100dp"
-                        android:text="@string/abis"/>
+                        android:text="@string/kernel_version"/>
 
                 <TextView
-                        android:id="@+id/abis"
+                        android:id="@+id/os"
                         style="@style/Device_Text"/>
 
             </LinearLayout>
 
+
             <LinearLayout style="@style/Device_Lin">
 
                 <TextView
                         style="@style/Device_Text"
                         android:layout_width="100dp"
-                        android:text="@string/radio_version"/>
+                        android:text="@string/abis"/>
 
                 <TextView
-                        android:id="@+id/radioVersion"
+                        android:id="@+id/abis"
                         style="@style/Device_Text"/>
 
             </LinearLayout>
 
+
         </LinearLayout>
     </ScrollView>
 </LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/item_main.xml b/app/src/main/res/layout/item_main.xml
index 8563932..2d578c9 100644
--- a/app/src/main/res/layout/item_main.xml
+++ b/app/src/main/res/layout/item_main.xml
@@ -4,7 +4,7 @@
                                    xmlns:app="http://schemas.android.com/apk/res-auto"
                                    android:layout_width="match_parent"
                                    android:layout_height="140dp"
-                                   app:cardElevation="10dp">
+                                   app:cardCornerRadius="6dp">
 
     <LinearLayout
             android:layout_width="match_parent"
diff --git a/app/src/main/res/menu/activity_main_drawer.xml b/app/src/main/res/menu/activity_main_drawer.xml
index 9aec4ca..a0c49e7 100644
--- a/app/src/main/res/menu/activity_main_drawer.xml
+++ b/app/src/main/res/menu/activity_main_drawer.xml
@@ -14,10 +14,6 @@
                 android:id="@+id/nav_about"
                 android:icon="@drawable/ic_menu_camera"
                 android:title="@string/about"/>
-        <item
-                android:id="@+id/nav_feedback"
-                android:icon="@drawable/ic_menu_gallery"
-                android:title="@string/feedback"/>
 
     </group>
 
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index 40f37ed..18d489b 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -98,7 +98,7 @@
     <style name="Device_Text">
         <item name="android:layout_width">match_parent</item>
         <item name="android:layout_height">wrap_content</item>
-        <item name="android:textSize">18sp</item>
+        <item name="android:textSize">16sp</item>
         <item name="android:textColor">@color/black</item>
     </style>