Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ELI5] Tweak wording on the signup screen #7136

Merged
merged 2 commits into from
Dec 17, 2024
Merged

[ELI5] Tweak wording on the signup screen #7136

merged 2 commits into from
Dec 17, 2024

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Dec 17, 2024

People get confused here unsure what "handle" is. "Input your handle" is also confusing.

The first commit replaces the wordings.

Screenshot 2024-12-17 at 00 02 59

This matches our usage of "username" on the login screen.

The second commit is a bit unusual — I'm updating all translations as if this was the wording all along. That's to avoid regressions to translations just before the release. It's fine to update them later (if needed per language) since conceptually the phrasing is the same and the meaning has not changed.

It's a little inconsistent with what we have in Settings. Maybe in the future we'll change it there too. But for now let's make sure people are able to create an account.

Test Plan

Verify above for English.

Then yarn intl:build and verify translations have not regressed and show old strings:

Screenshot 2024-12-17 at 00 01 50

@arcalinea arcalinea temporarily deployed to your-username - social-app PR #7136 December 17, 2024 00:05 — with Render Destroyed
Copy link

Old size New size Diff
6.78 MB 6.78 MB 0 B (0.00%)

@gaearon gaearon merged commit c9e4297 into main Dec 17, 2024
6 checks passed
@gaearon gaearon deleted the your-username branch December 17, 2024 00:11
@gaearon
Copy link
Collaborator Author

gaearon commented Dec 17, 2024

got verbal ok from @pfrazee

Copy link
Member

@mozzius mozzius left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code lgtm for what it's worth

drash-course added a commit to drash-course/social-app that referenced this pull request Dec 18, 2024
* main: (58 commits)
  Fix tests
  Layout tweaks (bluesky-social#7150)
  Trending (Beta) (bluesky-social#7144)
  Fix emoji picker position (bluesky-social#7146)
  Tweak Follow dialog Search placeholder (bluesky-social#7147)
  New progress guide - 10 follows (bluesky-social#7128)
  Pipe statsig events to logger (bluesky-social#7141)
  Fix notifications borders (bluesky-social#7140)
  Refetch empty feed on focus (bluesky-social#7139)
  Read storage on window.onstorage (bluesky-social#7137)
  [ELI5] Tweak wording on the signup screen (bluesky-social#7136)
  alf error screen (bluesky-social#7135)
  add safe area view to profile error screen (bluesky-social#7134)
  Adjust gates (bluesky-social#7132)
  disable automaticallAdjustsScrollIndicatorInsets (bluesky-social#7131)
  Bump more native deps (bluesky-social#7129)
  Update more Expo packages (bluesky-social#7127)
  feat: widen recent search profile link for mobile devices (bluesky-social#7119)
  Fix video uploads on native (bluesky-social#7126)
  Fix post time localization on Android (bluesky-social#6742)
  ...

# Conflicts:
#	src/view/com/profile/ProfileSubpageHeader.tsx
#	src/view/screens/ProfileList.tsx
Signez pushed a commit to Signez/bsky-social-app that referenced this pull request Dec 26, 2024
* Tweak signup wordings

* Update translations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants