Page MenuHomePhabricator

[native] Unify alert behavior when changing selected edited message
ClosedPublic

Authored by kuba on May 31 2023, 2:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 21, 10:38 AM
Unknown Object (File)
Thu, Nov 21, 10:37 AM
Unknown Object (File)
Thu, Nov 21, 10:37 AM
Unknown Object (File)
Thu, Nov 21, 10:37 AM
Unknown Object (File)
Oct 22 2024, 12:54 PM
Unknown Object (File)
Oct 22 2024, 12:53 PM
Unknown Object (File)
Oct 22 2024, 12:52 PM
Unknown Object (File)
Oct 22 2024, 5:01 AM
Subscribers

Details

Summary
  • removed confirmation alert, when changing edited message and didn't make changes to the previous message,
  • removed the Edit button for the currently edited message.
Test Plan

Tested on iOS:

  • entered edit mode,
  • tried to edit the same message, checked if the edit button doesn't appear,
  • made some changes in the edited message,
  • tried to edit another message, checked if the confirmation alert is displayed,
  • entered edit mode, didn't make changes, checked if can select another message to edit without confirmation alert,

Diff Detail

Repository
rCOMM Comm
Branch
kuba/check-enter-edit
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

native/chat/chat-input-bar.react.js
847–848

I had to add newText, to always have the latest version of the draft. It is used when we want to both set a new text state and check if the message has any changes (line 737).

This revision is now accepted and ready to land.Jun 5 2023, 3:12 AM