small improvements to translation feature

Author: PalmerAL

ext/franc-min/franc-min-6.2.0.bundle.mjs

@@ -2,7 +2,6 @@ const webviews = require('webviews.js')
 const statistics = require('js/statistics.js')
 
 const pageTranslations = {
-  apiURL: 'https://translate-api.minbrowser.org/translate',
   languages: [
     {
       code: 'en'
@@ -78,7 +77,7 @@ const pageTranslations = {
     const userPrefs = navigator.languages.map(lang => lang.split('-')[0])
     const topLangs = allLangs.filter(lang => userPrefs.includes(lang.code))
 
-    // Translations to/from English are the highest quality in Libretranslate, so always show that near the top
+    // English is the base/pivot language in Bergamot, so always show that near the top
     if (!topLangs.some(lang => lang.code === 'en')) {
       topLangs.push(allLangs.find(lang => lang.code === 'en'))
     }

ext/franc/franc-6.2.0.bundle.mjs

@@ -1,4 +1,4 @@
-const maxCharsToTranslate = 15000
+const maxCharsToTranslate = 50000
 
 function isVisible (el) {
   // https://github.com/jquery/jquery/blob/305f193aa57014dc7d8fa0739a3fefd47166cd44/src/css/hiddenVisibleSelectors.js

js/pageTranslations.js

@@ -1,6 +1,6 @@
 import { LatencyOptimisedTranslator, TranslatorBacking } from '../../node_modules/@browsermt/bergamot-translator/translator.js'
 
-import { franc } from '../../ext/franc-min/franc-min-6.2.0.bundle.mjs'
+import { franc } from '../../ext/franc/franc-6.2.0.bundle.mjs'
 import iso6393To1Mapping from './iso3To1Mapping.mjs'
 
 const registryUrl = 'https://services.minbrowser.org/bergamot-models-v1/registry.json'
@@ -17,6 +17,7 @@ class CustomBacking extends TranslatorBacking {
         if (files[file].modelType !== 'prod') {
           return null
         }
+        // The translation module expects absolute URLs but the registry entries are relative by default
         files[file].name = `${modelBaseUrl}/${key}/${files[file].name}`
       }
       return {
@@ -34,7 +35,7 @@ window.addEventListener('message', async function (e) {
   if (e.source === window && e.data === 'page-translation-session-create') {
     const options = {
       registryUrl: 'https://services.minbrowser.org/bergamot-models-v1/registry.json',
-      cacheSize: 0, // TODO implications?
+      cacheSize: 0, // This is a cache of translations, not models
       downloadTimeout: null // Disable timeout
     }