drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix underfull content bug in inbox and big text templates.

Browse Source 
Bug: 7172721
Change-Id: I22019009eec31ca18b54eec5f6ed5b626cee1540
This commit is contained in:
Chris Wren
2012-09-20 10:35:54 -04:00
parent e0de5bfff2
commit 683ab00a2e
5 changed files with 164 additions and 145 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
core/java/android/app/Notification.java
View File

@ -1923,6 +1923,7 @@ public class Notification implements Parcelable
contentView.setViewVisibility(rowId, View.GONE); contentView.setViewVisibility(rowId, View.GONE);
} }
int i=0; int i=0;
while (i < mTexts.size() && i < rowIds.length) { while (i < mTexts.size() && i < rowIds.length) {
CharSequence str = mTexts.get(i); CharSequence str = mTexts.get(i);
@ -1933,11 +1934,11 @@ public class Notification implements Parcelable
i++; i++;
} }
if (mTexts.size() > rowIds.length) { contentView.setViewVisibility(R.id.inbox_end_pad,
contentView.setViewVisibility(R.id.inbox_more, View.VISIBLE); mTexts.size() > 0 ? View.VISIBLE : View.GONE);
} else {
contentView.setViewVisibility(R.id.inbox_more, View.GONE); contentView.setViewVisibility(R.id.inbox_more,
} mTexts.size() > rowIds.length ? View.VISIBLE : View.GONE);
return contentView; return contentView;
} }

1
core/res/res/layout/notification_action_list.xml
View File

@ -21,6 +21,7 @@
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:orientation="horizontal" android:orientation="horizontal"
android:visibility="gone" android:visibility="gone"
android:layout_marginBottom="8dp"
android:showDividers="middle" android:showDividers="middle"
android:divider="?android:attr/listDivider" android:divider="?android:attr/listDivider"
android:dividerPadding="12dp" android:dividerPadding="12dp"

5
core/res/res/layout/notification_template_big_text.xml
View File

@ -34,13 +34,14 @@
android:layout_gravity="fill_vertical" android:layout_gravity="fill_vertical"
android:layout_marginStart="@dimen/notification_large_icon_width" android:layout_marginStart="@dimen/notification_large_icon_width"
android:orientation="vertical" android:orientation="vertical"
android:paddingTop="2dp" android:paddingTop="0dp"
android:paddingBottom="2dp" android:paddingBottom="2dp"
android:gravity="top" android:gravity="top"
> >
<LinearLayout <LinearLayout
android:layout_width="match_parent" android:layout_width="match_parent"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:minHeight="@dimen/notification_large_icon_height"
android:orientation="vertical" android:orientation="vertical"
android:layout_marginStart="8dp" android:layout_marginStart="8dp"
android:layout_marginEnd="8dp" android:layout_marginEnd="8dp"
@ -50,7 +51,7 @@
android:id="@+id/line1" android:id="@+id/line1"
android:layout_width="match_parent" android:layout_width="match_parent"
android:layout_height="wrap_content" android:layout_height="wrap_content"
android:paddingTop="6dp" android:paddingTop="8dp"
android:orientation="horizontal" android:orientation="horizontal"
android:layout_gravity="top" android:layout_gravity="top"
android:layout_weight="0" android:layout_weight="0"

25
core/res/res/layout/notification_template_inbox.xml
View File

@ -36,10 +36,17 @@
android:layout_marginStart="@dimen/notification_large_icon_width" android:layout_marginStart="@dimen/notification_large_icon_width"
android:minHeight="@dimen/notification_large_icon_height" android:minHeight="@dimen/notification_large_icon_height"
android:orientation="vertical" android:orientation="vertical"
android:paddingTop="2dp" android:paddingTop="0dp"
android:paddingBottom="2dp" android:paddingBottom="2dp"
android:gravity="top" android:gravity="top"
> >
<LinearLayout
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:minHeight="@dimen/notification_large_icon_height"
android:paddingTop="2dp"
android:orientation="vertical"
>
<LinearLayout <LinearLayout
android:id="@+id/line1" android:id="@+id/line1"
android:layout_width="match_parent" android:layout_width="match_parent"
@ -188,11 +195,19 @@
android:layout_weight="1" android:layout_weight="1"
android:text="@android:string/ellipsis" android:text="@android:string/ellipsis"
/> />
<FrameLayout
android:id="@+id/inbox_end_pad"
android:layout_width="match_parent"
android:layout_height="8dip"
android:visibility="gone"
android:layout_weight="0"
/>
</LinearLayout>
<ImageView <ImageView
android:layout_width="match_parent" android:layout_width="match_parent"
android:layout_height="1px" android:layout_height="1dip"
android:layout_marginTop="-1px"
android:id="@+id/action_divider" android:id="@+id/action_divider"
android:visibility="gone"
android:background="?android:attr/dividerHorizontal" /> android:background="?android:attr/dividerHorizontal" />
<include <include
layout="@layout/notification_action_list" layout="@layout/notification_action_list"
@ -202,9 +217,9 @@
/> />
<ImageView <ImageView
android:layout_width="match_parent" android:layout_width="match_parent"
android:layout_height="1px" android:layout_height="1dip"
android:layout_marginTop="-1px"
android:id="@+id/overflow_divider" android:id="@+id/overflow_divider"
android:layout_marginTop="8dp"
android:visibility="visible" android:visibility="visible"
android:background="?android:attr/dividerHorizontal" /> android:background="?android:attr/dividerHorizontal" />
<LinearLayout <LinearLayout

1
core/res/res/values/symbols.xml
View File

@ -208,6 +208,7 @@
<java-symbol type="id" name="inbox_text5" /> <java-symbol type="id" name="inbox_text5" />
<java-symbol type="id" name="inbox_text6" /> <java-symbol type="id" name="inbox_text6" />
<java-symbol type="id" name="inbox_more" /> <java-symbol type="id" name="inbox_more" />
<java-symbol type="id" name="inbox_end_pad" />
<java-symbol type="id" name="status_bar_latest_event_content" /> <java-symbol type="id" name="status_bar_latest_event_content" />
<java-symbol type="id" name="action_divider" /> <java-symbol type="id" name="action_divider" />
<java-symbol type="id" name="overflow_divider" /> <java-symbol type="id" name="overflow_divider" />