Closed Bug 1190683 Opened 9 years ago Closed 9 years ago

"start communicating" is odd phrasing

Categories

(Firefox OS Graveyard :: Gaia::Dialer, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(tracking-b2g:backlog, b2g-v2.5 affected, b2g-master verified)

RESOLVED FIXED
2.6 S2 - 12/4
tracking-b2g backlog
Tracking Status
b2g-v2.5 --- affected
b2g-master --- verified

People

(Reporter: morpheus, Assigned: oleg.slepov)

References

Details

Attachments

(4 files)

Attached image 2015-08-04-10-59-58.png
"The Messaging app's "Start communicating" call to action trips me up because it is almost actionable, but not quite, therefore it might be a bad CTA. "communicating" is an abstract concept ("expressing ideas"), not a concrete action.

"Start by writing a message" or something similar would be a concrete thing I can achieve with this app, and seems to be a better CTA."  - Fred Wenzel 

The Phone app has the same issue, so filing this bug for consistency.
Blocks: 1179464
Please find the attachment for the updated welcome message.
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

Gabriele, please review the changes or ask someone else to proceed with review. Thanks in advance.
Attachment #8691253 - Flags: review?(gsvelto)
Assignee: nobody → oleg.slepov
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

I think that you should change the l10n-id if you're changing the string so that the people working on the translations can spot the changes. I'm also not sure if "Start a conversation now" is appropriate here since this is the call log, not the messaging app. Zibi can you give us your feedback here?
Attachment #8691253 - Flags: feedback?(gandalf)
As per the PDF file attached by Morpheus the sentense should be "Start a conversation now". 

In regards to new l10n-id, we checked that this string is not used anywhere except this screen, so no other functionality is affected. If it is necessary to change l10n-id anyway, it is fine and we will make the required changes. Please advice how to proceed here.
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

yup, update the ID please.
Attachment #8691253 - Flags: feedback?(gandalf)
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

OK, thanks for the clarification Zibi. Alexey please also change the l10n id and ask for review again, thanks.
Attachment #8691253 - Flags: review?(gsvelto) → review-
We noticed that there is a Chinese version of such string avilable here (in test folder): gaia/build/test/resources/locales/zh-CN/apps/communications/dialer/dialer.properties

Please advise whether we should keep this file as it is now or should we request someone to provide a traslation?
(In reply to Alexey Yakimov from comment #8)
> We noticed that there is a Chinese version of such string avilable here (in
> test folder):
> gaia/build/test/resources/locales/zh-CN/apps/communications/dialer/dialer.
> properties
> 
> Please advise whether we should keep this file as it is now or should we
> request someone to provide a traslation?

I have no idea where that's coming from; I was under the impression that we had removed all non-English localizations from the sources. Zibi, sorry to bother you again but do you happen to know anything about those files?
Flags: needinfo?(gandalf)
Please, keep the file as is. It's just a fixture used for build tests.
Flags: needinfo?(gandalf)
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

The commit has been updated with new localization id, please review.
Attachment #8691253 - Flags: review- → review?(gsvelto)
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

This works but it's a bit excessive; please only change the l10n id. You don't need to change the element id in index.html, that can still be called no-result-msg2 so that you won't have to change the JS code and the test too. Last but not least remove the no-logs-msg-2 l10n id since we won't be using it anymore.
Attachment #8691253 - Flags: review?(gsvelto) → review-
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

The requested changes have been made, hope it is fine now, please review.
Attachment #8691253 - Flags: review- → review?(gsvelto)
Comment on attachment 8691253 [details] [review]
[gaia] ruoslepov:bug-1190683-start-communicating-is-odd-phrasing > mozilla-b2g:master

Excellent, thanks!
Attachment #8691253 - Flags: review?(gsvelto) → review+
Keywords: checkin-needed
https://github.com/mozilla-b2g/gaia/commit/1753636c36d782663773fd8376c9af769bf83045
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.6 S2 - 12/4
This bug has been verified as "pass" on the latest build of Flame master and Aries KK master by the STR in comment 0.

Actual results: It shows "Start a conversation now" in call log.
See attachment: verified_Flame_master_calllog.png
Reproduce rate: 0/10


Device: Flame master_512mb (Pass)
Build ID               20151206150208
Gaia Revision          b930e25e9f2e5ec2106dd31005468575cdf87d50
Gaia Date              2015-12-05 03:41:55
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/cc9c6cd756cb744596ba039dcc5ad3065a7cc3ea
Gecko Version          45.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20151206.184313
Firmware Date          Sun Dec  6 18:43:25 EST 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device: Aries KK master (Pass)
Build ID               20151206002630
Gaia Revision          b930e25e9f2e5ec2106dd31005468575cdf87d50
Gaia Date              2015-12-05 03:41:55
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/cc9c6cd756cb744596ba039dcc5ad3065a7cc3ea
Gecko Version          45.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151205.234548
Firmware Date          Sat Dec  5 23:45:56 UTC 2015
Bootloader             s1
QA Whiteboard: [MGSEI-Triage+]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: