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

Remove shadows from shade

Browse Source 
Removed base elevation from shade, but kept it on overlaping
notifications or heads-up. Othwerise we'd have legibility issues
depending on where the heads-up is rendered.

Test: visual
Bug: 173561906
Change-Id: If4f275437853e394fbd49378f2af489ba1abcead
This commit is contained in:
Lucas Dupin
2020-12-14 12:17:57 -08:00
parent 475acd3f2b
commit c3cfd025d0
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/AmbientState.java
View File

@@ -20,7 +20,6 @@ import android.annotation.NonNull;
import android.annotation.Nullable;
import android.content.Context;
import android.util.MathUtils;
import android.view.View;
import com.android.systemui.R;
import com.android.systemui.statusbar.NotificationShelf;
@@ -31,14 +30,13 @@ import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow
import com.android.systemui.statusbar.notification.row.ExpandableView;
import com.android.systemui.statusbar.notification.stack.StackScrollAlgorithm.SectionProvider;
import java.util.ArrayList;
/**
* A global state to track all input states for the algorithm.
*/
public class AmbientState {
private static final float MAX_PULSE_HEIGHT = 100000f;
private static final boolean NOTIFICATIONS_HAVE_SHADOWS = false;
private final SectionProvider mSectionProvider;
private int mScrollY;
@@ -106,7 +104,7 @@ public class AmbientState {
}
private static int getBaseHeight(int zdistanceBetweenElements) {
return 4 * zdistanceBetweenElements;
return NOTIFICATIONS_HAVE_SHADOWS ? 4 * zdistanceBetweenElements : 0;
}
/**