1412230 - [Form Autofill] Replace the word "Company" with "Organization"

Status: RESOLVED FIXED

(Toolkit :: Form Manager, enhancement, P2)

Description

[Scott Wu [:scottwu]]

Although we have used the word "Company" for organization, after discussing with UX (:mark_liang), we felt it's better to just use "Organization", given that "Company" is a more restrictive term of "Organization".

Comment 1

[Scott Wu [:scottwu]]

If we were to make this change now, do you think a string ID change is still necessary?

Comment 2

[Scott Wu [:scottwu]]

Attachment: Bug 1412230 - Replace the word "Company" with "Organization".

Review commit: https://reviewboard.mozilla.org/r/193838/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/193838/

Comment 3

[Scott Wu [:scottwu]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/193838/diff/1-2/

Comment 4

[Francesco Lodolo [:flod]]

(In reply to Scott Wu [:scottwu] from comment #1)
> If we were to make this change now, do you think a string ID change is still
> necessary?

Definitely yes. I expect languages to have different translations for those two words, and better to make sure they pick the right one.

Comment 5

[Francesco Lodolo [:flod]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

https://reviewboard.mozilla.org/r/193838/#review198960

::: browser/extensions/formautofill/locales/en-US/formautofill.properties:54
(Diff revision 2)
>  autocompleteFooterOptionOSXShort = Preferences
>  # LOCALIZATION NOTE (category.address, category.name, category.organization, category.tel, category.email):
>  # Used in autofill drop down suggestion to indicate what other categories Form Autofill will attempt to fill.
>  category.address = address
>  category.name = name
> -category.organization = company
> +category.organization = organization

Please use new string IDs.

::: browser/extensions/formautofill/locales/en-US/formautofill.properties:103
(Diff revision 2)
>  addNewAddressTitle = Add New Address
>  editAddressTitle = Edit Address
>  givenName = First Name
>  additionalName = Middle Name
>  familyName = Last Name
> -organization = Company
> +organization = Organization

Same here, use a new string ID

Comment 6

[Scott Wu [:scottwu]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/193838/diff/2-3/

Comment 7

[Francesco Lodolo [:flod]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

https://reviewboard.mozilla.org/r/193838/#review199444

::: browser/extensions/formautofill/locales/en-US/formautofill.properties:50
(Diff revisions 2 - 3)
>  # LOCALIZATION NOTE (autocompleteFooterOptionShort, autocompleteFooterOptionOSXShort): Used as a label for the button,
>  # displayed at the bottom of the drop down suggestion, to open Form Autofill browser preferences. This version is used
>  # instead of autocompleteFooterOption* when the menu width is below 185px.
>  autocompleteFooterOptionShort = More Options
>  autocompleteFooterOptionOSXShort = Preferences
>  # LOCALIZATION NOTE (category.address, category.name, category.organization, category.tel, category.email):

Nit: update the reference to category.organization2

Comment 8

[Scott Wu [:scottwu]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

https://reviewboard.mozilla.org/r/193838/#review199444

> Nit: update the reference to category.organization2

Good catch! thanks.

Comment 9

[Scott Wu [:scottwu]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/193838/diff/3-4/

Comment 10

[Francesco Lodolo [:flod]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

https://reviewboard.mozilla.org/r/193838/#review199448

Strings-wise it looks good, thanks.

Comment 11

[Luke Chang [:lchang] (inactive)]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

https://reviewboard.mozilla.org/r/193838/#review199458

::: browser/extensions/formautofill/content/formautofill.xml:200
(Diff revision 4)
>              // Show the categories in certain order to conform with the spec.
> -            let orderedCategoryList = ["address", "name", "organization", "tel", "email"];
> +            let orderedCategoryList = [{id: "address", string: "category.address"},

We should mention that `string` is used for L10N id. Maybe either change it to `l10nId` or comment it on the top.

Comment 12

[Scott Wu [:scottwu]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/193838/diff/4-5/

Comment 13

[Scott Wu [:scottwu]]

Comment on attachment 8922690 [details]
Bug 1412230 - Replace the word "Company" with "Organization".

https://reviewboard.mozilla.org/r/193838/#review199458

> We should mention that `string` is used for L10N id. Maybe either change it to `l10nId` or comment it on the top.

I changed `string` to `l10nId`. Think that would make it clear what it does.

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e10c2f71fdcc
Replace the word "Company" with "Organization". r=flod,lchang

Comment 15

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

https://hg.mozilla.org/mozilla-central/rev/e10c2f71fdcc