935976 - [first run] tutorial: "fi" in "notifications" is one character, should be two (= "notifications") and other nitpicking

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::First Time Experience, defect)

Description

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

In gaia 1.2 and on master:

tutorial-step3-large = Swipe down from top to access recent notifications, credit information and settings.
"fi" is one letter, should be two (it's bug 815084 calling).

tutorial-step1-large = Swipe left or right to discover new apps on home screen
The full stop at the end is missing. If you regard "home screen" as location it's fine like this, if it should be an object, it should be "the home screen" in my opinion (be aware that I am not a native speaker).

tutorial-step4-large = Tap on an icon to access application.
There should be a "the" between "access" and "application".

tutorial-step5-large = Swipe up from bottom to go back to home screen.
Same for "home screen" like for step1.

Because none of these changes alter the meaning of the string or how it should be translated, the entities should remain unchanged.

Comment 1

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Attachment: pull request

Comment 2

[Fernando Campo (:fcampo)]

Comment on attachment 8355923 [details] [review]
pull request

Nice to come back from holidays and seeing this little nits fixed :)
Thanks!

Comment 3

[Ryan VanderMeulen [:RyanVM]]

Master: https://github.com/mozilla-b2g/gaia/commit/6bf6def868fd1671f9fec8f62a33da564e32d244

Comment 5

[Toni Hermoso Pulido]

I still see weird characters in en-US strings repo:
http://hg.mozilla.org/gaia-l10n/en-US/file/1c25a98e1383/apps/communications/ftu/ftu.properties

On the other hand, I don't know whether we would be still on time to port it to 1.3 as well.

Comment 6

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Attachment: pull request to kill again this issue, v1

Comment 7

[Ryan VanderMeulen [:RyanVM]]

Github can't merge this due to conflicts. Please rebase. Also, IMHO, this patch should really be in a new bug. Having two different commits 2 months apart in the same bug makes tracking a PITA. Thanks!

Comment 8

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Closing this bug, filed bug 988900 for the second incarnation.