@uxland/primary-shell 7.39.0 → 7.40.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uxland/primary-shell",
-  "version": "7.39.0",
+  "version": "7.40.1",
   "description": "Primaria Shell",
   "author": "UXLand <dev@uxland.es>",
   "homepage": "https://github.com/uxland/harmonix/tree/app#readme",

package/src/api/import-data-manager/component/styles.css

@@ -49,10 +49,30 @@
   box-sizing: border-box;
 }
 
+/* Right column: stacks plugin cards and scrolls as a whole. */
 .content > primaria-region[name="import-data-region"] > div {
   overflow-y: auto;
 }
 
+/* Left column: vertical stack of multiple views with a gap. The last view
+   takes the remaining space and handles its own scroll; preceding views
+   keep their natural height. */
+.content > primaria-region[name="import-data-activity-history-region"] > div {
+  display: flex;
+  flex-direction: column;
+  gap: 16px;
+  overflow: hidden;
+}
+
+.content > primaria-region[name="import-data-activity-history-region"] > div > * {
+  flex-shrink: 0;
+}
+
+.content > primaria-region[name="import-data-activity-history-region"] > div > *:last-child {
+  flex: 1;
+  min-height: 0;
+}
+
 .footer {
   display: flex;
   flex-direction: row;

package/src/api/pdf-viewer-manager/pdf-viewer-manager.test.ts

@@ -2,7 +2,6 @@ import { describe, it, expect, vi, beforeEach } from "vitest";
 import { createPdfViewerManager } from "./pdf-viewer-manager";
 import { pdfViwerEvents } from "./events";
 import { registerPdfViewerNavItem } from "./handle-views";
-import { showNavItemTooltip } from "../../UI/components/navigation-tooltip";
 
 vi.mock("@primaria/plugins-core", () => ({
   generateId: () => "generated-id",
@@ -16,10 +15,6 @@ vi.mock("../../locales", () => ({
   translate: (key: string) => key,
 }));
 
-vi.mock("../../UI/components/navigation-tooltip", () => ({
-  showNavItemTooltip: vi.fn(),
-}));
-
 describe("PdfViewerManager", () => {
   let manager: ReturnType<typeof createPdfViewerManager>;
   let broker: any;
@@ -64,20 +59,10 @@ describe("PdfViewerManager", () => {
     });
   });
 
-  it("should show tooltip when adding a valid pdf", async () => {
-    vi.useFakeTimers();
-
+  it("should show success notification when adding a valid pdf", () => {
     manager.add({ id: "doc-1", name: "Doc1", date: "2024-01-15 - 10:00", url: "https://test.com/doc1.pdf" });
 
-    // Fast-forward the setTimeout(100ms)
-    vi.advanceTimersByTime(100);
-
-    expect(showNavItemTooltip).toHaveBeenCalledWith(
-      "primaria-shell::pdf-viewer",
-      "pdfManager.tooltipMessage"
-    );
-
-    vi.useRealTimers();
+    expect(notificationService.success).toHaveBeenCalledWith("pdfManager.tooltipMessage");
   });
 
   it("should warn if pdf with same name already exists", () => {
@@ -88,36 +73,21 @@ describe("PdfViewerManager", () => {
     expect(manager.getPdfs()).toHaveLength(1); // solo el primero
   });
 
-  it("should not show tooltip when pdf with same name already exists", () => {
-    vi.useFakeTimers();
-
-    // First add succeeds and shows tooltip
+  it("should not show success notification when pdf with same name already exists", () => {
     manager.add({ id: "doc-1", name: "Doc1", date: "2024-01-15 - 10:00", b64: "xxx" });
-    vi.advanceTimersByTime(100);
-
-    // Clear mocks after first add
     vi.clearAllMocks();
 
-    // Second add with same name should show warning but NOT tooltip
     manager.add({ id: "doc-1", name: "Doc1", date: "2024-01-15 - 10:00", b64: "xxx" });
-    vi.advanceTimersByTime(100);
 
     expect(notificationService.warning).toHaveBeenCalledWith("pdfManager.alreadyUploaded");
-    expect(showNavItemTooltip).not.toHaveBeenCalled();
-
-    vi.useRealTimers();
+    expect(notificationService.success).not.toHaveBeenCalled();
   });
 
-  it("should not show tooltip when pdf data is invalid", () => {
-    vi.useFakeTimers();
-
+  it("should not show success notification when pdf data is invalid", () => {
     manager.add({ id: "doc-1", name: "Doc1", date: "2024-01-15 - 10:00" });
-    vi.advanceTimersByTime(100);
 
     expect(notificationService.error).toHaveBeenCalledWith("pdfManager.missingData");
-    expect(showNavItemTooltip).not.toHaveBeenCalled();
-
-    vi.useRealTimers();
+    expect(notificationService.success).not.toHaveBeenCalled();
   });
 
   it("should not call registerNavButton when adding PDFs", () => {

package/src/api/pdf-viewer-manager/pdf-viewer-manager.ts

@@ -5,9 +5,7 @@ import { PrimariaNotificationService } from "../notification-service/notificatio
 import { pdfViwerEvents } from "./events";
 import { PdfSelector } from "./pdf-visor/pdf-selector/pdf-selector";
 import { PdfVisor } from "./pdf-visor/pdf-visor";
-import { showNavItemTooltip } from "../../UI/components/navigation-tooltip";
 import { pdfViewerId } from "./constants";
-import { primariaShellId } from "../../constants";
 
 export interface PdfData {
   id: string;
@@ -72,11 +70,7 @@ export class PdfViewerManager {
           });
         }, 150);
       } else {
-        // Show tooltip to guide user to the PDF viewer
-        setTimeout(() => {
-          const navItemKey = `${primariaShellId}::${pdfViewerId}`;
-          showNavItemTooltip(navItemKey, translate("pdfManager.tooltipMessage"));
-        }, 100);
+        this.notificationService.success(translate("pdfManager.tooltipMessage"));
       }
     }