summaryrefslogtreecommitdiff
path: root/tools/codegen/src
diff options
context:
space:
mode:
author Jeff DeCew <jeffdq@google.com> 2022-07-15 22:41:02 +0000
committer Jeff DeCew <jeffdq@google.com> 2023-04-26 19:33:28 -0400
commitc0823b6c0b2f5216c4f742120cbf30de3b6bc3ef (patch)
tree89070b18b9071b366739de8d313e7ca6488b702d /tools/codegen/src
parent1cdd2a76ae5f82b84e1ed2f5024b370a5f251faa (diff)
Fix LightBarController's dark/light logic for the Navigation Bar
The old logic failed to account for the notification shade's white scrim in light theme appearing underneath the navigation bar and didn't correctly keep the navigation bar in light mode for that state, making for white buttons on a nearly white background. However, when accounting for the new possibility of an override to light mode, a pair of bug was discovered that cancelled each other out but made intermediate logic very confusing to follow. It's still confusing, but now it's more correct. * GradientColors.supportsDarkText() was set backward by the ScrimController. This has been fixed. * Clarified in the docs for the fields that mNavigationLight and related fields use "Light" to mean the theme / background, which means that the button color would be dark. These locals were interpreted incorrectly in some places, cancelling out the above incorrectness. * We never had light background scrims before, but we do now, so the other main thing this logical change does is introduce the ability for the scrim to override the theme to light (thus forcing the buttons to be dark). * GlobalActionsDialogLite needed to be updated to let the LightBarController know when it was visible, since that would require light status bars. * Finally, I added a ton of debug logging (off by default) and more info in the dumps that allows validating the progression of the logic for any state. Bug: 195622153 Test: atest LightBarControllerTest Flag: NEW_LIGHT_BAR_LOGIC Change-Id: I380a8d03fdebace7dacba5f6c0b610540acc0c5a
Diffstat (limited to 'tools/codegen/src')
0 files changed, 0 insertions, 0 deletions