diff options
author | Tony Wickham <twickham@google.com> | 2017-01-27 13:18:30 -0800 |
---|---|---|
committer | Tony Wickham <twickham@google.com> | 2017-02-22 15:33:35 -0800 |
commit | 6d66c1cfeb8194e74789dd698f68b9a2a8753153 (patch) | |
tree | 8e861481c398c6bdb0194fa86ec8145b260f55dc /src/com/android/launcher3/notification/NotificationItemView.java | |
parent | 9f0fa84439bb177ed48758b6d15a8e62c80f1bf4 (diff) | |
download | android_packages_apps_Trebuchet-6d66c1cfeb8194e74789dd698f68b9a2a8753153.tar.gz android_packages_apps_Trebuchet-6d66c1cfeb8194e74789dd698f68b9a2a8753153.tar.bz2 android_packages_apps_Trebuchet-6d66c1cfeb8194e74789dd698f68b9a2a8753153.zip |
Remove first icon from notification footer after it animates.
Before, we had a hack where the notifation view would tell the
footer to trim its notifications, which is inefficient and ugly
since we already know exactly what notification to remove. So
now we move the relevant removal logic (keeping things like the
overflow text in sync) to a removeViewFromIconRow() that is
used after the icon animates as well as when trimming notifications.
Bug: 32410600
Change-Id: I19e54e0c28daea869b45649c7f5c74843412bb86
Diffstat (limited to 'src/com/android/launcher3/notification/NotificationItemView.java')
-rw-r--r-- | src/com/android/launcher3/notification/NotificationItemView.java | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/com/android/launcher3/notification/NotificationItemView.java b/src/com/android/launcher3/notification/NotificationItemView.java index c9b3940b8..d58bef697 100644 --- a/src/com/android/launcher3/notification/NotificationItemView.java +++ b/src/com/android/launcher3/notification/NotificationItemView.java @@ -37,7 +37,6 @@ import com.android.launcher3.logging.UserEventDispatcher.LogContainerProvider; import com.android.launcher3.popup.PopupItemView; import com.android.launcher3.userevent.nano.LauncherLogProto; -import java.util.ArrayList; import java.util.List; import static com.android.launcher3.LauncherAnimUtils.animateViewHeight; @@ -145,12 +144,6 @@ public class NotificationItemView extends PopupItemView implements LogContainerP public void onIconAnimationEnd(NotificationInfo newMainNotification) { if (newMainNotification != null) { mMainView.applyNotificationInfo(newMainNotification, mIconView, true); - // Remove the animated notification from the footer by calling trim - // TODO: Remove the notification in NotificationFooterLayout directly - // instead of relying on this hack. - List<String> footerNotificationKeys = new ArrayList<>(notificationKeys); - footerNotificationKeys.remove(newMainNotification.notificationKey); - mFooter.trimNotifications(footerNotificationKeys); mMainView.setVisibility(VISIBLE); } mAnimatingNextIcon = false; |