676792 - Improve string for 32-bit plugin restart.

Status: RESOLVED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Philip Chee]

ref Firefox Bug 656152: Improve string for 32-bit plugin restart.

Comment 1

[Philip Chee]

Attachment: Patch v1.0

Someone was too clever by half when writing notification.xml, mutter mutter.
Again need a Mac OS X person to test this patch.

Comment 2

[Philip Chee]

Attachment: Patch v1.1 Simpler patch.

<NeilAway>	can't we just post a note to localisers suggesting that they might want to change the wording slightly?
<NeilAway>	do we have a mailing list for seamonkey localisers? I don't want to keep annoying just one of them all the time
<KaiRo>	mozilla.dev.l10n

Comment 3

[[e-mail]]

(In reply to Philip Chee from comment #2)
> <NeilAway>	can't we just post a note to localisers suggesting that they
> might want to change the wording slightly?
> <NeilAway>	do we have a mailing list for seamonkey localisers? I don't want
> to keep annoying just one of them all the time
> <KaiRo>	mozilla.dev.l10n

Shame you failed to read the note carefully enough ;-)

Comment 4

[Philip Chee]

Didn't see the note.

> On 05.08.11 13:53, Neil wrote:
>>> suite/locales/en-US/chrome/common/notification.properties
>>> carbonfailurepluginsMessage.title=This page requires a plugin that can
>>> only run in 32-bit mode.
>>>
>>> Someone wants to change the word "requires" to "requests". Unfortunately
>>> due to the way I wrote the code that reads the string, I can't easily
>>> rename the string, because that involves renaming other strings which
>>> haven't changed. Is the sky really going to fall in if I don't rename
>>> it? Will you guys mind renaming a bunch of strings that haven't really
>>> changed?
>>>
> I would think that both are totally fine, they just show a slightly 
> different political view on flash from the author ;-). I'd suggest to 
> keep the key as is and to add a comment suggesting that localizers can 
> use a word corresponding to one or both, depending on what matches the 
> language best.
> 
> Axel

Comment 5

[Philip Chee]

Attachment: Patch v1.2 with localization note.

> Didn't see the note.
Have now read the note and added a localization note.

Comment 6

[[e-mail]]

Comment on attachment 551124 [details] [diff] [review]
Patch v1.2 with localization note.

>-carbonfailurepluginsMessage.title=This page requires a plugin that can only run in 32-bit mode.
>+carbonfailurepluginsMessage.title=This page requests to use a plugin that can only run in 32-bit mode.
Oops. s/requires/requests/ only, not "requests to use"! r=me with that fixed.

Comment 7

[Stefan [:stefanh]]

Comment on attachment 551124 [details] [diff] [review]
Patch v1.2 with localization note.

What Neil said (good that it doesn't change the length of the string).

Comment 8

[Philip Chee]

> Oops. s/requires/requests/ only, not "requests to use"! r=me with that fixed.
Fixed.

Pushed.
http://hg.mozilla.org/comm-central/rev/c63fefad39c0