Closed Bug 1218586 Opened 4 years ago Closed 4 years ago

use \u0020 instead of backspace protected whitespace in watchExpressionsSeparatorLabel to suppress warning from compare-locales

Categories

(DevTools :: Debugger, defect)

defect
Not set

Tracking

(firefox44 fixed)

RESOLVED FIXED
Firefox 44
Tracking Status
firefox44 --- fixed

People

(Reporter: aryx, Assigned: aryx)

Details

Attachments

(1 file)

In debugger.properties, there is
watchExpressionsSeparatorLabel=\ →
The backspace prevents the whitespace from being stripped. The compare-locales script which checks translations for errors got stricter ( https://groups.google.com/d/msg/mozilla.dev.l10n/tIjyUTV36a0/vZBjawscSjkJ ) and now warns for this string. The recommendation is to use the unicode representation of the whitespace.
Attached patch patch, v1Splinter Review
The string ID gets changed so the locales will pick this up.
Attachment #8679129 - Flags: review?(jryans)
Comment on attachment 8679129 [details] [diff] [review]
patch, v1

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

Makes sense, thanks!
Attachment #8679129 - Flags: review?(jryans) → review+
https://hg.mozilla.org/mozilla-central/rev/9a0ee09782a3
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.