Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#4575 closed defect (fixed)

refactor editor clipboard error messages

Reported by: Adam Peller Owned by: Adam Peller
Priority: high Milestone: 1.0.1
Component: Editor Version: 0.9
Keywords: i18n Cc: alex
Blocked By: Blocking:

Description

See #4136

The error strings are mostly redundant. This should really be a single string with a substitution pattern, for maximum efficiency in translation and for downloading bandwidth.

Also, I'm not sure the string "due to its design decision" adds much for the user.

Change History (13)

comment:1 Changed 12 years ago by Adam Peller

Cc: alex added; i18n removed
Keywords: i18n added

original:

	'copyErrorFF': 'Copy action is only available in Firefox via keyboard shortcut due to its design decision: please use ctrl+c.'

In addition to the wording, there's the problem that keyboard shortcuts are different on different platforms. Do we try to accomodate this with a switch and logic in the code, or just leave the user to figure this out (simpler)

To change this, we'll have to wait a translation cycle before changing the code that uses it, but let's settle on the new root resource ASAP.

Suggestions:

  1. 'systemShortcut': 'The "${0}" action is only available using the system keyboard shortcut on this browser. Please use ${1}'
  2. 'systemShortcut': 'The "${0}" action is only available using the system keyboard shortcut on this browser.'

I vote for option 2. Glad to hear this problem goes away in FF3 :-)

comment:2 Changed 12 years ago by Adam Peller

'The "${0}" action is only available in Mozilla Firefox using a keyboard shortcut. Use ${1}'

comment:3 Changed 12 years ago by liucougar

Resolution: fixed
Status: newclosed

(In [10811]) fixes #4575, but needs to update all translations accordingly, any one?

comment:4 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: closedreopened

we'll have to insert the string, but not enable it until we make a pass through all the translations, I guess

comment:5 Changed 12 years ago by Adam Peller

Owner: changed from liucougar to Adam Peller
Status: reopenednew

comment:6 Changed 12 years ago by Adam Peller

(In [10816]) Comment out new alert string until translations are ready. Refs #4575

comment:7 Changed 12 years ago by Adam Peller

Milestone: 1.01.1

comment:8 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [11238]) Improved FF clipboard error messages. Fixes #4575

comment:9 Changed 12 years ago by Adam Peller

Milestone: 1.11.0

comment:10 Changed 12 years ago by Adam Peller

(In [11575]) Forgot to look up string in translation table for clipboard message. Fixes #4575

comment:11 Changed 12 years ago by Adam Peller

Milestone: 1.01.0.2
Resolution: fixed
Status: closedreopened

comment:12 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: reopenedclosed

(In [11576]) Forgot to look up string in translation table for clipboard message. Fixes #4575

comment:13 Changed 12 years ago by Adam Peller

Milestone: 1.0.21.0.1
Note: See TracTickets for help on using tickets.