Change "Read" header in Reading List

Change "Read" header in Reading List

Change "Read" header to a phrase in reading to be clearer in all languages.

BUG=673335
Committed: https://crrev.com/a13fc8e6a3b66d2f08b4ffde5a59277c49fe366a
Cr-Commit-Position: refs/heads/master@{#438141}

[gambard, 2016-12-12 16:09:23]

gambard@chromium.org changed reviewers:
+ jif@chromium.org

[gambard, 2016-12-12 16:09:24]

PTAL

[gambard, 2016-12-12 16:09:27]

The CQ bit was checked by gambard@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-12 16:09:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-12 16:45:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-12 16:45:51]

Dry run: This issue passed the CQ dry run.

[jif, 2016-12-12 16:58:46]

lgtm

[gambard, 2016-12-12 16:59:28]

The CQ bit was checked by gambard@chromium.org

[commit-bot: I haz the power, 2016-12-12 16:59:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-12 17:48:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-12 17:48:47]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[gambard, 2016-12-12 17:49:19]

The CQ bit was checked by gambard@chromium.org

[commit-bot: I haz the power, 2016-12-12 17:49:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-12 18:10:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-12 18:10:44]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[gambard, 2016-12-13 10:21:39]

gambard@chromium.org changed reviewers:
+ marq@chromium.org

[gambard, 2016-12-13 10:21:40]

marq@: PTAL as owner

[marq (ping after 24h), 2016-12-13 10:26:45]

https://codereview.chromium.org/2568163002/diff/1/ios/chrome/app/strings/ios_...
File ios/chrome/app/strings/ios_strings.grd (right):

https://codereview.chromium.org/2568163002/diff/1/ios/chrome/app/strings/ios_...
ios/chrome/app/strings/ios_strings.grd:1078: Articles you've read
I'm worried about consistency with the rest of the UI here; we have "Mark Read",
"Mark Unread", and then "Unread", so it seems that in context "Read" is fine. Is
the issue that this was mistranslated into a verb in some languages? 

This seems like the kind of thing that a UX writer would be able to help with.

[gambard, 2016-12-13 10:28:46]

https://codereview.chromium.org/2568163002/diff/1/ios/chrome/app/strings/ios_...
File ios/chrome/app/strings/ios_strings.grd (right):

https://codereview.chromium.org/2568163002/diff/1/ios/chrome/app/strings/ios_...
ios/chrome/app/strings/ios_strings.grd:1078: Articles you've read
On 2016/12/13 10:26:45, marq wrote:
> I'm worried about consistency with the rest of the UI here; we have "Mark
Read",
> "Mark Unread", and then "Unread", so it seems that in context "Read" is fine.
Is
> the issue that this was mistranslated into a verb in some languages? 
> 
> This seems like the kind of thing that a UX writer would be able to help with.

It was asked my mardini@ (see bug).

[marq (ping after 24h), 2016-12-13 11:50:15]

Per offline discussion, LGTM.

[gambard, 2016-12-13 11:50:38]

The CQ bit was checked by gambard@chromium.org

[gambard, 2016-12-13 11:50:44]

Thanks!

[commit-bot: I haz the power, 2016-12-13 11:50:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[gambard, 2016-12-13 11:52:36]

The CQ bit was unchecked by gambard@chromium.org

[gambard, 2016-12-13 11:55:58]

The CQ bit was checked by gambard@chromium.org

[gambard, 2016-12-13 11:55:58]

The patchset sent to the CQ was uploaded after l-g-t-m from jif@chromium.org,
marq@chromium.org
Link to the patchset: https://codereview.chromium.org/2568163002/#ps10002
(title: "Update text")

[commit-bot: I haz the power, 2016-12-13 11:56:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-13 12:23:27]

CQ is committing da patch.

Bot data: {"patchset_id": 10002, "attempt_start_ts": 1481630158296490,
"parent_rev": "40b7a8106edbf430f065dbd678bda50e54880194", "commit_rev":
"d3faa5eff7cfa13ec62c67ac2279bc74d1f3a119"}

[commit-bot: I haz the power, 2016-12-13 12:23:51]

Description was changed from

==========
Change "Read" header in Reading List

Change "Read" header to a phrase in reading to be clearer in all languages.

BUG=673335
==========

to

==========
Change "Read" header in Reading List

Change "Read" header to a phrase in reading to be clearer in all languages.

BUG=673335

Review-Url: https://codereview.chromium.org/2568163002
==========

[commit-bot: I haz the power, 2016-12-13 12:23:52]

Committed patchset #2 (id:10002)

[commit-bot: I haz the power, 2016-12-13 12:25:54]

Description was changed from

==========
Change "Read" header in Reading List

Change "Read" header to a phrase in reading to be clearer in all languages.

BUG=673335

Review-Url: https://codereview.chromium.org/2568163002
==========

to

==========
Change "Read" header in Reading List

Change "Read" header to a phrase in reading to be clearer in all languages.

BUG=673335

Committed: https://crrev.com/a13fc8e6a3b66d2f08b4ffde5a59277c49fe366a
Cr-Commit-Position: refs/heads/master@{#438141}
==========

[commit-bot: I haz the power, 2016-12-13 12:25:55]

Patchset 2 (id:??) landed as
https://crrev.com/a13fc8e6a3b66d2f08b4ffde5a59277c49fe366a
Cr-Commit-Position: refs/heads/master@{#438141}