fix(ui): notifications positioning regression (#1445)

pujitm · web-flow · commit f73e5e0058fc · 2025-06-27T13:33:25.000-04:00
Downgrades the `vue-sonner` dependency from `2.0.0` to `1.3.0` because the newer version relies on unocss under the hood, which can't pipe across the web component build/boundary without modifying our css plumbing. The older version relied on a `styles.css` stylesheet. That stylesheet also couldn't make it through our build pipeline, but since it was a single asset, I vendored it via `sonner.css`.  ## Summary by CodeRabbit * **Bug Fixes** * Improved logic to prevent duplicate insertion of the toaster element in the page layout. * **Chores** * Downgraded the "vue-sonner" dependency to version ^1.3.0.  --- - To see the specific tasks where the Asana app for GitHub is being used, see below: - https://app.asana.com/0/0/1210640791480730
diff --git a/api/src/unraid-api/unraid-file-modifier/modifications/default-page-layout.modification.ts b/api/src/unraid-api/unraid-file-modifier/modifications/default-page-layout.modification.ts
@@ -10,7 +10,7 @@ export default class DefaultPageLayoutModification extends FileModification {
     public readonly filePath: string = '/usr/local/emhttp/plugins/dynamix/include/DefaultPageLayout.php';
 
     private addToaster(source: string): string {
-        if (source.includes('unraid-toaster')) {
+        if (source.includes('uui-toaster')) {
             return source;
         }
         const insertion = `<uui-toaster rich-colors close-button position="<?= ($notify['position'] === 'center') ? 'top-center' : $notify['position'] ?>"></uui-toaster>`;
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/unraid-ui/package.json b/unraid-ui/package.json
@@ -61,7 +61,7 @@
     "marked": "^15.0.0",
     "reka-ui": "^2.1.1",
     "tailwind-merge": "^2.6.0",
-    "vue-sonner": "^2.0.0"
+    "vue-sonner": "^1.3.0"
   },
   "devDependencies": {
     "@ianvs/prettier-plugin-sort-imports": "^4.4.1",

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ export default class DefaultPageLayoutModification extends FileModification {`
`10`	`10`	`public readonly filePath: string = '/usr/local/emhttp/plugins/dynamix/include/DefaultPageLayout.php';`
`11`	`11`
`12`	`12`	`private addToaster(source: string): string {`
`13`		`- if (source.includes('unraid-toaster')) {`
	`13`	`+ if (source.includes('uui-toaster')) {`
`14`	`14`	`return source;`
`15`	`15`	`}`
`16`	`16`	const insertion = `<uui-toaster rich-colors close-button position="<?= ($notify['position'] === 'center') ? 'top-center' : $notify['position'] ?>"></uui-toaster>`;