diff --git a/app/package.json b/app/package.json index 1355148..0c4e598 100644 --- a/app/package.json +++ b/app/package.json @@ -2,7 +2,7 @@ "name": "church-translate-engine", "productName": "Church Translate", "version": "3.23.2", - "buildNumber": 124, + "buildNumber": 125, "private": true, "description": "Live sermon translation — self-contained: captures audio, transcribes, translates, and serves the congregation page itself", "main": "main.js", @@ -106,7 +106,7 @@ ] }, "asar": false, - "buildVersion": "124", + "buildVersion": "125", "win": { "target": [ { diff --git a/app/server/public/listen.js b/app/server/public/listen.js index f99de3f..48c6007 100644 --- a/app/server/public/listen.js +++ b/app/server/public/listen.js @@ -163,10 +163,21 @@ function renderFeatured() { var wrap = $('featured'); wrap.textContent = ''; + /* + * THE OPERATOR'S LIST IS THE LIST. This used to unshift the source + * language in unconditionally, on the reasoning that plenty of people want + * the English captions for accessibility rather than translation — which + * is true, and still available, but it is not this code's call to make. + * The operator has a switch for every language including English, and for + * the source language alone that switch did nothing: English was put back + * on every render, so a desk that had deliberately turned it off still + * offered it as the first button on the page. A control that silently + * refuses is worse than no control. + * + * Wanting English captions is now one click in Settings, in the same list + * as every other language, and it stays where it is put. + */ var list = (app.cfg.featured || []).slice(); - /* The source language belongs in the list too — plenty of people want the - English captions for accessibility, not translation. */ - if (list.indexOf(app.cfg.sourceLanguage) === -1) list.unshift(app.cfg.sourceLanguage); list.forEach(function (code) { var b = langButton(code); if (b) wrap.appendChild(b);