- From: Daniel Murphy <notifications@github.com>
- Date: Thu, 01 Aug 2024 16:15:12 -0700
- To: w3c/manifest <manifest@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/manifest/pull/1101/review/2214186976@github.com>
@dmurph commented on this pull request. Still LGTM, one suggestion to change the example. > + <aside class="example"> + <p> + For example, the [=manifest/name=] member is a [=localizable + member=], and `name_localized` is its corresponding + [=manifest/*_localized=] member. + </p> + </aside> + <p> + A <dfn>language map</dfn> is an [=ordered map=] whose key is a + [=language tag=] and whose value is a [=localized value=]. The + <dfn>localized value</dfn> is content localized in the language given + by the key. + </p> + <aside class="example" title="Localizing the application name"> + <pre class="json"> + { In order to make it obvious why one might want to have a different 'lang' for an entry in here, perhaps we should use a company that so it would have opinions about name pronunciations in other languages. L'Occitane is a good example - using wikipedia you can see that sometimes is uses the french version, while other times it has a translated name. For cases like English & German, screen readers should read the name in a french pronunciation. ```suggestion { "lang": "fr", "dir": "ltr", "name": "L'Occitane", "name_localized": { "en": { value: "L'Occitane", "lang": "fr" }, "de": { value: "L'Occitane", "lang": "fr" }, "zh": "歐舒丹" "en-GB": {"value": "L'Occitane en Provence", "lang": "fr" }, "fr": "L'Occitane", "ar": {value: "لوکسیتان", "dir": "rtl"} } } ``` > @@ -2298,7 +2556,9 @@ <h3> </h3> <p> The <dfn>application's name</dfn> is derived from either the - [=manifest/name=] member or [=manifest/short_name=] member. + [=manifest/name=] member or [=manifest/short_name=] member. The user In Chromium we use them depending on UX needs. It's not implemented, but I would like to follow similar rules for extensions - `short_name` is truncated to 12 characters and `name` is truncated to 75. This sets expectations for devs, and allows the user agent to show an app name where there might not be much space. -- Reply to this email directly or view it on GitHub: https://github.com/w3c/manifest/pull/1101#pullrequestreview-2214186976 You are receiving this because you are subscribed to this thread. Message ID: <w3c/manifest/pull/1101/review/2214186976@github.com>
Received on Thursday, 1 August 2024 23:15:16 UTC