Fix a lot of casing and ellipsis bugs on options dialog buttons

Fix a lot of casing and ellipsis bugs on options dialog buttons

BUG=68809
TEST=manual

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=70820

[Evan Stade, 2011-01-07 01:47:24]

first pass. not complete.

[stuartmorgan, 2011-01-07 17:06:58]

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
File chrome/app/generated_resources.grd (right):

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:4673: <message
name="IDS_PASSWORDS_PAGE_VIEW_REMOVE_ALL_BUTTON" desc="Passwords page view's
'Remove all' button text">
Should these 3(6) descriptions have the "In <foo> case" annotation?

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:5392: <if expr="not
pp_ifdef('use_titlecase')">
Swap block order for consistency?

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:6968: <if expr="not
pp_ifdef('use_titlecase')">
Swap blocks

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:7059: Autofill Options...
Shouldn't this be "settings" on one of the platforms?

[Evan Stade, 2011-01-07 19:32:28]

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
File chrome/app/generated_resources.grd (right):

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:4673: <message
name="IDS_PASSWORDS_PAGE_VIEW_REMOVE_ALL_BUTTON" desc="Passwords page view's
'Remove all' button text">
On 2011/01/07 17:06:58, stuartmorgan wrote:
> Should these 3(6) descriptions have the "In <foo> case" annotation?

welllll, I'm not sure how useful it actually is to the translators. We have a
ton of strings where it's not explicitly mentioned. We should probably be
consistent but I'd lean towards that meaning we should remove all "In foo case:"
prefixes. What do you think?

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:5392: <if expr="not
pp_ifdef('use_titlecase')">
On 2011/01/07 17:06:58, stuartmorgan wrote:
> Swap block order for consistency?

Done.

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:6968: <if expr="not
pp_ifdef('use_titlecase')">
On 2011/01/07 17:06:58, stuartmorgan wrote:
> Swap blocks

Done.

http://codereview.chromium.org/6098006/diff/2001/chrome/app/generated_resourc...
chrome/app/generated_resources.grd:7059: Autofill Options...
On 2011/01/07 17:06:58, stuartmorgan wrote:
> Shouldn't this be "settings" on one of the platforms?

Done.

[stuartmorgan, 2011-01-07 19:39:05]

I don't know enough about how the translators work to have an informed opinion,
I just think we should be consistent, and you were already adding it in several
other places.

[Evan Stade, 2011-01-07 23:42:59]

ok done.

[stuartmorgan, 2011-01-08 00:08:40]

LGTM