[native] [33/34] ReactNav6: Update HeaderTitle to use flex positioning
ClosedPublic
Actions

Authored by ashoat on Oct 20 2022, 1:21 PM.

Details

Reviewers

atul
tomek
bartek

Commits

rCOMMce393d6b5986: [native] [33/34] ReactNav6: Update HeaderTitle to use flex positioning

Summary

The relevant part here is actually just removing headerTitleContainerStyle... without that, the HeaderTitle was way too narrow.

But I also did the work to forward props along since it seemed like the right thing.

Context on change here.

Depends on D5425

Test Plan

Flow + ran the app on my phone and played around with it

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Oct 20 2022, 1:21 PM

Herald added a subscriber: • abosh. · View Herald TranscriptOct 20 2022, 1:21 PM

ashoat added a child revision: D5427: [native] [34/34] ReactNav6: Update package versions.Oct 20 2022, 1:21 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Oct 20 2022, 1:43 PM

Harbormaster failed remote builds in B12913: Diff 17704!

Build failures are caused by use of @react-navigation/elements in D5420, which isn't introduced until D5427. Sorry!!

atul accepted this revision.Oct 20 2022, 8:20 PM

This revision is now accepted and ready to land.Oct 20 2022, 8:20 PM

This revision was landed with ongoing or failed builds.Oct 20 2022, 8:48 PM

Closed by commit rCOMMce393d6b5986: [native] [33/34] ReactNav6: Update HeaderTitle to use flex positioning. · Explain Why

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMce393d6b5986: [native] [33/34] ReactNav6: Update HeaderTitle to use flex positioning.

Revision Contents
Changeset List

Path

Size

native/

chat/

chat.react.js

7 lines

message-list-header-title.react.js

41 lines

Diff 17704

View Options

native/chat/chat.react.js