diff options
| author | 2022-07-14 12:54:23 -0400 | |
|---|---|---|
| committer | 2022-07-29 17:35:01 -0700 | |
| commit | 30649549dd72c9fb0b94239d8fe379a856e8bcef (patch) | |
| tree | 9001449c7cae743461b148ef0ed27298c08ff750 /libs/hwui/FrameInfoVisualizer.cpp | |
| parent | bd614b00a0afadea10fb106789a7a02a43e767ff (diff) | |
Fix dot/icon overlap in notification shelf
This bug was caused by an off-by-one error where
isOverflowing = iconX > overflowX
should have been
isOverflowing = iconX >= overflowX
As a result, the icon that should have been hidden
by the dot was considered "not overflowing"
(iconX=overflowX) so it remained visible.
This change corrects the definition of isOverflowing;
refactors isOverflowing and forceOverflow out of
calculateIconXTranslations; adds tests.
Bug: 232056690
Test: NotificationIconContainer
Test: add notifs to short shelf until overflow
swipe notifs away until short shelf empty
(no dot/icon overlap)
Test: swipe down on ls notif then let go
swipe down on ls notif to go to full shade
add notifs/swipe away notifs in full shade
scroll in full shade with long shelf
(no regressions)
Change-Id: Ie270c55f6c9d27b5d0fee73c22fedb5335760179
Diffstat (limited to 'libs/hwui/FrameInfoVisualizer.cpp')
0 files changed, 0 insertions, 0 deletions