Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android" | |
| 2 xmlns:app="http://schemas.android.com/apk/res-auto" | |
| 3 xmlns:tools="http://schemas.android.com/tools" | |
| 4 android:layout_width="match_parent" | |
| 5 android:layout_height="match_parent" | |
| 6 android:paddingLeft="@dimen/activity_horizontal_margin" | |
|
Ian Wen
2016/02/09 00:14:16
IIRC you can just use android:padding, instead of
BigBossZhiling
2016/02/09 01:03:12
Done.
| |
| 7 android:paddingRight="@dimen/activity_horizontal_margin" | |
| 8 android:paddingTop="@dimen/activity_vertical_margin" | |
| 9 android:paddingBottom="@dimen/activity_vertical_margin" | |
| 10 android:orientation="vertical" | |
| 11 tools:context=".AddActionBarItemDialog"> | |
| 12 | |
| 13 <EditText | |
| 14 android:id="@+id/description" | |
| 15 android:hint="Description for the action." | |
| 16 android:layout_width="match_parent" | |
| 17 android:textSize="@dimen/abc_text_size_medium_material" | |
| 18 android:text="Great Feature" | |
| 19 android:layout_height="wrap_content" | |
| 20 android:inputType="text" /> | |
| 21 | |
| 22 <LinearLayout | |
| 23 android:layout_width="match_parent" | |
| 24 android:paddingTop="@dimen/view_vertical_margin" | |
| 25 android:layout_height="@dimen/view_height" | |
| 26 android:orientation="horizontal"> | |
| 27 | |
| 28 <TextView | |
| 29 android:layout_width="wrap_content" | |
| 30 android:layout_height="wrap_content" | |
| 31 android:paddingLeft="5sp" | |
| 32 android:textSize="@dimen/abc_text_size_medium_material" | |
| 33 android:text="Action:"/> | |
| 34 | |
| 35 <Spinner | |
| 36 android:paddingLeft="@dimen/spinner_padding_left" | |
| 37 android:layout_width="match_parent" | |
| 38 android:layout_height="wrap_content" | |
| 39 android:id="@+id/intent" /> | |
| 40 </LinearLayout> | |
| 41 | |
| 42 <ImageView | |
| 43 android:id="@+id/imageInDialogue" | |
| 44 android:layout_gravity="center" | |
| 45 android:layout_width="100sp" | |
| 46 android:layout_height="100sp" | |
| 47 android:src="@drawable/ic_action_name" | |
| 48 /> | |
| 49 | |
| 50 <LinearLayout | |
|
Ian Wen
2016/02/09 00:14:16
What is the point of nesting a single button in a
| |
| 51 android:layout_width="match_parent" | |
| 52 android:paddingTop="@dimen/view_vertical_margin" | |
| 53 android:gravity="center" | |
| 54 android:layout_height="@dimen/view_height"> | |
| 55 | |
| 56 <Button | |
| 57 android:id="@+id/select_file" | |
| 58 android:layout_gravity="center_horizontal" | |
| 59 android:paddingTop="@dimen/button_padding_top" | |
| 60 android:layout_width="@dimen/button_width" | |
| 61 android:layout_height="@dimen/button_height" | |
| 62 android:textSize="@dimen/abc_text_size_medium_material" | |
| 63 android:text="Select Action Image" /> | |
| 64 </LinearLayout> | |
| 65 | |
| 66 <LinearLayout | |
| 67 android:layout_width="match_parent" | |
| 68 android:paddingTop="@dimen/view_vertical_margin" | |
| 69 android:gravity="center" | |
| 70 android:layout_height="@dimen/view_height"> | |
| 71 | |
| 72 <Button | |
| 73 android:id="@+id/save_item" | |
| 74 android:layout_gravity="center_horizontal" | |
| 75 android:paddingTop="@dimen/button_padding_top" | |
| 76 android:layout_width="@dimen/button_width" | |
| 77 android:layout_height="@dimen/button_height" | |
| 78 android:textSize="@dimen/abc_text_size_medium_material" | |
| 79 android:text="@string/label_save" /> | |
| 80 </LinearLayout> | |
| 81 | |
| 82 <LinearLayout | |
| 83 android:layout_width="match_parent" | |
| 84 android:paddingTop="@dimen/view_vertical_margin" | |
| 85 android:gravity="center" | |
| 86 android:layout_height="@dimen/view_height"> | |
| 87 | |
| 88 <Button | |
| 89 android:id="@+id/cancel" | |
| 90 android:layout_gravity="center_horizontal" | |
| 91 android:paddingTop="@dimen/button_padding_top" | |
| 92 android:layout_width="@dimen/button_width" | |
| 93 android:layout_height="@dimen/button_height" | |
| 94 android:textSize="@dimen/abc_text_size_medium_material" | |
| 95 android:text="cancel" /> | |
| 96 </LinearLayout> | |
| 97 | |
| 98 | |
| 99 </LinearLayout> | |
| OLD | NEW |