Differential D13477

[native] fix action sheet padding
ClosedPublic
Actions

Authored by varun on Sep 26 2024, 3:50 AM.

Details

Reviewers

ashoat

Commits

rCOMMdf4780d7fc65: [native] fix action sheet padding

Summary

figured out why the home pill was covering the action sheet options

this was broken in D7506

Test Plan

action sheet looks right on android now

Screenshot_1727322367.png (3×1 px, 161 KB)

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Sep 26 2024, 3:50 AM

Herald added a subscriber: tomek. · View Herald TranscriptSep 26 2024, 3:50 AM

varun edited the test plan for this revision. (Show Details)Sep 26 2024, 3:50 AM

varun attached a referenced file: F2811477: Screenshot_1727322367.png. (Show Details)

varun mentioned this in D13476: [native] add farcaster icon on native.Sep 26 2024, 3:52 AM

Harbormaster completed remote builds in B31863: Diff 44580.Sep 26 2024, 4:07 AM

varun requested review of this revision.Sep 26 2024, 4:07 AM

Nice find!

This revision is now accepted and ready to land.Sep 26 2024, 1:03 PM

Closed by commit rCOMMdf4780d7fc65: [native] fix action sheet padding. · Explain WhySep 26 2024, 2:41 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMdf4780d7fc65: [native] fix action sheet padding.

Revision Contents
Changeset List

Path

Size

native/

avatars/

avatar-hooks.js

2 lines

Diff 44580

View Options

native/avatars/avatar-hooks.js

[native] fix action sheet paddingClosedPublicActions