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

Commit c05ff705 authored by Caitlin Shkuratov's avatar Caitlin Shkuratov Committed by Android (Google) Code Review
Browse files

Merge "[SB][Notifs] Make notification chips icon-only for now." into main

parents 98d6bd44 3b8a4b19

packages/SystemUI/multivalentTests/src/com/android/systemui/statusbar/chips/notification/ui/viewmodel/NotifChipsViewModelTest.kt

+1 −2
Original line number Diff line number Diff line
@@ -168,8 +168,7 @@ class NotifChipsViewModelTest : SysuiTestCase() { 


    companion object {

        fun assertIsNotifChip(latest: OngoingActivityChipModel?, expectedIcon: StatusBarIconView) {

            assertThat(latest)

                .isInstanceOf(OngoingActivityChipModel.Shown.ShortTimeDelta::class.java)

            assertThat(latest).isInstanceOf(OngoingActivityChipModel.Shown.IconOnly::class.java)

            assertThat((latest as OngoingActivityChipModel.Shown).icon)

                .isEqualTo(OngoingActivityChipModel.ChipIcon.StatusBarView(expectedIcon))

        }

packages/SystemUI/src/com/android/systemui/statusbar/chips/notification/ui/viewmodel/NotifChipsViewModel.kt

+2 −7
Original line number Diff line number Diff line
@@ -68,13 +68,8 @@ constructor( 
                    notifChipsInteractor.onPromotedNotificationChipTapped(this@toChipModel.key)

                }

            }

        return OngoingActivityChipModel.Shown.ShortTimeDelta(

            icon,

            colors,

            time = this.whenTime,

            onClickListener,

        )

        // TODO(b/364653005): If Notification.showWhen = false, don't show the time delta.

        return OngoingActivityChipModel.Shown.IconOnly(icon, colors, onClickListener)

        // TODO(b/364653005): Use Notification.showWhen to determine if we should show the time.

        // TODO(b/364653005): If Notification.whenTime is in the past, show "ago" in the text.

        // TODO(b/364653005): If Notification.shortCriticalText is set, use that instead of `when`.

        // TODO(b/364653005): If the app that posted the notification is in the foreground, don't