Page MenuHomePhabricator

[native] Rename avatar action sheet actions
ClosedPublic

Authored by ashoat on Jun 1 2023, 1:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 3, 9:26 AM
Unknown Object (File)
Wed, Apr 3, 9:26 AM
Unknown Object (File)
Wed, Apr 3, 9:25 AM
Unknown Object (File)
Wed, Apr 3, 9:19 AM
Unknown Object (File)
Feb 11 2024, 2:34 AM
Unknown Object (File)
Feb 11 2024, 2:33 AM
Unknown Object (File)
Feb 11 2024, 2:30 AM
Unknown Object (File)
Feb 11 2024, 12:06 AM
Subscribers

Details

Summary

Initially I just wanted to rename the "Reset to default" option for clarity, but ended up making several changes:

  1. Rename "Clear avatar" to "Reset to default" to avoid implying to the user that they can use Comm without an avatar
    • In the "update avatar" flow, the verb "clear" makes more sense since it clears changes. In the registration flow, it seems more ambiguous... feels like it might imply that you can register without an avatar
  2. Be more consistent with capitalization
  3. Start each option with a verb for consistency
Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable