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

Change "initially paused" preference description #220

Merged
merged 3 commits into from
Jan 10, 2023

Conversation

Twoomatch
Copy link
Contributor

Since we have separate strings for on/off states, I think it's best to tell user what will happen in the current state, not what they can tell us to do (subtle difference between imperative "start in the paused state" and "will be paused after starting").

#219 has the same wording now, and I thought the English version might use some changes as well.

The "on" description could also have the "displayed during a call" part, but this would get too long, wouldn't it?

Since we have separate strings for on/off states, I think it's best to tell user what _will_ happen in the current state, not what they can _tell_ us to do (subtle difference between imperative "start in the paused state" and "will be paused after starting").

chenxiaolong#219 has more or less the same wording, and I thought the English version might use some changes as well.
Copy link
Owner

@chenxiaolong chenxiaolong left a comment

Choose a reason for hiding this comment

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

Thanks, I like this wording a lot better. Just minor change needed (see above).

app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
@chenxiaolong chenxiaolong merged commit b0653f4 into chenxiaolong:master Jan 10, 2023
chenxiaolong added a commit that referenced this pull request Jan 10, 2023
Signed-off-by: Andrew Gunnerson <chillermillerlong@hotmail.com>
@Twoomatch Twoomatch deleted the patch-3 branch January 10, 2023 20:52
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.

2 participants