| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
Test: atest RoundableTest
Test: manual test
Bug: 257000427
Change-Id: Ib9f2f35cc67f93da818830b807573bc54b075b8f
|
| |
|
|
|
|
|
|
|
|
| |
This CL adjusts the Dagger scoping for StatusBarNotifActivityStarter to
StatusBarScope, so that we don't need to use a Builder that was
previously scoped as SysUISingleton.
Test: mp droid, atest
Bug: 219939676
Change-Id: Iff2a298c19f89e234db2719fb0bc64e2b82c831b
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Show all notifications in the same section at the same time.
On each invocation of StackScrollAlgorithm
- compute fraction of section showing based on current expansion amount
- apply that fraction to each view's intrinsic height
The notification icon shelf no longer slides down from top of screen,
instead it shows if the notification section before it is showing.
Bug: 172289889
[Test accordian effect]
Test: open shade with single section, no shelf
Test: open shade with single section, silent section, no shelf
Test: open shade with first section overflowing into shelf
Test: open shade with first section, silent section overflowing into shelf
Test: open shade with no notifications (empty shade view)
Test: open shade with shelf, scroll notifications, then close shade
=> accordian effect applies to scrolled state
[Test for regressions]
Test: add/remove delayed notification that arrives while shade opens
=> animation instantly updates for new notifs (while expansion runs)
Test: open shade from heads up notification
Test: open shade from pulsing (incoming notif on aod)
Test: open shade from lockscreen
Change-Id: If3236b9dc202ee75db7cac51a66c49620556ec10
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of updating in NotificationViewHierarchyManager. Now both the
old and new pipeline will show the correct # max notifications in
the shade. This fixes the bug where only a max 3 notifications was
showing in the notification shade when the new notification rendering
pipeline was enabled. (onKeyguard wasn't being updated and the max # of
notifications on the keyguard is 3.)
This fix includes:
- Update ExpandableNotificationRow's keyguard state via status bar state
controller listener instead of relying on a direct call to update from NVHM/StatusBar
- Removing NotificationPresenter getMaxNotificationsWhileLocked method.
Instead, if a package wants to override this value, it should override the config
value keyguard_max_notification_count
- This also fixes regular notification expansion when enabling the
new notification pipeline because now onKeyguard is being updated
appropriately, so the expansion states can evaluate correctly
Test: atest SystemUITests
Change-Id: I07593ef5d95674911f6f96a5c03cf4128a885a99
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the first step towards removing view injection from
NotificationStackScrollLayout.
For right now, this doesn't seem to do too much. This CL passes the
controller into most places the view used to go, and that in turn
simply proxies requests through to the view.
A follow up CL will work to actually remove injection from
NotificationStackScrollLayout.
Bug: 149989572
Test: atest SytemUITests && manual
Change-Id: Ic161c36f090479fb1ccbfdd735a2a9ab5616f290
|
|
|
This moves injection from NotificationShelf into its controller,
creating a subcomponent for the view.
This is a relatively light-weight version of this change. Very little
logic is actually moved into the controller, with a handful of public
methods on the controller simply acting as proxies to the view for
now.
Bug: 147245740
Test: manual && atest SystemUITests
Change-Id: Ic74dc6a2ef83bcc2e1a86080d77b156b2b3e8c15
|