Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Increased the text size for colorized notifications" into oc-mr1-dev

Browse Source
This commit is contained in:
Selim Cinek
2017-09-15 21:20:44 +00:00
committed by Android (Google) Code Review
parent 0fe3b1aa13 299b3302ef
commit dd3b6c83c6
7 changed files with 74 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -67,6 +67,7 @@ import android.text.style.TextAppearanceSpan;
import android.util.ArraySet;
import android.util.Log;
import android.util.SparseArray;
import android.util.TypedValue;
import android.view.Gravity;
import android.view.NotificationHeaderView;
import android.view.View;
@@ -3898,6 +3899,7 @@ public class Notification implements Parcelable
if (p.title != null) {
contentView.setViewVisibility(R.id.title, View.VISIBLE);
contentView.setTextViewText(R.id.title, processTextSpans(p.title));
updateTextSizePrimary(contentView, R.id.title);
if (!p.ambient) {
setTextViewColorPrimary(contentView, R.id.title);
}
@@ -3909,6 +3911,7 @@ public class Notification implements Parcelable
int textId = showProgress ? com.android.internal.R.id.text_line_1
: com.android.internal.R.id.text;
contentView.setTextViewText(textId, processTextSpans(p.text));
updateTextSizeSecondary(contentView, textId);
if (!p.ambient) {
setTextViewColorSecondary(contentView, textId);
}
@@ -3920,6 +3923,25 @@ public class Notification implements Parcelable
return contentView;
}
private void updateTextSizeSecondary(RemoteViews contentView, int textId) {
updateTextSizeColorized(contentView, textId,
com.android.internal.R.dimen.notification_text_size_colorized,
com.android.internal.R.dimen.notification_text_size);
}
private void updateTextSizePrimary(RemoteViews contentView, int textId) {
updateTextSizeColorized(contentView, textId,
com.android.internal.R.dimen.notification_title_text_size_colorized,
com.android.internal.R.dimen.notification_title_text_size);
}
private void updateTextSizeColorized(RemoteViews contentView, int textId,
int colorizedDimen, int normalDimen) {
int size = mContext.getResources().getDimensionPixelSize(isColorized()
? colorizedDimen : normalDimen);
contentView.setTextViewTextSize(textId, TypedValue.COMPLEX_UNIT_PX, size);
}
private CharSequence processTextSpans(CharSequence text) {
if (hasForegroundColor()) {
return NotificationColorUtil.clearColorSpans(text);
@@ -5845,6 +5867,7 @@ public class Notification implements Parcelable
builder.setTextViewColorSecondary(contentView, R.id.big_text);
contentView.setViewVisibility(R.id.big_text,
TextUtils.isEmpty(bigTextText) ? View.GONE : View.VISIBLE);
builder.updateTextSizeSecondary(contentView, R.id.big_text);
contentView.setBoolean(R.id.big_text, "setHasImage", builder.mN.hasLargeIcon());
}
}
@@ -6178,6 +6201,7 @@ public class Notification implements Parcelable
contentView.setViewVisibility(rowId, View.VISIBLE);
contentView.setTextViewText(rowId, mBuilder.processTextSpans(
makeMessageLine(m, mBuilder)));
mBuilder.updateTextSizeSecondary(contentView, rowId);
mBuilder.setTextViewColorSecondary(contentView, rowId);
if (contractedMessage == m) {
@@ -6545,6 +6569,7 @@ public class Notification implements Parcelable
contentView.setViewVisibility(rowIds[i], View.VISIBLE);
contentView.setTextViewText(rowIds[i],
mBuilder.processTextSpans(mBuilder.processLegacyText(str)));
mBuilder.updateTextSizeSecondary(contentView, rowIds[i]);
mBuilder.setTextViewColorSecondary(contentView, rowIds[i]);
contentView.setViewPadding(rowIds[i], 0, topPadding, 0, 0);
handleInboxImageMargin(contentView, rowIds[i], first);

5
core/res/res/values/dimens.xml
View File

@@ -346,6 +346,11 @@
<dimen name="notification_text_size">14sp</dimen>
<!-- Size of notification text titles (see TextAppearance.StatusBar.EventContent.Title) -->
<dimen name="notification_title_text_size">14sp</dimen>
<!-- Size of notification text (see TextAppearance.StatusBar.EventContent) when colorized -->
<dimen name="notification_text_size_colorized">16sp</dimen>
<!-- Size of notification text titles (see TextAppearance.StatusBar.EventContent.Title) when colorized -->
<dimen name="notification_title_text_size_colorized">20sp</dimen>
<!-- Size of smaller notification text (see TextAppearance.StatusBar.EventContent.Line2, Info, Time) -->
<dimen name="notification_subtext_size">12sp</dimen>

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

@@ -3050,6 +3050,9 @@
<java-symbol type="array" name="config_allowedSystemInstantAppSettings" />
<java-symbol type="array" name="config_allowedSecureInstantAppSettings" />
<java-symbol type="dimen" name="notification_text_size_colorized" />
<java-symbol type="dimen" name="notification_title_text_size_colorized" />
<java-symbol type="bool" name="config_handleVolumeKeysInWindowManager" />
<java-symbol type="dimen" name="config_inCallNotificationVolume" />
<java-symbol type="string" name="config_inCallNotificationSound" />

6
packages/SystemUI/res/values/dimens.xml
View File

@@ -80,6 +80,12 @@
<!-- Height of a small notification in the status bar which was used before android N -->
<dimen name="notification_min_height_legacy">64dp</dimen>
<!-- The increase in minHeight that is allowed when the notification is colorized -->
<dimen name="notification_height_increase_colorized">11sp</dimen>
<!-- The increase in minHeight that is allowed when the notification is colorized and has increased height (i.e messages) -->
<dimen name="notification_height_increase_colorized_increased">13sp</dimen>
<!-- Height of a large notification in the status bar -->
<dimen name="notification_max_height">284dp</dimen>

10
packages/SystemUI/src/com/android/systemui/statusbar/ExpandableNotificationRow.java
View File

@@ -64,10 +64,10 @@ import com.android.systemui.plugins.statusbar.NotificationMenuRowPlugin;
import com.android.systemui.plugins.statusbar.NotificationMenuRowPlugin.MenuItem;
import com.android.systemui.statusbar.NotificationGuts.GutsContent;
import com.android.systemui.statusbar.notification.AboveShelfChangedListener;
import com.android.systemui.statusbar.notification.AboveShelfObserver;
import com.android.systemui.statusbar.notification.HybridNotificationView;
import com.android.systemui.statusbar.notification.NotificationInflater;
import com.android.systemui.statusbar.notification.NotificationUtils;
import com.android.systemui.statusbar.notification.NotificationViewWrapper;
import com.android.systemui.statusbar.notification.VisualStabilityManager;
import com.android.systemui.statusbar.phone.NotificationGroupManager;
import com.android.systemui.statusbar.phone.StatusBar;
@@ -436,6 +436,9 @@ public class ExpandableNotificationRow extends ActivatableNotificationView
} else {
minHeight = mNotificationMinHeight;
}
NotificationViewWrapper collapsedWrapper = layout.getVisibleWrapper(
NotificationContentView.VISIBLE_TYPE_CONTRACTED);
minHeight += collapsedWrapper.getMinHeightIncrease(mUseIncreasedCollapsedHeight);
boolean headsUpCustom = layout.getHeadsUpChild() != null &&
layout.getHeadsUpChild().getId()
!= com.android.internal.R.id.status_bar_latest_event_content;
@@ -447,6 +450,11 @@ public class ExpandableNotificationRow extends ActivatableNotificationView
} else {
headsUpheight = mMaxHeadsUpHeight;
}
NotificationViewWrapper headsUpWrapper = layout.getVisibleWrapper(
NotificationContentView.VISIBLE_TYPE_HEADSUP);
if (headsUpWrapper != null) {
headsUpheight += headsUpWrapper.getMinHeightIncrease(mUseIncreasedCollapsedHeight);
}
layout.setHeights(minHeight, headsUpheight, mNotificationMaxHeight,
mNotificationAmbientHeight);
}

17
packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationTemplateViewWrapper.java
View File

@@ -25,6 +25,7 @@ import android.widget.ImageView;
import android.widget.ProgressBar;
import android.widget.TextView;
import com.android.systemui.R;
import com.android.systemui.statusbar.CrossFadeHelper;
import com.android.systemui.statusbar.ExpandableNotificationRow;
import com.android.systemui.statusbar.TransformableView;
@@ -47,6 +48,7 @@ public class NotificationTemplateViewWrapper extends NotificationHeaderViewWrapp
private int mContentHeight;
private int mMinHeightHint;
private boolean mColorized;
protected NotificationTemplateViewWrapper(Context ctx, View view,
ExpandableNotificationRow row) {
@@ -162,7 +164,9 @@ public class NotificationTemplateViewWrapper extends NotificationHeaderViewWrapp
public void onContentUpdated(ExpandableNotificationRow row) {
// Reinspect the notification. Before the super call, because the super call also updates
// the transformation types and we need to have our values set by then.
resolveTemplateViews(row.getStatusBarNotification());
StatusBarNotification sbn = row.getStatusBarNotification();
resolveTemplateViews(sbn);
mColorized = sbn.getNotification().isColorized();
super.onContentUpdated(row);
}
@@ -265,6 +269,17 @@ public class NotificationTemplateViewWrapper extends NotificationHeaderViewWrapp
updateActionOffset();
}
@Override
public int getMinHeightIncrease(boolean useIncreasedCollapsedHeight) {
if (mColorized) {
int dimen = useIncreasedCollapsedHeight
? R.dimen.notification_height_increase_colorized_increased
: R.dimen.notification_height_increase_colorized;
return mRow.getResources().getDimensionPixelSize(dimen);
}
return super.getMinHeightIncrease(useIncreasedCollapsedHeight);
}
private void updateActionOffset() {
if (mActionsContainer != null) {
// We should never push the actions higher than they are in the headsup view.

10
packages/SystemUI/src/com/android/systemui/statusbar/notification/NotificationViewWrapper.java
View File

@@ -190,4 +190,14 @@ public abstract class NotificationViewWrapper implements TransformableView {
public boolean disallowSingleClick(float x, float y) {
return false;
}
/**
* Get the amount that the minheight is allowed to be increased based on this layout.
*
* @param increasedHeight is the view allowed to show even bigger, i.e for messaging layouts
* @return
*/
public int getMinHeightIncrease(boolean increasedHeight) {
return 0;
}
}