<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body>
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">
<p>Hi Marcel,</p>
<p><br>
</p>
<p>1. Please see <a href="https://lists.squeakfoundation.org/pipermail/squeak-dev/2022-May/220311.html" class="OWAAutoLink" id="LPlnk66129" previewremoved="true">https://lists.squeakfoundation.org/pipermail/squeak-dev/2022-May/220311.html</a>. I will keep this
version in the inbox until the encoding issues are solved, as a reminder that of this possible slip. :-)</p>
<p><br>
</p>
<p>2. Thanks for the clarification about translations. I find it generally hard to find the right bounds - every error will become harder to debug when the message is translated, UI automation will be harder, etc. :-)</p>
<p><br>
</p>
<p>Best,</p>
<p>Christoph</p>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>
<b>Gesendet:</b> Freitag, 8. April 2022 09:32:04<br>
<b>An:</b> squeak-dev<br>
<b>Betreff:</b> Re: [squeak-dev] The Inbox: SystemReporter-ct.62.mcz</font>
<div> </div>
</div>
<div>
<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
Hi Christoph --
<div><br>
</div>
<div>Would you extract this from the report method? I also find it strange that CompressedSources would not have a converter. After decompressing the content, you must convert them somehow.</div>
<div><br>
</div>
<div>> <span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px">And was it a deliberate decision not to add multilingual support to the system reporter or has it just been forgotten? :-)</span></div>
<div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"><br>
</span></div>
<div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px">You tell me. You should not #translate all the way down. Especially if it is about such fundamental information as the "Image Locale" ... how would you be able to debug? And especially
since there is extra whitespace to layout things in this monospaced text container, translation would be tricky anyway.</span></div>
<div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"><br>
</span></div>
<div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px">All information in the System Reporter should stay untranslated. Makes bug reports easier, where people copy stuff out of there.</span></div>
<div><span style="font-family: Calibri, Helvetica, sans-serif;font-size: 16px"><br>
</span></div>
<div><span style="font-family: Calibri, Helvetica, sans-serif"><span style="font-size: 16px">Best,</span></span></div>
<div><span style="font-family: Calibri, Helvetica, sans-serif"><span style="font-size: 16px">Marcel</span></span></div>
<div class="mb_sig"></div>
<blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
<p style="color: #AAAAAA; margin-top: 10px;">Am 07.04.2022 17:58:36 schrieb Thiede, Christoph <christoph.thiede@student.hpi.uni-potsdam.de>:</p>
<div style="font-family:Arial,Helvetica,sans-serif">
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size: 12pt;color: #000000;font-family: Calibri,Helvetica,sans-serif">
<p>Hi Marcel,</p>
<p><br>
</p>
<p>would this fix be okay for you?</p>
<p><br>
</p>
<p>And was it a deliberate decision not to add multilingual support to the system reporter or has it just been forgotten? :-)</p>
<p><br>
</p>
<p>Best,</p>
<p>Christoph</p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><span style="font-family: Calibri, sans-serif;color: #000000"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von commits@source.squeak.org <commits@source.squeak.org><br>
<b>Gesendet:</b> Donnerstag, 7. April 2022 17:57:06<br>
<b>An:</b> squeak-dev@lists.squeakfoundation.org<br>
<b>Betreff:</b> [squeak-dev] The Inbox: SystemReporter-ct.62.mcz</span>
<div> </div>
</div>
</div>
<span style="font-size: 10pt"><span style="font-size: 10pt">
<div class="PlainText">A new version of SystemReporter was added to project The Inbox:<br>
<a href="http://source.squeak.org/inbox/SystemReporter-ct.62.mcz">http://source.squeak.org/inbox/SystemReporter-ct.62.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: SystemReporter-ct.62<br>
Author: ct<br>
Time: 7 April 2022, 5:57:05.904555 pm<br>
UUID: 9807e6b3-6613-374c-ab14-5f055f8e750d<br>
Ancestors: SystemReporter-ct.61<br>
<br>
Fixes "image locale" report when the new preference for compressed sources is turned on.<br>
<br>
=============== Diff against SystemReporter-ct.61 ===============<br>
<br>
Item was changed:<br>
----- Method: SystemReporter>>reportImageLocale: (in category 'reporting') -----<br>
reportImageLocale: aStream<br>
<br>
| imageLocale platformLocale |<br>
imageLocale := Locale current.<br>
platformLocale := Locale currentPlatform.<br>
<br>
self header: 'Image Locale Information' on: aStream.<br>
aStream<br>
nextPutAll: ' Locale ID: ', imageLocale localeID; cr;<br>
nextPutAll: ' Translator: ', imageLocale languageTranslator; cr;<br>
nextPutAll: ' Environment: ', imageLocale languageEnvironment class; nextPutAll: ' (see languages below)'; cr;<br>
nextPutAll: 'Leading char: ', imageLocale leadingChar; nextPutAll: ' (ie. tag in characters)'; cr.<br>
<br>
aStream cr.<br>
self header: 'Image Default Language(s)' on: aStream.<br>
aStream<br>
nextPutAll: (imageLocale languageEnvironment class supportedLanguageNames sorted joinSeparatedBy: ', '); cr;<br>
cr; nextPutAll: '(These languages belong to the same category when Squeak uses the environment or leading char above to derive rules for language-specific text composition and display. Strings are not limited to this default. See LanguageEnvironment
and its subclasses.)'; cr.<br>
<br>
aStream cr.<br>
self header: 'Data Interpretation and Conversion' on: aStream.<br>
<br>
aStream<br>
+ nextPutAll: ' Source code: ',<br>
+ (SourceFiles<br>
+ "Some source files, e.g., CompressedSources, do not have a converter"<br>
+ detect: [:file | file respondsTo: #converter]<br>
+ ifFound: [:file | file converter class]<br>
+ ifNone: ['(none)' translated]);<br>
+ nextPutAll: ' (ie. .sources .changes .cs .st)'; cr;<br>
- nextPutAll: ' Source code: ', SourceFiles anyOne converter class; nextPutAll: ' (ie. .sources .changes .cs .st)'; cr;<br>
nextPutAll: ' Platform data: ', platformLocale systemConverter class; nextPutAll: ' (aka. #systemConverter)'; cr;<br>
nextPutAll: ' File names: ', platformLocale fileNameConverter class; nextPutAll: ' (ie. no content but paths)'; cr;<br>
nextPutAll: ' User input: ', platformLocale inputInterpreter class; cr;<br>
nextPutAll: 'Clipboard data: ', platformLocale clipboardInterpreter class; cr.<br>
<br>
aStream cr.<br>
self header: 'Platform Locale Information' on: aStream.<br>
<br>
aStream<br>
nextPutAll: ' Locale ID: ', platformLocale localeID; cr;<br>
nextPutAll: ' Currency symbol: ', platformLocale fetchCurrencySymbol;<br>
nextPutAll: (platformLocale primCurrencyNotation ifTrue: [' (prefix)'] ifFalse: [' (postfix)']); cr;<br>
nextPutAll: (' Numbers: 1{1}234{2}56'<br>
format: { platformLocale primDigitGrouping. platformLocale primDecimalSymbol}); cr;<br>
nextPutAll: ' Measurement: ', (platformLocale primMeasurement ifTrue: ['metric'] ifFalse: ['imperial']); cr;<br>
nextPutAll: ' Timezone: UTC', (platformLocale primTimezone >= 0 ifTrue: ['+']), (platformLocale primTimezone / 60);
<br>
nextPutAll: (platformLocale primDST ifTrue: [' (daylight saving time)'] ifFalse: ['']); cr;<br>
nextPutAll: ' Time format: ', platformLocale primTimeFormat; cr;<br>
nextPutAll: '(Short) Date format: ', platformLocale primShortDateFormat; cr;
<br>
nextPutAll: ' (Long) Date format: ', platformLocale primLongDateFormat; cr;
<br>
cr.!<br>
<br>
<br>
</div>
</span></span></div>
</blockquote>
</div>
</div>
</body>
</html>