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

add strings needed for instant onboarding #3001

Merged
merged 6 commits into from
Apr 29, 2024
Merged

Conversation

r10s
Copy link
Member

@r10s r10s commented Apr 22, 2024

this PR extracts new strings from deltachat/deltachat-desktop#3773.

the following strings were not added as they are existing already in a sufficient way (desktop PR needs to be changed):

onboarding_title -> welcome_desktop (exclamation mark can be added later)
onboarding_subtitle -> welcome_chat_over_email (dot can be removed later)
onboarding_scan_qr_code -> qrscan_title
multidevice_setup_option -> multidevice_receiver_title
import_backup_option -> import_backup_title (iirc, it was renamed to "restore" on purpose, let's stay with that for now)
instant_onboarding_confirm_label -> ok (this is used in a message box, right? i would stay calm there, otherwise use the other "Let's Get Started")

for the other strings, i stayed with existing keys - they're good enough and i did not want to add additional noise.

for upper/lower case: we voted for capitalising titles and buttons at some point and do not want to change this currently; i adapted the strings accordingly.

@adzialocha will add some screenshots to deltachat/deltachat-desktop#3773 so one can get a better idea

closes #2995

@r10s r10s added the enhancement actually in development, user visible enhancement label Apr 22, 2024
@r10s r10s force-pushed the instant-onboarding-strings branch from ac26898 to 583200d Compare April 22, 2024 16:53
@adzialocha
Copy link
Member

Thank you for this @r10s, I've adjusted the strings now in the PR, they should reflect this current state

Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

@r10s
Copy link
Member Author

r10s commented Apr 29, 2024

ftr, the strings are discussed with at least @adzialocha and @hpk42

@adzialocha adzialocha self-requested a review April 29, 2024 14:28
@r10s r10s merged commit 862d96e into main Apr 29, 2024
2 checks passed
@r10s r10s deleted the instant-onboarding-strings branch April 29, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement actually in development, user visible enhancement
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

add and refine instant onboarding strings
3 participants