Paths

Table of Contentst

[native] Fix appearance of `TabNavigator` in light mode
ClosedPublic
Actions

Authored by atul on Nov 10 2022, 5:21 PM.

Details

Reviewers

tomek
marcin
ginsu
rohan
• abosh
varun
ashoat

Group Reviewers

Restricted Owners Package

(Owns No Changed Paths)

Commits

rCOMM078cd822453e: [native] Fix appearance of `TabNavigator` in light mode

Summary

Context: https://linear.app/comm/issue/ENG-2052/re-introduce-light-mode-in-native

The backgroundColor of TabNavigator was previously hardcoded to a dark value that looked fine in dark mode but looked broken in light mode.

This diff sets the backgroundColor to tabBarBackground which will adapt based on the theme.

Test Plan

Before:

After:

Things look as expected.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Nov 10 2022, 5:21 PM

Herald added a subscriber: ashoat. · View Herald TranscriptNov 10 2022, 5:21 PM

atul edited the test plan for this revision. (Show Details)Nov 10 2022, 5:22 PM

Harbormaster completed remote builds in B13386: Diff 18361.Nov 10 2022, 5:34 PM

atul requested review of this revision.Nov 10 2022, 5:34 PM

ashoat added inline comments.Nov 10 2022, 6:48 PM

native/navigation/tab-navigator.react.js
94 ↗	(On Diff #18361)	Think this should be changed in light mode, it's kinda hard to tell which tab is open. Not sure if that's meant for a separate diff tho

atul added inline comments.Nov 10 2022, 7:01 PM

native/navigation/tab-navigator.react.js
94 ↗	(On Diff #18361)	Yeah agree that change is in: https://phab.comm.dev/D5615

great fix

This revision is now accepted and ready to land.Nov 11 2022, 8:52 AM

atul added a reviewer: Restricted Owners Package.Nov 11 2022, 8:53 AM

This revision now requires review to proceed.Nov 11 2022, 8:53 AM

ashoat accepted this revision.Nov 11 2022, 1:02 PM

This revision is now accepted and ready to land.Nov 11 2022, 1:02 PM

Closed by commit rCOMM078cd822453e: [native] Fix appearance of `TabNavigator` in light mode (authored by atul). · Explain WhyNov 11 2022, 1:09 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM078cd822453e: [native] Fix appearance of `TabNavigator` in light mode.

Revision Contents
Changeset List

Path

Size

native/

navigation/

tab-navigator.react.js

26 lines

Diff 18382

View Options

native/navigation/tab-navigator.react.js

[native] Fix appearance of `TabNavigator` in light modeClosedPublicActions