fix bugs in previous release (#1827)

* add korean and indonesian to the local picker

* add ui language picker to project sidebar

* use a fallback for crypto random to make it easier to test on mobile

* make it easy to run a local prod ui build of fw lite

* make sure display: block is not overridden, don't try to center text horizontally in prose mirror

* after login trigger a sync to update the sync dialog login state

* show synced status when hovering over sync menu

* upgrade bits ui to fix an issue with a tooltip not closing when it was open before a dialog opened. This also required upgrading to svelte 5.33

* fix some failing tests when running on windows against a linux client

* don't try to use versions of svelte greater than 5.33 due to storybook issues

Author: hahn-kev

backend/FwLite/FwLiteWeb/FwLiteWebKernel.cs

@@ -23,10 +23,6 @@ public static IServiceCollection AddFwLiteWebServices(this IServiceCollection se
         services.AddFwLiteProjectSync();
         services.AddMiniLcmRouteServices();
         services.AddFwLiteShared(environment);
-        if (environment.IsDevelopment())
-        {
-            services.Configure<FwLiteConfig>(config => config.UseDevAssets = true);
-        }
 
         services.AddOptions<FwLiteWebConfig>().BindConfiguration("FwLiteWeb");
 

backend/FwLite/FwLiteWeb/appsettings.Development.json

@@ -14,6 +14,7 @@
     "LogFileName": null
   },
   "FwLite": {
-    "UpdateCheckCondition": "Never"
+    "UpdateCheckCondition": "Never",
+    "UseDevAssets": true
   }
 }

backend/FwLite/Taskfile.yml

@@ -14,10 +14,9 @@ tasks:
     dir: ./FwLiteWeb
     cmd: dotnet watch --no-hot-reload
   web-prod:
+    deps: [ ui:build-viewer-app ]
     dir: ./FwLiteWeb
-    cmds:
-      - dotnet publish --artifacts-path ../artifacts
-      - cd ../artifacts/publish/FwLiteWeb/release && dotnet FwLiteWeb.dll
+    cmd: dotnet run -- --FwLite:UseDevAssets=false
   web-chaos:
     label: Run FwLiteWeb with some Chaos injected to http requests to lexbox, requests will have chaos 30% of the time, this includes some latency of 5 seconds
     dir: ./FwLiteWeb

backend/Testing/FwHeadless/MediaFileTestFixture.cs

@@ -57,7 +57,11 @@ public async Task DisposeAsync()
         }
         var request = new HttpRequestMessage(HttpMethod.Get, url);
         var result = await HttpClient.SendAsync(request);
-        return (await result.Content.ReadFromJsonAsync<FileListing>(), result);
+        if (!result.IsSuccessStatusCode) return (null, result);
+        var fileListing = await result.Content.ReadFromJsonAsync<FileListing>();
+        //ensure the file listing is consistent with the os the tests are running on, keeps the tests simpler
+        if (fileListing is not null) fileListing = new ([..fileListing.Files.Select(f => f.Replace('/', Path.DirectorySeparatorChar).Replace('\\', Path.DirectorySeparatorChar)) ?? []]);
+        return (fileListing, result);
     }
 
     public async Task<(Guid, HttpResponseMessage)> PostFile(string localPath, string? overrideFilename = null, string? overrideSubfolder = null, string? contentType = null, bool deleteContentLengthHeader = false, FileMetadata? metadata = null, string loginAs = "admin", IDictionary<string, string>? extraFields = null)

frontend/pnpm-lock.yaml

@@ -4,7 +4,7 @@ packages:
 
 catalog:
   vite: ^6.0.3
-  svelte: ^5.28.0
+  svelte: ~5.33.19
   svelte-check: ^4.1.4
   svelte-preprocess: ^6.0.3
   "@sveltejs/vite-plugin-svelte": ^5.0.3

frontend/pnpm-workspace.yaml

@@ -65,7 +65,7 @@
     "@vitest/browser": "catalog:",
     "@vitest/ui": "catalog:",
     "autoprefixer": "^10.4.20",
-    "bits-ui": "1.8.0",
+    "bits-ui": "2.8.11",
     "clsx": "^2.1.1",
     "eslint": "catalog:",
     "eslint-plugin-storybook": "^9.0.8",

frontend/viewer/package.json

@@ -278,8 +278,8 @@
 
 <style lang="postcss" global>
   .ProseMirror {
-    display: block;
-    place-content: center;
+    display: block !important; /*prevent display from being overridden by .flex in the inputVariant class */
+    align-content: center;
     flex-grow: 1;
     outline: none;
     cursor: text;

frontend/viewer/src/lib/components/lcm-rich-text-editor/lcm-rich-text-editor.svelte

@@ -9,7 +9,9 @@
   const languages: Record<string, string> = {
     'en': 'English',
     'fr': 'Français',
-    'es': 'Español'
+    'es': 'Español',
+    'id': 'Bahasa Indonesia',
+    'ko': '한국어'
   };
   const currentLanguage = $derived(languages[$locale] ?? 'Unknown: ' + $locale);
 </script>

frontend/viewer/src/lib/i18n/LocalizationPicker.svelte

@@ -8,7 +8,14 @@ export function cn(...inputs: ClassValue[]) {
 }
 
 export function randomId(): string {
-  return crypto.randomUUID();
+  if (typeof crypto !== 'undefined' && 'randomUUID' in crypto) {
+    return crypto.randomUUID();
+  }
+  return 'xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx'.replace(/[xy]/g, (c) => {
+    const r = Math.random() * 16 | 0;
+    const v = c === 'x' ? r : (r & 0x3 | 0x8);
+    return v.toString(16);
+  });
 }
 
 export function firstTruthy<T, U>(items: T[], mapper: (item: T) => U): U | undefined {
@@ -21,7 +28,7 @@ export function firstTruthy<T, U>(items: T[], mapper: (item: T) => U): U | undef
 
 export function defaultEntry(): IEntry {
   return {
-    id: crypto.randomUUID(),
+    id: randomId(),
     citationForm: {},
     lexemeForm: {},
     note: {},