HomePhabricator
Diffusion Comm f79fafb48068

[native] Use currently selected theme for alerts.

Tags
None
Referenced Files
F646511: t2.png
Aug 3 2023, 12:02 PM
File Not Attached
F646510: t1.png
Aug 3 2023, 12:02 PM
File Not Attached
Subscribers
None

Description

[native] Use currently selected theme for alerts.

Summary:
Solution for ENG-3935.

This diff introduces a wrapper for React Native alerts, which synchronizes currently selected theme in the app with alert style. Right now userInterfaceStyle is only supported on iOS.

Light theme alert:

t1.png (956×453 px, 117 KB)

Dark theme alert:
t2.png (956×453 px, 113 KB)

Test Plan:

  1. Select a location where you want to implement an alert (for example, you can add useEffect in root.react.js).
  2. Verify whether the alert theme is consistent with the app theme.
  3. Navigate to profile > appearance, alter the theme, and check if the same transformation applies to the other theme.

Reviewers: bartek, tomek, michal, inka

Reviewed By: tomek, inka

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D8586

Details

Provenance
patrykAuthored on Aug 3 2023, 11:15 AM
Reviewer
tomek
Differential Revision
D8586: [native] Use currently selected theme for alerts.
Parents
rCOMM6421b301872e: [web] Warn the user when deleting a chat that has contained chats
Branches
Unknown
Tags
Unknown