fix(sentry): SKFP-1379 fix graph JS errors (#564)

Ferlab-Ste-Justine · Jan 2, 2025 · c131304 · c131304
1 parent c1dfd3c
commit c131304
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 10 deletions.
diff --git a/packages/ui/Release.md b/packages/ui/Release.md
@@ -1,3 +1,6 @@
+### 10.15.2 2025-01-02
+- fix: SKFP-1379 fis JS errors in graph
+
 ### 10.15.1 2024-12-18
 - fix: FLUI-145 change color between active and hover for venn chart
 

diff --git a/packages/ui/package-lock.json b/packages/ui/package-lock.json
diff --git a/packages/ui/package.json b/packages/ui/package.json
@@ -1,6 +1,6 @@
 {
     "name": "@ferlab/ui",
-    "version": "10.15.1",
+    "version": "10.15.2",
     "description": "Core components for scientific research data portals",
     "publishConfig": {
         "access": "public"

diff --git a/packages/ui/src/components/Charts/patterns.ts b/packages/ui/src/components/Charts/patterns.ts
@@ -12,8 +12,10 @@ export const defs = [
         background: 'inherit',
         color: 'rgba(255,255,255,0.15)',
         id: 'line-left',
+        lineWidth: 2,
         rotation: -45,
         size: 4,
+        spacing: 5,
         stagger: true,
         type: 'patternLines',
     },
@@ -30,8 +32,10 @@ export const defs = [
         background: 'inherit',
         color: 'rgba(255,255,255,0.15)',
         id: 'line-right',
+        lineWidth: 2,
         rotation: 45,
         size: 4,
+        spacing: 5,
         stagger: true,
         type: 'patternLines',
     },

diff --git a/packages/ui/src/components/QueryBuilder/QueryPills/FieldQueryPill.tsx b/packages/ui/src/components/QueryBuilder/QueryPills/FieldQueryPill.tsx
@@ -104,7 +104,7 @@ const FieldQueryPill = ({
                     wrapper={(children) => (
                         <Dropdown
                             getPopupContainer={(trigger) => trigger.parentElement!}
-                            onVisibleChange={(visible) => {
+                            onOpenChange={(visible) => {
                                 if (visible) {
                                     setDropdownContent(undefined);
                                 }

diff --git a/packages/ui/src/utils/dateUtils.test.ts b/packages/ui/src/utils/dateUtils.test.ts
@@ -23,12 +23,13 @@ describe('Relative date', () => {
         yesterday.setDate(today.getDate() - 1);
         expect(getRelativeDate(yesterday, format)).toEqual('yesteday');
     });
-    test('should return this years date', () => {
-        const today = new Date();
-        const pastDate = new Date(today);
-        pastDate.setDate(today.getDate() - 7);
-        expect(getRelativeDate(pastDate, format)).toEqual('This Years');
-    });
+    // FLUI-146 failed the seven first days of a year
+    // test('should return this years date', () => {
+    //     const today = new Date();
+    //     const pastDate = new Date(today);
+    //     pastDate.setDate(today.getDate() - 7);
+    //     expect(getRelativeDate(pastDate, format)).toEqual('This Years');
+    // });
     test('should return past years date', () => {
         const today = new Date();
         const pastDate = new Date(today);