Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(173)

Issue 145273019: Clean up sign-in grammar (Closed)

Created:
6 years, 11 months ago by michaelpg
Modified:
6 years, 10 months ago
CC:
chromium-reviews, kalyank, sadrul, ben+ash_chromium.org, jennschen
Visibility:
Public.

Description

Clean up sign-in grammar --------------------------------- "Signin" is not a word. "Sign into" doesn't make any more sense than "sign outof". As a phrasal verb, it stands as a unit. * "Bob signed in to Chromium to sync his settings." * "Bob signed into Chromium, and it hurt. Next time he'll be more careful while signing and walking in the chemistry lab." Don't hurt Bob. As a phrasal noun, "sign-in" should be hyphenated. Follow these simple[1] rules for correct usage: * "signin" is not a word * "sign in to", not "sign into" (you wouldn't "sign outof" Chrome) * as a noun or noun adjunct, "sign-in", not "sign in": "Sign-in happens at the sign-in screen." * as a participle + adverb: * "signed-in" before a noun: "who is the signed-in cat?" * "signed in" after a noun: "why is the cat signed in?" Additionally, do not capitalize "in" or "out" in title case. * "Sign in Again", not "Sign In Again" [1] May not be simple. --------------------------------- Secondly, we should say "sign in" instead of "log in" in most places. --------------------------------- Sources: * go/uxwordlist * https://net.educause.edu/elements/attachments/educause/pdf/ch07_tab01.pdf I'd like to have kuscher check my work. Including aruslan and derat for sign-off because the changes are somewhat extensive. BUG=315617 NOPRESUBMIT=true R=kuscher@chromium.org, bulach@chromium.org, derat@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248881

Patch Set 1 #

Total comments: 4

Patch Set 2 : Rebase and comments addressed #

Patch Set 3 : go away .gitmodules nobody likes you #

Patch Set 4 : Move product-specific strings #

Patch Set 5 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+127 lines, -115 lines) Patch
M ash/ash_strings.grd View 1 1 chunk +1 line, -1 line 0 comments Download
M chrome/android/java/strings/android_chrome_strings.grd View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/app/chromeos_strings.grdp View 1 6 chunks +9 lines, -9 lines 0 comments Download
M chrome/app/chromium_strings.grd View 1 2 3 5 chunks +24 lines, -10 lines 0 comments Download
M chrome/app/generated_resources.grd View 27 chunks +58 lines, -70 lines 0 comments Download
M chrome/app/google_chrome_strings.grd View 1 2 3 5 chunks +24 lines, -10 lines 0 comments Download
M chrome/browser/signin/signin_global_error.cc View 1 2 3 1 chunk +6 lines, -9 lines 0 comments Download
M chrome/browser/sync/sync_ui_util.cc View 1 2 3 1 chunk +2 lines, -3 lines 0 comments Download

Messages

Total messages: 28 (0 generated)
michaelpg
https://codereview.chromium.org/145273019/diff/1/chrome/app/generated_resources.grd File chrome/app/generated_resources.grd (right): https://codereview.chromium.org/145273019/diff/1/chrome/app/generated_resources.grd#newcode6421 chrome/app/generated_resources.grd:6421: + Enables SAML sign-in support for ChromeOS sign-in. Next ...
6 years, 11 months ago (2014-01-24 23:53:35 UTC) #1
Daniel Erat
+jennschen this change warms my heart. https://codereview.chromium.org/145273019/diff/1/chrome/app/chromeos_strings.grdp File chrome/app/chromeos_strings.grdp (right): https://codereview.chromium.org/145273019/diff/1/chrome/app/chromeos_strings.grdp#newcode3696 chrome/app/chromeos_strings.grdp:3696: Sign-in has been ...
6 years, 11 months ago (2014-01-25 00:09:22 UTC) #2
Kuscher
lgtm
6 years, 10 months ago (2014-01-27 21:33:03 UTC) #3
michaelpg
https://codereview.chromium.org/145273019/diff/1/chrome/app/chromeos_strings.grdp File chrome/app/chromeos_strings.grdp (right): https://codereview.chromium.org/145273019/diff/1/chrome/app/chromeos_strings.grdp#newcode3696 chrome/app/chromeos_strings.grdp:3696: Sign-in has been resticted to the owner account only. ...
6 years, 10 months ago (2014-01-27 21:38:21 UTC) #4
aruslan
Replacing myself with Newton as he knows more about English, strings, the branding :)
6 years, 10 months ago (2014-01-27 22:01:52 UTC) #5
newt (away)
android_chrome_strings.grd LGTM!! (I'm not technically an owner though, adding back aruslan@ so he can LGTM ...
6 years, 10 months ago (2014-01-27 22:22:26 UTC) #6
aruslan
chrome/android/java/strings/android_chrome_strings.grd LGTM
6 years, 10 months ago (2014-01-27 22:26:43 UTC) #7
michaelpg
The CQ bit was checked by michaelpg@chromium.org
6 years, 10 months ago (2014-02-03 19:02:14 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/michaelpg@chromium.org/145273019/130001
6 years, 10 months ago (2014-02-03 19:03:37 UTC) #9
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 10 months ago (2014-02-03 19:26:37 UTC) #10
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=47697
6 years, 10 months ago (2014-02-03 19:26:38 UTC) #11
commit-bot: I haz the power
CQ bit was unchecked on CL. Ignoring.
6 years, 10 months ago (2014-02-03 19:26:44 UTC) #12
commit-bot: I haz the power
CQ bit was unchecked on CL. Ignoring.
6 years, 10 months ago (2014-02-03 19:26:47 UTC) #13
michaelpg
The CQ bit was checked by michaelpg@chromium.org
6 years, 10 months ago (2014-02-03 19:40:12 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/michaelpg@chromium.org/145273019/130001
6 years, 10 months ago (2014-02-03 19:41:53 UTC) #15
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 10 months ago (2014-02-03 20:11:01 UTC) #16
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=47714
6 years, 10 months ago (2014-02-03 20:11:02 UTC) #17
commit-bot: I haz the power
CQ bit was unchecked on CL. Ignoring.
6 years, 10 months ago (2014-02-03 20:11:09 UTC) #18
commit-bot: I haz the power
CQ bit was unchecked on CL. Ignoring.
6 years, 10 months ago (2014-02-03 20:11:10 UTC) #19
michaelpg
Moved sign-in error messages to product-specific string files because of presubmit warning. atwilson: can you ...
6 years, 10 months ago (2014-02-03 21:34:39 UTC) #20
Andrew T Wilson (Slow)
.cc changes LGTM - great to finally clean up those uses of product_name...
6 years, 10 months ago (2014-02-04 10:07:13 UTC) #21
michaelpg
The CQ bit was checked by michaelpg@chromium.org
6 years, 10 months ago (2014-02-04 17:31:06 UTC) #22
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/michaelpg@chromium.org/145273019/390001
6 years, 10 months ago (2014-02-04 17:31:44 UTC) #23
michaelpg
The CQ bit was unchecked by michaelpg@chromium.org
6 years, 10 months ago (2014-02-04 17:50:16 UTC) #24
michaelpg
The CQ bit was checked by michaelpg@chromium.org
6 years, 10 months ago (2014-02-05 01:44:23 UTC) #25
michaelpg
The CQ bit was checked by michaelpg@chromium.org
6 years, 10 months ago (2014-02-05 01:47:55 UTC) #26
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/michaelpg@chromium.org/145273019/720001
6 years, 10 months ago (2014-02-05 03:47:10 UTC) #27
commit-bot: I haz the power
6 years, 10 months ago (2014-02-05 06:41:09 UTC) #28
Message was sent while issue was closed.
Change committed as 248881

Powered by Google App Engine
This is Rietveld 408576698