HomePhabricator
Diffusion Comm ea404a7f7f31

[web][native] Update invite link copy

Tags
None
Referenced Files
F620630: 9c.png
Jul 10 2023, 5:49 AM
File Not Attached
F620629: 9b.png
Jul 10 2023, 5:49 AM
File Not Attached
F620628: 9a.png
Jul 10 2023, 5:49 AM
File Not Attached
F620627: 8a.png
Jul 10 2023, 5:49 AM
File Not Attached
F620626: 7.png
Jul 10 2023, 5:49 AM
File Not Attached
F620641: 62.png
Jul 10 2023, 5:49 AM
File Not Attached
F620625: 6.png
Jul 10 2023, 5:49 AM
File Not Attached
F620624: 5b.png
Jul 10 2023, 5:49 AM
File Not Attached
Subscribers
None

Description

[web][native] Update invite link copy

Summary:
Solves https://linear.app/comm/issue/ENG-4304/update-invite-links-copy

  1. web/invite-links/accept-invite-modal.react.js
    1. buttons in Comm typically don't capitalize every word. "Accept Invite" should probably be "Accept invite"
      1a.png (766×956 px, 44 KB)
    2. the string "This invite link may be expired, please try again with another invite link" contains a grammatical error called a "comma splice". perhaps it should be replaced with two sentences: "This invite link may be expired. Please try again with another invite link"
      1b.png (688×1 px, 51 KB)
  2. web/invite-links/invite-links-menu.react.js
    1. buttons shouldn't capitalize every word. "Invite Link" should probably be "Invite link"
      2.png (232×422 px, 17 KB)
  3. web/invite-links/view-invite-link-modal.react.js
    1. I worry about including user-generated content in modal titles. "Invite people to ${resolvedThreadInfo.uiName}" could perhaps just be "Invite link"? this one is hard to judge without a screenshot... would it be confusing to not see the community name anywhere?
      3a.png (530×924 px, 51 KB)
      3a2.png (554×1 px, 54 KB)
    2. I'd like to replace "Use this public link to invite your friends into the community!" with "Share this invite link with your friends to help them join your community!", but I'm not sure if it's too long. would like to see a screenshot
      3b.png (540×1 px, 58 KB)
  4. native/invite-links/invite-links-button.react.js
    1. buttons shouldn't capitalize every word. "Invite Link" should probably be "Invite link", and "Manage Invite Links" should be "Manage invite links"
      4.png (195×400 px, 25 KB)
  5. native/navigation/invite-link-modal.react.js
    1. buttons shouldn't capitalize every word. "Accept Invite" should probably be "Accept invite"
      5a.png (289×392 px, 17 KB)
    2. "This invite link may be expired, please try again with another invite link" contains a comma splice". we should update it to match my recommendation on web above. (separately, should we store this copy somewhere in lib to avoid copy-pasting between web and native?)
      5b.png (258×396 px, 18 KB)
  6. native/invite-links/view-invite-links-header-title.react.js
    1. same concern as web/invite-links/view-invite-link-modal.react.js above. can we avoid including the community name in the title?
      6.png (209×404 px, 17 KB)
      62.png (248×404 px, 15 KB)
  7. native/invite-links/invite-links-navigator.react.js
    1. "Public Link" should probably be "Public link". I think we take the same approach to capitalization of titles that we do with buttons
      7.png (486×406 px, 40 KB)
  8. native/invite-links/view-invite-links-screen.react.js
    1. same as web: "Use this public link to invite your friends into the community!" should be "Share this invite link with your friends to help them join your community!", if there is enough space for it
      8a.png (233×406 px, 16 KB)
  9. native/invite-links/manage-public-link-screen.react.js
    1. I think we can remove this sentence: "Members who have your community’s public link but have not joined will not able to with the disabled link." the Alert is already too long, and I think this should be obvious to the reader
      9a.png (239×267 px, 65 KB)
    2. "Other communities may also claim your previous public link url." <-- first of all, URL should always be capitalized. I think we can replace this with "Other communities will be able to claim the same URL."
      9b.png (234×263 px, 64 KB)
    3. the text that starts with "Let your community be more accessible" should be replaced: Invite links make it easy for your friends to join your community. Anybody who knows your community's invite link will be able to join it. Note that if you change your public link's URL, other communities will be able to claim the old URL.
      9c.png (458×402 px, 37 KB)

Depends on D8419

Test Plan: Check if the app still works, but most of the changes is only about changing text.

Reviewers: kamil, inka, ashoat

Reviewed By: ashoat

Subscribers: ted

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

Details

Provenance
tomekAuthored on Jul 4 2023, 7:24 AM
Reviewer
ashoat
Differential Revision
D8423: [web][native] Update invite link copy
Parents
rCOMMc38470e72f08: [web] Introduce disable link modal
Branches
Unknown
Tags
Unknown