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

Commit e2c251b5 authored by Daniel Sandler's avatar Daniel Sandler
Browse files

Tapping the ticker again activates the notification. 

(Previous behavior: the tap would pass through to the clock,
which would open the shade. Only sometimes it wouldn't,
because you'd have hit the left-hand-side of the ticker
where there's no clock underneath. So this fixes that too.)

Additionally, tapping the ticker will now immediately
dismiss the ticker.

Bug: 3365129
Change-Id: Ic641184c518b18d799a560c8da6b4c5844c912de
parent a85b789a

packages/SystemUI/src/com/android/systemui/statusbar/tablet/TabletTicker.java

+11 −3
Original line number Diff line number Diff line
@@ -53,7 +53,7 @@ public class TabletTicker 


    private static final String TAG = "StatusBar.TabletTicker";



    private static final boolean CLICKABLE_TICKER = false;

    private static final boolean CLICKABLE_TICKER = true;



    // 3 is enough to let us see most cases, but not get so far behind that it's too annoying.

    private static final int QUEUE_LENGTH = 3;
@@ -300,8 +300,16 @@ public class TabletTicker 
        if (CLICKABLE_TICKER) {

            PendingIntent contentIntent = notification.notification.contentIntent;

            if (contentIntent != null) {

                group.setOnClickListener(mBar.makeClicker(contentIntent,

                            notification.pkg, notification.tag, notification.id));

                // create the usual notification clicker, but chain it together with a halt() call

                // to abort the ticker too

                final View.OnClickListener clicker = mBar.makeClicker(contentIntent,

                                            notification.pkg, notification.tag, notification.id);

                group.setOnClickListener(new View.OnClickListener() {

                    public void onClick(View v) {

                        halt();

                        clicker.onClick(v);

                    }

                });

            } else {

                group.setOnClickListener(null);

            }