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

Commit bb08ef79 authored by Ioana Alexandru's avatar Ioana Alexandru
Browse files

Remove unused config toggle for hiding the clear-all button. 

Fix: 302100163
Test: builds
Change-Id: If0118cd5306559950fe55f51d1704027238226de
parent 71a57780

packages/SystemUI/res/values/config.xml

+0 −3
Original line number Diff line number Diff line
@@ -344,9 +344,6 @@ 
    <!-- Whether to show activity indicators in the status bar -->

    <bool name="config_showActivity">false</bool>



    <!-- Whether or not the button to clear all notifications will be shown. -->

    <bool name="config_enableNotificationsClearAll">true</bool>



    <!-- Whether or not to show the notification shelf that houses the icons of notifications that

     have been scrolled off-screen. -->

    <bool name="config_showNotificationShelf">true</bool>

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackScrollLayout.java

+1 −6
Original line number Diff line number Diff line
@@ -485,8 +485,6 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
    private ShadeController mShadeController;

    private Consumer<Boolean> mOnStackYChanged;



    protected boolean mClearAllEnabled;



    private Interpolator mHideXInterpolator = Interpolators.FAST_OUT_SLOW_IN;



    private final NotificationSectionsManager mSectionsManager;
@@ -667,7 +665,6 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
            mDebugPaint.setStyle(Paint.Style.STROKE);

            mDebugPaint.setTextSize(25f);

        }

        mClearAllEnabled = res.getBoolean(R.bool.config_enableNotificationsClearAll);

        mGroupMembershipManager = Dependency.get(GroupMembershipManager.class);

        mGroupExpansionManager = Dependency.get(GroupExpansionManager.class);

        setImportantForAccessibility(IMPORTANT_FOR_ACCESSIBILITY_YES);
@@ -755,8 +752,7 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
    }



    private boolean shouldShowDismissView() {

        return mClearAllEnabled

                && mController.hasActiveClearableNotifications(ROWS_ALL);

        return mController.hasActiveClearableNotifications(ROWS_ALL);

    }



    private boolean shouldShowFooterView(boolean showDismissView) {
@@ -5160,7 +5156,6 @@ public class NotificationStackScrollLayout extends ViewGroup implements Dumpable 
                    DumpUtilsKt.withIncreasedIndent(

                            pw,

                            () -> {

                                pw.println("mClearAllEnabled: " + mClearAllEnabled);

                                pw.println(

                                        "hasActiveClearableNotifications: "

                                                + mController.hasActiveClearableNotifications(