Page MenuHomePhabricator

[native] Add explicit borderColor for non-selected RegistrationTile
ClosedPublic

Authored by ashoat on May 19 2023, 11:31 AM.
Tags
None
Referenced Files
F1632652: D7889.id26708.diff
Mon, Apr 22, 10:58 AM
F1632646: D7889.id26692.diff
Mon, Apr 22, 10:57 AM
F1632630: D7889.id.diff
Mon, Apr 22, 10:57 AM
F1632617: D7889.diff
Mon, Apr 22, 10:53 AM
Unknown Object (File)
Sun, Apr 7, 5:13 PM
Unknown Object (File)
Feb 23 2024, 11:57 PM
Unknown Object (File)
Feb 23 2024, 11:46 PM
Unknown Object (File)
Feb 23 2024, 10:53 PM
Subscribers

Details

Summary

This wasn't noticeable in dark mode, but is much more noticeable in light mode. Since we set a borderWidth, React Native color sets a border. I'm not sure how it determines a default color for that border, but I think it just picks black.

Depends on D7888

Test Plan
beforeafter
light mode (not accessible; just showing for context)
Screenshot 2023-05-19 at 2.33.56 PM.png (2×1 px, 868 KB)
Screenshot 2023-05-19 at 2.30.32 PM.png (2×1 px, 866 KB)
dark mode (looks mostly the same)
Screenshot 2023-05-19 at 2.39.13 PM.png (2×1 px, 865 KB)
Screenshot 2023-05-19 at 2.38.56 PM.png (2×1 px, 865 KB)

The only change in dark mode is that the tile looks slightly larger in the "after" version, since we're using the backgroundColor of the tile as the borderColor, instead of using a more black backgroundColor (which fades into the background).

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat edited the test plan for this revision. (Show Details)
ashoat edited the test plan for this revision. (Show Details)
ashoat added a reviewer: ted.
This revision is now accepted and ready to land.May 19 2023, 11:50 AM