Page MenuHomePhabricator
Differential D6538

[native] add SafeAreaView to MessageReactionsModal
ClosedPublic
Actions

Authored by ginsu on Feb 2 2023, 12:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 14, 10:21 AM
Unknown Object (File)
Thu, Jun 13, 10:59 PM
Unknown Object (File)
Thu, Jun 13, 10:59 PM
Unknown Object (File)
Thu, Jun 13, 7:27 PM
Unknown Object (File)
Tue, Jun 11, 8:29 AM
Unknown Object (File)
Sun, Jun 9, 6:46 AM
Unknown Object (File)
Fri, Jun 7, 9:11 PM
Unknown Object (File)
Fri, Jun 7, 9:10 PM
View All 63 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
atul
tomek
rohan
Commits
rCOMMfea59cff31f5: [native] add SafeAreaView to MessageReactionsModal
Summary

add SafeAreaView to MessageReactionsModal. The SafeAreaView pushes the container of MessageReactionsModal up a bit to a place where things like the Android nav buttons won't overlap with the content of the modal

Linear Task: ENG-2846

Test Plan

Please review the screenshots below to see the changes I made

Before:

iOS:

Screenshot 2023-02-02 at 3.32.11 PM.png (1×1 px, 852 KB)

Android:

Screenshot 2023-02-02 at 1.17.10 PM.png (1×978 px, 415 KB)

After:

iOS:

Screenshot 2023-02-02 at 3.30.59 PM.png (1×1 px, 850 KB)

Android:

Screenshot 2023-02-02 at 3.29.27 PM.png (1×978 px, 422 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Feb 2 2023, 12:37 PM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptFeb 2 2023, 12:37 PM
ginsu edited the summary of this revision. (Show Details)Feb 2 2023, 12:41 PM
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: atul, tomek, rohan.
ginsu attached a referenced file: F358614: Screenshot 2023-02-02 at 3.29.27 PM.png. (Show Details)
ginsu attached a referenced file: F358613: Screenshot 2023-02-02 at 3.30.59 PM.png. (Show Details)
ginsu attached a referenced file: F358612: Screenshot 2023-02-02 at 1.17.10 PM.png. (Show Details)
ginsu attached a referenced file: F358611: Screenshot 2023-02-02 at 3.32.11 PM.png. (Show Details)
Harbormaster completed remote builds in B16028: Diff 21900.Feb 2 2023, 12:49 PM
ginsu requested review of this revision.Feb 2 2023, 12:49 PM
atul accepted this revision.Feb 2 2023, 3:46 PM
This revision is now accepted and ready to land.Feb 2 2023, 3:46 PM
Closed by commit rCOMMfea59cff31f5: [native] add SafeAreaView to MessageReactionsModal (authored by ginsu). · Explain WhyFeb 2 2023, 8:13 PM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMMfea59cff31f5: [native] add SafeAreaView to MessageReactionsModal.
ashoat added inline comments.Feb 3 2023, 10:55 AM
native/chat/message-reactions-modal.react.js
65

There's no reason to nest a SafeAreaView inside another SafeAreaView

ashoat added inline comments.Feb 3 2023, 11:03 AM
native/chat/message-reactions-modal.react.js
65

Never mind, ignore this! I confused myself

Revision Contents
Changeset List

PathSize
native/
chat/
40 lines

Diff 21925

View Options

native/chat/message-reactions-modal.react.js

Loading...