feat(report): fixer les entêtes du tableau (#203)

* fix(report): alert title required * fix(report): alert type warning unknown * fix(report): remove unused code * chore(report): update jest snapshots * feat(report): fixer les entêtes du tableau Fixes #144 * fix: release * fix(report): declaration rgpd alert title missing * fix: fix some dsfr icons issues Co-authored-by: = <=> Co-authored-by: Julien Bouquillon <[email protected]>
SocialGouv · Jul 7, 2022 · 54db068 · 54db068
1 parent 61a6275
commit 54db068
Show file tree

Hide file tree

Showing 14 changed files with 163 additions and 130 deletions.
diff --git a/report/www/package.json b/report/www/package.json
@@ -14,8 +14,7 @@
     "test": "jest"
   },
   "dependencies": {
-    "@dataesr/react-dsfr": "^0.9.8",
-    "@gouvfr/dsfr": "^1.3.0",
+    "@dataesr/react-dsfr": "^2.5.0",
     "@socialgouv/matomo-next": "^1.2.2",
     "@testing-library/user-event": "^13.5.0",
     "country-flag-icons": "^1.4.20",

diff --git a/report/www/pages/_app.tsx b/report/www/pages/_app.tsx
@@ -5,7 +5,8 @@ import { Container } from "@dataesr/react-dsfr";
 import Head from "next/head";
 import { init } from "@socialgouv/matomo-next";
 
-import "@gouvfr/dsfr/dist/dsfr/dsfr.min.css";
+import "@gouvfr/dsfr/dist/utility/utility.main.min.css";
+import "@gouvfr/dsfr/dist/dsfr/dsfr.main.min.css";
 import "rc-tooltip/assets/bootstrap.css";
 
 import { HeaderSite } from "../src/components/HeaderSite";

diff --git a/report/www/src/__tests__/__snapshots__/Dashboard.test.js.snap b/report/www/src/__tests__/__snapshots__/Dashboard.test.js.snap
@@ -4,7 +4,7 @@ exports[`Should render empty Dashboard 1`] = `null`;
 
 exports[`Should render full Dashboard 1`] = `
 <div
-  className="fr-table tableWrapper"
+  className="fr-table fr-table--header-fixed tableWrapper"
 >
   <table
     className="table"
@@ -6952,7 +6952,7 @@ exports[`Should render full Dashboard 1`] = `
 
 exports[`Tools config Should render Dashboard with limited tools 1`] = `
 <div
-  className="fr-table tableWrapper"
+  className="fr-table fr-table--header-fixed tableWrapper"
 >
   <table
     className="table"

diff --git a/report/www/src/__tests__/__snapshots__/Intro.test.js.snap b/report/www/src/__tests__/__snapshots__/Intro.test.js.snap
@@ -93,7 +93,6 @@ exports[`Should render Intro 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--success"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -104,7 +103,6 @@ exports[`Should render Intro 1`] = `
       </div>
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -114,8 +112,7 @@ exports[`Should render Intro 1`] = `
         </p>
       </div>
       <div
-        class="fr-alert fr-alert--warning"
-        role="alert"
+        class="fr-alert fr-alert--error"
       >
         <p
           class="fr-alert__title"
@@ -174,7 +171,6 @@ exports[`Should render Intro 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -225,7 +221,6 @@ exports[`Should render Intro 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -306,7 +301,6 @@ exports[`Should render Intro 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -388,7 +382,6 @@ exports[`Should render Intro 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -537,7 +530,6 @@ exports[`Should render Intro 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -619,12 +611,13 @@ exports[`Should render Intro 1`] = `
       </li>
       <br />
       <div
-        class="fr-alert fr-alert--warning"
-        role="alert"
+        class="fr-alert fr-alert--error"
       >
         <p
           class="fr-alert__title"
-        />
+        >
+          thirdparties
+        </p>
         <p>
           Certains script légitimes peuvent apparaitre dans cette catégorie s'ils sont hébergés sur d'autres serveurs
         </p>
@@ -867,7 +860,6 @@ exports[`Tools config Should render Intro with limited tools 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -918,7 +910,6 @@ exports[`Tools config Should render Intro with limited tools 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -999,7 +990,6 @@ exports[`Tools config Should render Intro with limited tools 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -1081,7 +1071,6 @@ exports[`Tools config Should render Intro with limited tools 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -1163,7 +1152,6 @@ exports[`Tools config Should render Intro with limited tools 1`] = `
       <br />
       <div
         class="fr-alert fr-alert--info"
-        role="alert"
       >
         <p
           class="fr-alert__title"
@@ -1245,12 +1233,13 @@ exports[`Tools config Should render Intro with limited tools 1`] = `
       </li>
       <br />
       <div
-        class="fr-alert fr-alert--warning"
-        role="alert"
+        class="fr-alert fr-alert--error"
       >
         <p
           class="fr-alert__title"
-        />
+        >
+          thirdparties
+        </p>
         <p>
           Certains script légitimes peuvent apparaitre dans cette catégorie s'ils sont hébergés sur d'autres serveurs
         </p>