782820 - typos in gclicommands.properties

Status: RESOLVED FIXED

(DevTools :: Console, defect)

Description

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

Replace "it's" with "its":

cookieSetManual=Set a cookie by specifying a key name, it's value and optionally one or more of the following attributes: expires (max-age in seconds or the expires date in GMTString format), path, domain, secure

The following two descriptions should contain a "that", but I could be wrong (it sounds wrong to a non-native speaker like me):

# LOCALIZATION NOTE (addonAlreadyEnabled) Used in the output of the
# 'addon enable' command when an attempt is made to enable an addon is already
# enabled.

# LOCALIZATION NOTE (addonAlreadyDisabled) Used in the output of the
# 'addon disable' command when an attempt is made to disable an addon is already
# disabled.

Comment 1

[John Engler]

Hi,
  If this is not already assigned, I will happily take it as my first submit.

Comment 2

[Ekanan Ketunuti]

> If this is not already assigned, I will happily take it as my first submit.
Sure.

Comment 3

[John Engler]

Attachment: Changes made as suggested.

Comment 4

[John Engler]

Also, I noticed that there are other inconsistencies within the gclicommands.properties file. Should I fix them and re-submit a patch under this bug or create a new bug?

Comment 5

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

You can also fix these issues in this bug.

Comment 6

[John Engler]

Thanks. Are there official guidelines one how the gclicommands should be structured? For example, when to use a period at the end of a sentence, etc?

Also, when referencing the document object, should it be capitalized? inspectNodeManual uses "Document.querySelector" but the actual command would be "document.querySelector". 

(In reply to Archaeopteryx [:aryx] from comment #5)
> You can also fix these issues in this bug.

Comment 7

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

> Thanks. Are there official guidelines one how the gclicommands should be
> structured? For example, when to use a period at the end of a sentence, etc?
I leave the gclicommands to the Style Editor owner, period at the end usually mean "launches something which requires user input" (CCing l10n for linking the documentation)

> Also, when referencing the document object, should it be capitalized?
> inspectNodeManual uses "Document.querySelector" but the actual command would
> be "document.querySelector". 
Yes, this should be changed to prevent confusion for people not being familiar with |document|. You will need to update the entity name because the meaning changes and the localizations should pick this up.

Comment 8

[John Engler]

Attachment: Further spelling/grammar corrections

Comment 9

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

Pike: This is in devtools, should all entity names be changed because some locales don't translate devtools or is there any other possibility to port the changes to theses localizations?

Comment 10

[Axel Hecht [:Pike]]

Some of these are borderline, so I've poked at mxr a bit to see if they're a problem.

The beauty vs pretty is mostly solved in the translations, the Javascript vs JavaScript isn't. The Document shows up about as often as document does.

I think we shouldn't change the keys, and just post in .l10n about the fixes.

Comment 11

[John Engler]

Is there anything I can (or need to) do to help?

(In reply to Axel Hecht [:Pike] from comment #10)
> Some of these are borderline, so I've poked at mxr a bit to see if they're a
> problem.
> 
> The beauty vs pretty is mostly solved in the translations, the Javascript vs
> JavaScript isn't. The Document shows up about as often as document does.
> 
> I think we shouldn't change the keys, and just post in .l10n about the fixes.

Comment 12

[Joe Walker [:jwalker]]

Comment on attachment 654097 [details] [diff] [review]
Further spelling/grammar corrections

Review of attachment 654097 [details] [diff] [review]:
-----------------------------------------------------------------

Many thanks.

Comment 13

[Joe Walker [:jwalker]]

https://tbpl.mozilla.org/?tree=Fx-Team&rev=bbd267f16fc7

Comment 14

[Dave Camp (:dcamp)]

https://hg.mozilla.org/mozilla-central/rev/76b7f323b8f3

Comment 15

[Axel Hecht [:Pike]]

(In reply to John Engler from comment #11)
> Is there anything I can (or need to) do to help?
> 

Send a post to mozilla.dev.l10n? Thanks.

Comment 16

[John Engler]

(In reply to Axel Hecht [:Pike] from comment #15)
> (In reply to John Engler from comment #11)
> > Is there anything I can (or need to) do to help?
> > 
> 
> Send a post to mozilla.dev.l10n? Thanks.

Post sent. Thank you for everyone's help. :)