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

Fixed grammar in 2 English strings #6132

Merged
merged 2 commits into from
May 30, 2022
Merged

Fixed grammar in 2 English strings #6132

merged 2 commits into from
May 30, 2022

Conversation

jtbx
Copy link
Contributor

@jtbx jtbx commented May 24, 2022

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other: English string fix

Content

Fixed 2 strings in English; room_list_people_empty_body and room_list_rooms_empty_body in /vector/src/main/res/values/strings.xml

Motivation and context

I'm submitting this PR because of the other PR I made in SchildiChat/SchildiChat-android: SchildiChat#125

Testing

Testing isn't necessary

Checklist

Signed-off-by: Jeremy Baxter [email protected]

Jeremy added 2 commits May 24, 2022 14:12
`room_list_people_empty_body` and `room_list_rooms_empty_body`
@bmarty bmarty added the Z-Community-PR Issue is solved by a community member's PR label May 24, 2022
@bmarty bmarty requested a review from ouchadam May 25, 2022 09:27
<string name="room_list_rooms_empty_title">Rooms</string>
<string name="room_list_rooms_empty_body">Your rooms will be displayed here. Tap the + bottom right to find existing ones or start some of your own.</string>
<string name="room_list_rooms_empty_body">Your rooms will be displayed here. Tap the + at the bottom right to find existing ones or start some of your own.</string>
Copy link
Member

Choose a reason for hiding this comment

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

Note to myself: add a comment above this 2 strings to let translators knows that on RTL languages the button will be at bottom left.

There are some mistake currently about that:

image

Since this string will be updated, the translators will have the opportunity to fix that.

Copy link
Contributor

Choose a reason for hiding this comment

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

this copy has been added to the FTUE copy review document, potentially we could avoid specifying the exact location of the fab (or even the fab at all 🤔 )

@ouchadam
Copy link
Contributor

The original screen for context

Screenshot_20220525_103337

@ouchadam
Copy link
Contributor

for context, we're about to start a copy review of the new onboarding process and this suggestion has highlighted that the current empty screen copy could be further improved, we'll add it to the review process

I'm happy to include this change but it may change again~!

@bmarty bmarty merged commit 483b1ab into element-hq:develop May 30, 2022
@bmarty bmarty mentioned this pull request May 30, 2022
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants