Page MenuHomePhabricator
Differential D7680

[native] Used predefined function for discard edit mode alert
ClosedPublic
Actions

Authored by • kuba on Apr 28 2023, 2:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 3, 11:32 AM
Unknown Object (File)
Mon, Dec 2, 5:30 PM
Unknown Object (File)
Mon, Dec 2, 2:16 PM
Unknown Object (File)
Sat, Nov 30, 8:14 PM
Unknown Object (File)
Thu, Nov 21, 8:18 PM
Unknown Object (File)
Oct 30 2024, 6:20 PM
Unknown Object (File)
Oct 15 2024, 6:26 PM
Unknown Object (File)
Oct 1 2024, 11:00 PM
View All 88 Files
Subscribers
atul
tomek

Details

Reviewers
michal
inka
ashoat
Commits
rCOMM3c31b6487e39: [native] Used predefined function for discard edit mode alert
Summary

Used function, instead of creating a new alert.

Test Plan

Checked if the alert still works.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• kuba created this revision.Apr 28 2023, 2:22 AM
Herald added subscribers: atul, tomek. · View Herald TranscriptApr 28 2023, 2:22 AM
• kuba added a parent revision: D7630: [native] Blocking navigation in edit mode in ChatRouter.Apr 28 2023, 2:23 AM
Harbormaster completed remote builds in B18909: Diff 25901.Apr 28 2023, 2:40 AM
• kuba requested review of this revision.Apr 28 2023, 2:40 AM
ashoat requested changes to this revision.Apr 28 2023, 5:45 AM
ashoat added inline comments.
native/chat/chat-input-bar.react.js
970 ↗(On Diff #25901)

I think "Are you sure?" is too generic of a title here. It doesn't clarify what the user is doing exactly

I think I prefer the copy on line 970 and 971, but with some changes:

Alert.alert(
  'Discard changes?',
  'You have unsaved changes which will be discarded if you navigate away.',
);

We can keep this change here, but can you update exitEditAlert to match this proposed new copy?

This revision now requires changes to proceed.Apr 28 2023, 5:45 AM
• kuba updated this revision to Diff 26347.May 10 2023, 6:51 AM

Changed discard alert content

Harbormaster completed remote builds in B19245: Diff 26347.May 10 2023, 7:08 AM
• kuba added a child revision: D7789: [native] Introduced new types for Reanimated.May 11 2023, 2:54 AM
• kuba removed a parent revision: D7630: [native] Blocking navigation in edit mode in ChatRouter.May 11 2023, 3:19 AM
ashoat accepted this revision.May 11 2023, 8:42 AM
This revision is now accepted and ready to land.May 11 2023, 8:42 AM
Closed by commit rCOMM3c31b6487e39: [native] Used predefined function for discard edit mode alert (authored by kosmydel <kukubaczek@gmail.com>). · Explain WhyMay 16 2023, 3:49 AM
This revision was automatically updated to reflect the committed changes.
kosmydel <kukubaczek@gmail.com> added a commit: rCOMM3c31b6487e39: [native] Used predefined function for discard edit mode alert.

Revision Contents
Changeset List

PathSize
native/
chat/
14 lines
utils/
28 lines

Diff 26505

View Options

native/chat/chat-input-bar.react.js

Loading...
View Options

native/utils/edit-messages-utils.js

Loading...