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
This commit is contained in:
@@ -53,7 +53,7 @@ public class TabletTicker
|
|||||||
|
|
||||||
private static final String TAG = "StatusBar.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.
|
// 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;
|
private static final int QUEUE_LENGTH = 3;
|
||||||
@@ -300,8 +300,16 @@ public class TabletTicker
|
|||||||
if (CLICKABLE_TICKER) {
|
if (CLICKABLE_TICKER) {
|
||||||
PendingIntent contentIntent = notification.notification.contentIntent;
|
PendingIntent contentIntent = notification.notification.contentIntent;
|
||||||
if (contentIntent != null) {
|
if (contentIntent != null) {
|
||||||
group.setOnClickListener(mBar.makeClicker(contentIntent,
|
// create the usual notification clicker, but chain it together with a halt() call
|
||||||
notification.pkg, notification.tag, notification.id));
|
// 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 {
|
} else {
|
||||||
group.setOnClickListener(null);
|
group.setOnClickListener(null);
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user