Re: [hatari-devel] Minor patch for InfoPlist.strings (only relevant for macOS) |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] Minor patch for InfoPlist.strings (only relevant for macOS)
- From: Andreas Grabher <andreas_g86@xxxxxxxxxx>
- Date: Tue, 16 Apr 2024 21:29:52 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1713295808; bh=DSagr6JQRjTnKlzj2AyutsWQqlNSToiz+TDRv/kFcbU=; h=Content-Type:From:Mime-Version:Subject:Date:Message-Id:To; b=qc4RXFjhqQPv/GMdgnZvQ6NPs5P7KGPUK6XDhVTvDH0roauu/V63tdHZqnBwZBcRf pdRfDLVB9QPnsXl38mVu0iao1ruU9JSIwERpL+kA0U94YJwKaS+z1tuORfraWGQvS0 orW+zZHuMoF41dgbLm7yH0DsCTHIJEO1PL2A729vnBUE3Ng42Tik+OoI5u/cszrhy7 AVpmd+PHC7aceDfWLojiXHcl6MVsLy4AEyN00KPOyecno1mBgasKp3Nr1LT5YRvna9 osuHtXsm16wzjMfHXys7k/ZlvOLEi/wAtkWwfCapPjtIILhI1lRVoIEjS3GtdoGWZQ wxMxcEQqfU1IA==
> Am 16.04.2024 um 19:35 schrieb Andreas Grabher <andreas_g86@xxxxxxxxxx>:
>
>
>
>> Am 16.04.2024 um 18:53 schrieb Nicolas Pomarède <npomarede@xxxxxxxxxxxxx>:
>>
>> Le 14/04/2024 à 16:25, Andreas Grabher a écrit :
>>>> Am 01.04.2024 um 22:29 schrieb Andreas Grabher <andreas_g86@xxxxxxxxxx>:
>>>>
>>>>
>>>>> Am 01.04.2024 um 19:45 schrieb Andreas Grabher <andreas_g86@xxxxxxxxxx>:
>>>>>
>>>>> Hello all,
>>>>>
>>>>> may I suggest this minor GUI fix for macOS (patch appended)?
>>>>>
>>>>> Short description:
>>>>> On macOS the short version string is usually just the version itself. You can see this for example in the appended "textedit get info": "Version: 1..19". On current Hatari this reads: "Version: Hatari version 2.4.1" which is somehow strange. Also it does not make much sense to add the language to the version string as on macOS the application is identical for all localizations. Furthermore Apple for some reason always adds the Copyright sign after the word copyright. You can also see this on the appended screenshots.
>>>>>
>>>>> Regards,
>>>>> Andreas
>>>>>
>>>>> <hatari get info.png>
>>>>> <hatari about.png>
>>>>> <textedit about.png>
>>>>> <textedit get info.png>
>>>>> <InfoPlist.diff>
>>>>
>>>> I just noticed that CFBundleGetInfoString is obsolete since Mac OS X 10..6. As Hatari supports 10.9 (or probably 10.13) and later this could be removed.
>>> Another addition to my first message:
>>> I think we only need to translate NSHumanReadableCopyright and NSMicrophoneUsageDescription. So the InfoPlist.strings could look like this (example from Previous):
>>> English:
>>> /* Localized versions of Info.plist keys */
>>> NSHumanReadableCopyright = "Copyright © 2011-2024 by Andreas Grabher, Simon Schubiger, Gilles Fetis and others (see readme.previous.txt)";
>>> NSMicrophoneUsageDescription = "Previous needs access to your microphone for sound input.";
>>> French:
>>> /* Localized versions of Info.plist keys */
>>> NSHumanReadableCopyright = "Copyright © 2011-2024 par Andreas Grabher, Simon Schubiger, Gilles Fetis et autres (voir readme.previous.txt)";
>>> NSMicrophoneUsageDescription = "Previous a besoin d'accéder à votre microphone pour l'entrée sonore.";
>>> The french translation is by google translator. So maybe someone in here can check and improve it.
>>
>> Hi
>>
>> regarding these changes, were you able to check the result on real macOS to confirm it works ? If so I can push similar changes to Hatari
>>
>> Nicolas
>>
> The copyright message works fine and removing the bundle name and version number did not cause any adverse effects.
>
> But there seems to be a problem with translating the microphone usage description. I always see the English version. Also some GUI elements like the menu bar remain in English while the entries in the menus are mixed. There seems to be some problem left.
>
I saw your patch. I think both CFBundleShortVersionString and CFBundleGetInfoString can be removed.
CFBundleShortVersionString because it matches the entry in Info-Hatari.plist and therefore does not need to be localized. Another benefit from removal: It has only to be changed once in Info-Hatari.plist when updating the version number.
CFBundleGetInfoString because it is obsolete since Mac OS X 10.6 (and SDL2 requires 10.6 or later).