OLD | NEW |
---|---|
1 <?xml version="1.0" encoding="utf-8"?> | 1 <?xml version="1.0" encoding="utf-8"?> |
please use gerrit instead
2016/08/26 16:46:15
Since you decided to go with ellipsis instead of m
sebsg
2016/08/26 19:36:45
That resource file is for the text value of the dr
| |
2 <!-- Copyright 2016 The Chromium Authors. All rights reserved. | |
2 | 3 |
3 <!-- Copyright 2016 The Chromium Authors. All rights reserved. | |
4 Use of this source code is governed by a BSD-style license that can be | 4 Use of this source code is governed by a BSD-style license that can be |
5 found in the LICENSE file. | 5 found in the LICENSE file. |
6 --> | 6 --> |
7 | 7 |
8 <TextView xmlns:android="http://schemas.android.com/apk/res/android" | 8 <TextView xmlns:android="http://schemas.android.com/apk/res/android" |
please use gerrit instead
2016/08/26 16:46:15
Is this a trailing whitespace? If so, remove it.
sebsg
2016/08/26 19:36:45
Done.
| |
9 android:id="@+id/control_spinner_drop_down" | 9 android:id="@android:id/text1" |
please use gerrit instead
2016/08/26 16:46:15
Remove the ID
sebsg
2016/08/26 19:36:45
Done.
| |
10 style="?android:attr/spinnerItemStyle" | |
11 android:singleLine="false" | |
please use gerrit instead
2016/08/26 16:46:15
If you want a single-line with ellipsis, then you
sebsg
2016/08/26 19:36:45
This one actually does multiline :)
| |
10 android:layout_width="match_parent" | 12 android:layout_width="match_parent" |
11 android:layout_height="wrap_content" | 13 android:layout_height="wrap_content" |
12 android:layout_marginTop="10dp" | 14 android:ellipsize="marquee" |
13 android:layout_marginBottom="10dp" | 15 android:textAlignment="inherit"/> |
14 android:background="?android:attr/selectableItemBackground" | |
15 android:ellipsize="end" | |
16 android:padding="10dp" | |
17 android:singleLine="true" /> | |
OLD | NEW |