Page MenuHomePhabricator

[native] Rename avatar action sheet actions
ClosedPublic

Authored by ashoat on Jun 1 2023, 1:57 PM.
Tags
None
Referenced Files
F1693497: D8065.diff
Thu, May 2, 3:44 PM
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
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
Branch
ashoat/avatars
Lint
No Lint Coverage
Unit
No Test Coverage