Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 82524bff authored by Ioana Alexandru's avatar Ioana Alexandru
Browse files

Stop greying out icons in group children 

In an effort to reduce duplicate information in groups, we would make the color of group icons grey and only show it in the header if it was the same. With the new design and themes though, the grey looks more out of place, and small icons can only be used by system apps anyway, which primarily use the default color. We can therefore remove this logic.

Fix: 414555351
Test: manual, temporarily made all apps show small icons to test
Flag: android.app.notifications_redesign_app_icons
Change-Id: Ibf5c11dd0be1e58221822352305c89da04ea43e3
parent 6e3b3b84

packages/SystemUI/src/com/android/systemui/statusbar/NotificationGroupingUtil.java

+9 −6
Original line number Diff line number Diff line
@@ -16,6 +16,7 @@ 


package com.android.systemui.statusbar;



import static android.app.Flags.notificationsRedesignAppIcons;

import static android.app.Flags.notificationsRedesignTemplates;



import android.app.Flags;
@@ -127,12 +128,14 @@ public class NotificationGroupingUtil { 
                ICON_EXTRACTOR,

                iconVisibilityComparator,

                VISIBILITY_APPLICATOR));

        if (!notificationsRedesignAppIcons()) {

            // To grey out the icons when they are not the same, or they have the same color

            mProcessors.add(new Processor(mRow,

                    com.android.internal.R.id.status_bar_latest_event_content,

                    ICON_EXTRACTOR,

                    greyComparator,

                    greyApplicator));

        }

        // To show the large icon on the left side instead if all the small icons are the same

        mProcessors.add(new Processor(mRow,

                com.android.internal.R.id.status_bar_latest_event_content,