Re: [w3c/manifest] Add members for localization (PR #1101)

@dmurph commented on this pull request.



> +            <li>If |languageMap| is not an [=ordered map=], return.
+            </li>
+            <li>Let |languageTags:ordered set| be the [=map/keys=] of
+            |languageMap:ordered map|.
+            </li>
+            <li>Set |map|[|member|] to a new [=ordered map=].
+            </li>
+            <li>[=Set/For each=] |languageTag:string| of |languageTags|, run
+            [=process a localized text value=], passing
+            |languageMap|[|languageTag|] as the [=localized value=],
+            |languageTag|, |map|, |member|, and |manifest direction|.
+            </li>
+          </ol>
+          <p>
+            To <dfn>process a localized text value</dfn>, given [=string=] or
+            [=ordered map=] |localizedValue|, [=string=] |languageTag:string|,

Same idea as my other comment - from the perspective this algorithm, this argument is the 'default value' if one isn't specified. totally optional suggestion, you can keep it the same too.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/w3c/manifest/pull/1101#discussion_r1678095387
You are receiving this because you are subscribed to this thread.

Message ID: <w3c/manifest/pull/1101/review/2178167456@github.com>

Received on Monday, 15 July 2024 16:22:00 UTC