From d83b1b69ca6e9078e16a4b0e40a93b41b0d70746 Mon Sep 17 00:00:00 2001
From: Lorenzo Corallo <66379281+lorenzocorallo@users.noreply.github.com>
Date: Thu, 22 Aug 2024 17:03:56 +0200
Subject: [PATCH 1/6] feat: add version in about page (#256)

---
 src/routes/about/index.tsx | 21 ++++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/src/routes/about/index.tsx b/src/routes/about/index.tsx
index a1a4b7c6..dc2910ec 100644
--- a/src/routes/about/index.tsx
+++ b/src/routes/about/index.tsx
@@ -62,16 +62,19 @@ export const aboutRoute = new Route({
             </a>
           </div>
 
-          <div className="text-right">
-            Sviluppato con amore da{" "}
-            <a
-              target="_blank"
-              rel="noreferrer noopener"
-              href="https://polinetwork.org"
-            >
-              PoliNetwork
-            </a>{" "}
+          <div className="flex items-center justify-between">
+            <p className="opacity-60">Versione: {APP_VERSION}</p>
+            <p>
+              Sviluppato con amore da{" "}
+              <a
+                target="_blank"
+                rel="noreferrer noopener"
+                href="https://polinetwork.org"
+              >
+                PoliNetwork
+              </a>{" "}
             ❤️
+          </p>
           </div>
         </div>
       </Page>

From cdf9bb15e94179ebca77f56c10cdfb62e4aee122 Mon Sep 17 00:00:00 2001
From: Lorenzo Corallo <66379281+lorenzocorallo@users.noreply.github.com>
Date: Thu, 22 Aug 2024 17:05:13 +0200
Subject: [PATCH 2/6] feat: close filter popover when clearing it (#257)

Co-authored-by: angeousta <132761637+angeousta@users.noreply.github.com>
---
 src/routes/viewer/Table/FilterBtn.tsx | 38 ++++++++++++++++-----------
 1 file changed, 22 insertions(+), 16 deletions(-)

diff --git a/src/routes/viewer/Table/FilterBtn.tsx b/src/routes/viewer/Table/FilterBtn.tsx
index 3466db22..f44dc8a4 100644
--- a/src/routes/viewer/Table/FilterBtn.tsx
+++ b/src/routes/viewer/Table/FilterBtn.tsx
@@ -19,6 +19,7 @@ import {
   PopoverTrigger,
 } from "@/components/ui/popover";
 import { Separator } from "@/components/ui/separator";
+import { useState } from "react";
 
 export type FilterOption<T> = {
   originalValue: T;
@@ -38,11 +39,29 @@ export function FilterBtn<TData, TValue>({
   title,
   options,
 }: Props<TData, TValue>) {
+  const [open, setOpen] = useState<boolean>(false);
   const facets = column.getFacetedUniqueValues();
   const selectedValues = new Set(column?.getFilterValue() as string[]);
 
+  function handleClearFilter(): void {
+    column?.setFilterValue(undefined);
+    setOpen(false);
+  }
+
+  function handleOptionSelect(
+    option: FilterOption<TValue>,
+    isSelected: boolean,
+  ): void {
+    if (options.length <= 2) selectedValues.clear(); // filter with radio behaviour
+    if (isSelected) selectedValues.delete(option.value); // toggle behaviour
+    else selectedValues.add(option.value);
+
+    const filterValues = Array.from(selectedValues); // get selected filter options
+    column?.setFilterValue(filterValues.length ? filterValues : undefined); // update table
+  }
+
   return (
-    <Popover>
+    <Popover open={open} onOpenChange={setOpen}>
       <PopoverTrigger asChild>
         <Button
           variant="outline"
@@ -100,20 +119,7 @@ export function FilterBtn<TData, TValue>({
                 return (
                   <CommandItem
                     key={option.value}
-                    onSelect={() => {
-                      if (isSelected) {
-                        selectedValues.delete(option.value);
-                      } else {
-                        if (options.length <= 2) {
-                          selectedValues.clear();
-                        }
-                        selectedValues.add(option.value);
-                      }
-                      const filterValues = Array.from(selectedValues);
-                      column?.setFilterValue(
-                        filterValues.length ? filterValues : undefined,
-                      );
-                    }}
+                    onSelect={() => handleOptionSelect(option, isSelected)}
                   >
                     <div
                       className={cn(
@@ -144,7 +150,7 @@ export function FilterBtn<TData, TValue>({
                 <CommandSeparator />
                 <CommandGroup>
                   <CommandItem
-                    onSelect={() => column?.setFilterValue(undefined)}
+                    onSelect={handleClearFilter}
                     className="justify-center text-center"
                   >
                     Pulisci i filtri

From 95451550ec4dbdede4be348dfd3d46a79f71d1f3 Mon Sep 17 00:00:00 2001
From: Lorenzo Corallo <66379281+lorenzocorallo@users.noreply.github.com>
Date: Thu, 22 Aug 2024 17:06:38 +0200
Subject: [PATCH 3/6] fix: table crash when switching to a phase without the
 selected course (#259)

Co-authored-by: angeousta <132761637+angeousta@users.noreply.github.com>
---
 src/routes/viewer/Table/index.tsx | 30 ++++++++++++++++++++++--------
 src/routes/viewer/index.tsx       |  1 -
 2 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/src/routes/viewer/Table/index.tsx b/src/routes/viewer/Table/index.tsx
index 92ea9325..20c412bc 100644
--- a/src/routes/viewer/Table/index.tsx
+++ b/src/routes/viewer/Table/index.tsx
@@ -38,6 +38,21 @@ interface TableProps extends React.HTMLAttributes<HTMLTableElement> {
 }
 
 function makeHas(rows: StudentResult[]): Record<StudentResultKeys, boolean> {
+  if (rows.length === 0)
+    return {
+      englishCorrectAnswers: true,
+      sectionsResults: true,
+      positionAbsolute: true,
+      positionCourse: true,
+      birthDate: true,
+      result: true,
+      ofa: true,
+      enrollStatus: true,
+      enrollAllowed: true,
+      enrollCourse: true,
+      id: true,
+    };
+
   function checkKey(key: StudentResultKeys): boolean {
     const isThereAny = rows.some((r) => r[key]);
     if (!isThereAny) return false;
@@ -74,11 +89,10 @@ type ColumnVisibility = {
 };
 
 export default function Table({ table: _table, csvFilename }: TableProps) {
-  const { rows } = _table;
-  const has = makeHas(rows);
+  const has = makeHas(_table.rows);
   const [columnVisibility, setColumnVisibility] =
     useState<ColumnVisibility>(has);
-  const columns = getColumns(rows);
+  const columns = getColumns(_table.rows);
   const [pagination, setPagination] = useState<PaginationState>({
     pageSize: 15,
     pageIndex: 0,
@@ -86,7 +100,7 @@ export default function Table({ table: _table, csvFilename }: TableProps) {
 
   const [columnFilters, setColumnFilters] = useState<ColumnFiltersState>([]);
   const table = useReactTable({
-    data: rows,
+    data: _table.rows,
     columns,
     state: {
       columnVisibility,
@@ -169,12 +183,12 @@ export default function Table({ table: _table, csvFilename }: TableProps) {
               ) : (
                 <TableRow>
                   <TableCell
-                    colSpan={table.getHeaderGroups()[1].headers.length}
+                    colSpan={table.getHeaderGroups()[1]?.headers.length ?? 1}
                     className="h-24 text-center"
                   >
-                    {rows.length > 0
-                      ? "Nessuna riga trovata"
-                      : "Nessun dato disponibile"}
+                    {_table.rows.length > 0
+                      ? "La ricerca ha restituito nessun risultato"
+                      : "La tabella di questo corso non contiene righe"}
                   </TableCell>
                 </TableRow>
               )}
diff --git a/src/routes/viewer/index.tsx b/src/routes/viewer/index.tsx
index 4a8ed2ba..64fbba79 100644
--- a/src/routes/viewer/index.tsx
+++ b/src/routes/viewer/index.tsx
@@ -160,7 +160,6 @@ export const viewerRoute = new Route({
       [selectedCourse, selectedLocation, store],
     );
 
-    console.log({ ranking, selectedPhaseLink });
     return (
       <Page
         className={`flex gap-4 px-0 ${

From ecf7e50778d4146e4232473c471f09bb7f3c8281 Mon Sep 17 00:00:00 2001
From: Lorenzo Corallo <66379281+lorenzocorallo@users.noreply.github.com>
Date: Thu, 22 Aug 2024 17:07:19 +0200
Subject: [PATCH 4/6] feat: add school emojis in homepage (#261)

* chore(lint): enforce noImplicitReturns and exhaustive switch blocks

* feat: add school emojis in homepage

---------

Co-authored-by: angeousta <132761637+angeousta@users.noreply.github.com>
---
 .eslintrc.cjs                        |  1 +
 src/components/custom-ui/Alert.tsx   | 32 +++++++++++++++++-----------
 src/routes/homepage/chooseSchool.tsx | 15 +++++++++++++
 tsconfig.json                        |  2 ++
 4 files changed, 38 insertions(+), 12 deletions(-)

diff --git a/.eslintrc.cjs b/.eslintrc.cjs
index 869e9ccd..43da3a12 100644
--- a/.eslintrc.cjs
+++ b/.eslintrc.cjs
@@ -10,6 +10,7 @@ module.exports = {
   parserOptions: { ecmaVersion: "latest", sourceType: "module" },
   plugins: ["react-refresh", "prettier"],
   rules: {
+    "@typescript-eslint/switch-exhaustiveness-check": "error",
     "react-refresh/only-export-components": "warn",
     "prettier/prettier": "warn",
   },
diff --git a/src/components/custom-ui/Alert.tsx b/src/components/custom-ui/Alert.tsx
index 347fe1a3..37a8394f 100644
--- a/src/components/custom-ui/Alert.tsx
+++ b/src/components/custom-ui/Alert.tsx
@@ -38,19 +38,27 @@ export default function Alert({
 }
 
 function GetIcon(level: Level) {
-  if (level === "error") return <LuAlertCircle />;
-  if (level === "warning") return <LuAlertTriangle />;
-  if (level === "info") return <LuInfo />;
-  if (level === "success") return <LuCheckCircle2 />;
+  switch (level) {
+    case "error":
+      return <LuAlertCircle />;
+    case "warning":
+      return <LuAlertTriangle />;
+    case "info":
+      return <LuInfo />;
+    case "success":
+      return <LuCheckCircle2 />;
+  }
 }
 
 const getColors = (level: Level) => {
-  if (level === "error")
-    return "bg-red-300/5 border-red-600 text-red-600 dark:bg-red-100/5 dark:border-red-300 dark:text-red-300";
-  if (level === "warning")
-    return "bg-amber-300/5 border-amber-600 text-amber-600 dark:bg-amber-100/5 dark:border-amber-200 dark:text-amber-200";
-  if (level === "info")
-    return "bg-sky-300/5 border-sky-600 text-sky-600 dark:bg-sky-100/5 dark:border-sky-200 dark:text-sky-200";
-  if (level === "success")
-    return "bg-green-300/5 border-green-600 text-green-600 dark:bg-green-100/5 dark:border-green-200 dark:text-green-200";
+  switch (level) {
+    case "error":
+      return "bg-red-300/5 border-red-600 text-red-600 dark:bg-red-100/5 dark:border-red-300 dark:text-red-300";
+    case "warning":
+      return "bg-amber-300/5 border-amber-600 text-amber-600 dark:bg-amber-100/5 dark:border-amber-200 dark:text-amber-200";
+    case "info":
+      return "bg-sky-300/5 border-sky-600 text-sky-600 dark:bg-sky-100/5 dark:border-sky-200 dark:text-sky-200";
+    case "success":
+      return "bg-green-300/5 border-green-600 text-green-600 dark:bg-green-100/5 dark:border-green-200 dark:text-green-200";
+  }
 };
diff --git a/src/routes/homepage/chooseSchool.tsx b/src/routes/homepage/chooseSchool.tsx
index 06e821a0..d82afda9 100644
--- a/src/routes/homepage/chooseSchool.tsx
+++ b/src/routes/homepage/chooseSchool.tsx
@@ -3,6 +3,20 @@ import { Button } from "@/components/ui/button";
 import { homepageRoute } from ".";
 import { ButtonGrid } from "@/components/Homepage/ButtonGrid";
 import DevSettings from "@/components/DevSettings";
+import School from "@/utils/types/data/School";
+
+function getSchoolEmoji(school: School) {
+  switch (school) {
+    case "Architettura":
+      return (<span className="mr-2 text-lg rotate-[270deg]">&#128208;</span>);
+    case "Design":
+      return (<span className="mr-2 text-lg">&#128396;&#65039;</span>);
+    case "Ingegneria":
+      return (<span className="mr-2 text-lg">&#128736;&#65039;</span>);
+    case "Urbanistica":
+      return (<span className="mr-2 text-lg">&#127969;</span>);
+  }
+}
 
 export const chooseSchoolRoute = new Route({
   getParentRoute: () => homepageRoute,
@@ -37,6 +51,7 @@ export const chooseSchoolRoute = new Route({
               className="h-full"
             >
               <Button size="card" variant="secondary" className="h-full w-full">
+                {getSchoolEmoji(school)}
                 <span className="text-lg">{school}</span>
               </Button>
             </Link>
diff --git a/tsconfig.json b/tsconfig.json
index ea0727c5..ad6b6e1c 100644
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -15,9 +15,11 @@
 
     /* Linting */
     "strict": true,
+    "strictNullChecks": true,
     "noUnusedLocals": true,
     "noUnusedParameters": true,
     "noFallthroughCasesInSwitch": true,
+    "noImplicitReturns": true,
 
     // for shadcn/ui
     "baseUrl": ".",

From a1b4b986c9ed38d66e948beccbd7c7d7f26201dd Mon Sep 17 00:00:00 2001
From: Lorenzo Corallo <66379281+lorenzocorallo@users.noreply.github.com>
Date: Thu, 22 Aug 2024 17:08:01 +0200
Subject: [PATCH 5/6] fix: update row total number when filtering the table
 (#265)

Co-authored-by: angeousta <132761637+angeousta@users.noreply.github.com>
---
 src/routes/viewer/Table/Pagination.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/routes/viewer/Table/Pagination.tsx b/src/routes/viewer/Table/Pagination.tsx
index e2c75fbf..bd23dbab 100644
--- a/src/routes/viewer/Table/Pagination.tsx
+++ b/src/routes/viewer/Table/Pagination.tsx
@@ -25,7 +25,7 @@ export default function Pagination<TData>({
     <div className="my-2 flex items-center justify-end max-sm:flex-col max-sm:items-start max-sm:gap-2 sm:space-x-6">
       <div className="sm:flex-1">
         <p className="text-sm font-medium">
-          Totale righe: {table.getCoreRowModel().rows.length}
+          Totale righe: {table.getFilteredRowModel().rows.length}
         </p>
       </div>
       <div className="flex items-center space-x-2">

From c47171ce53905b9e24114f25056df35e6f32270d Mon Sep 17 00:00:00 2001
From: Lorenzo Corallo <66379281+lorenzocorallo@users.noreply.github.com>
Date: Thu, 22 Aug 2024 17:16:00 +0200
Subject: [PATCH 6/6] feat: matricola search (#264)

* fix: matricola searching

before it was not working at all

* feat: improve matricola search filter

* style: change matricola input width to show error in one line
---
 src/routes/viewer/CourseCombobox.tsx          |  2 +-
 src/routes/viewer/LocationSelect.tsx          |  2 +-
 .../viewer/PhaseSelect/GroupSelect/index.tsx  |  2 +-
 .../viewer/PhaseSelect/LangSelect/index.tsx   |  2 +-
 .../PhaseSelect/RankingSelect/index.tsx       |  2 +-
 src/routes/viewer/Table/FilterBtn.tsx         |  4 +-
 src/routes/viewer/Table/Toolbar.tsx           | 95 ++++++++++++++++---
 src/routes/viewer/Table/columns.tsx           |  4 +-
 8 files changed, 91 insertions(+), 22 deletions(-)

diff --git a/src/routes/viewer/CourseCombobox.tsx b/src/routes/viewer/CourseCombobox.tsx
index d6e81e49..7ff13626 100644
--- a/src/routes/viewer/CourseCombobox.tsx
+++ b/src/routes/viewer/CourseCombobox.tsx
@@ -38,7 +38,7 @@ export function CourseCombobox({ value, courses: c, onSelect }: Props) {
 
   return (
     <div className="flex items-center space-x-4">
-      <p className="text-muted-foreground text-sm">Corso</p>
+      <p className="text-sm">Corso</p>
       {selectedCourse ? (
         <Removable onRemove={() => handleSelect(absCourse.value)}>
           {selectedCourse.label}
diff --git a/src/routes/viewer/LocationSelect.tsx b/src/routes/viewer/LocationSelect.tsx
index 324b8545..873a7af1 100644
--- a/src/routes/viewer/LocationSelect.tsx
+++ b/src/routes/viewer/LocationSelect.tsx
@@ -35,7 +35,7 @@ export default function LocationsSelect(props: Props) {
   if (locations.length === 0) return <></>;
   return (
     <div className="flex items-center space-x-4">
-      <p className="text-muted-foreground text-sm">Sede</p>
+      <p className="text-sm">Sede</p>
       {locations.length >= 2 ? (
         isMobile ? (
           LocationCombobox(props)
diff --git a/src/routes/viewer/PhaseSelect/GroupSelect/index.tsx b/src/routes/viewer/PhaseSelect/GroupSelect/index.tsx
index 1918331f..df12e6ae 100644
--- a/src/routes/viewer/PhaseSelect/GroupSelect/index.tsx
+++ b/src/routes/viewer/PhaseSelect/GroupSelect/index.tsx
@@ -11,7 +11,7 @@ export default function GroupSelect(props: GroupSelectProps) {
   const { groups, isCombobox, selectedGroup, groupOpen, onChange } = props;
   return (
     <div className="flex items-center space-x-4">
-      <p className="text-muted-foreground text-sm">Fase</p>
+      <p className="text-sm">Fase</p>
       {groups.size >= 2 ? (
         isCombobox ? (
           <GroupCombobox
diff --git a/src/routes/viewer/PhaseSelect/LangSelect/index.tsx b/src/routes/viewer/PhaseSelect/LangSelect/index.tsx
index 562f4933..f4596a77 100644
--- a/src/routes/viewer/PhaseSelect/LangSelect/index.tsx
+++ b/src/routes/viewer/PhaseSelect/LangSelect/index.tsx
@@ -29,7 +29,7 @@ export default function LangSelect({
 
   return (
     <div className="flex items-center space-x-4">
-      <p className="text-muted-foreground text-sm">Lingua</p>
+      <p className="text-sm">Lingua</p>
       {canChoose ? (
         <Tabs
           value={selectedLang}
diff --git a/src/routes/viewer/PhaseSelect/RankingSelect/index.tsx b/src/routes/viewer/PhaseSelect/RankingSelect/index.tsx
index 2f1c3b1d..a4bbf65d 100644
--- a/src/routes/viewer/PhaseSelect/RankingSelect/index.tsx
+++ b/src/routes/viewer/PhaseSelect/RankingSelect/index.tsx
@@ -11,7 +11,7 @@ export default function RankingSelect(props: RankingSelectProps) {
   const { phases, isCombobox, rankingOpen, selectedPhase, onChange } = props;
   return (
     <div className="flex items-center space-x-4">
-      <p className="text-muted-foreground text-sm">Graduatoria</p>
+      <p className="text-sm">Graduatoria</p>
       {phases.length >= 2 ? (
         isCombobox ? (
           <RankingCombobox
diff --git a/src/routes/viewer/Table/FilterBtn.tsx b/src/routes/viewer/Table/FilterBtn.tsx
index f44dc8a4..ad1563c3 100644
--- a/src/routes/viewer/Table/FilterBtn.tsx
+++ b/src/routes/viewer/Table/FilterBtn.tsx
@@ -132,9 +132,7 @@ export function FilterBtn<TData, TValue>({
                     >
                       <CheckIcon className={cn("h-4 w-4")} />
                     </div>
-                    {option.icon && (
-                      <option.icon className="text-muted-foreground mr-2 h-4 w-4" />
-                    )}
+                    {option.icon && <option.icon className="mr-2 h-4 w-4" />}
                     <span>{option.label}</span>
                     {facet && (
                       <span className="ml-auto flex h-4 w-4 items-center justify-center font-mono text-xs">
diff --git a/src/routes/viewer/Table/Toolbar.tsx b/src/routes/viewer/Table/Toolbar.tsx
index 7ff49ca5..367d223a 100644
--- a/src/routes/viewer/Table/Toolbar.tsx
+++ b/src/routes/viewer/Table/Toolbar.tsx
@@ -6,6 +6,10 @@ import { FilterBtn } from "./FilterBtn";
 import { enrollStatusOpts, enrollAllowedOpts } from "./columns";
 import { StudentResultKeys } from ".";
 import StudentResult from "@/utils/types/data/parsed/Ranking/StudentResult";
+import { sha256 } from "@/utils/strings/crypto";
+import { useState } from "react";
+import { LuXCircle } from "react-icons/lu";
+import { Removable } from "@/components/custom-ui/Removable";
 
 type Props = {
   has: Record<StudentResultKeys, boolean>;
@@ -16,20 +20,89 @@ type Props = {
 export function Toolbar({ has, onCsvClick, table }: Props) {
   const enrollStatusCol = table.getColumn("enrollStatus");
   const enrollAllowedCol = table.getColumn("enrollAllowed");
+  const [matricolaFilter, setMatricolaFilter] = useState<string>("");
+  const [matricolaFilterSubmitted, setMatricolaFilterSubmitted] =
+    useState<boolean>(false);
+  const { rows: filteredRows } = table.getFilteredRowModel();
+
+  function clearMatricolaTableFilter() {
+    table.getColumn("id")?.setFilterValue(undefined);
+  }
+
+  function handleClearMatricolaFilter() {
+    setMatricolaFilter("");
+    clearMatricolaTableFilter();
+    setMatricolaFilterSubmitted(false);
+  }
+
+  function handleMatricolaFilterChange(
+    event: React.ChangeEvent<HTMLInputElement>,
+  ) {
+    if (matricolaFilterSubmitted) clearMatricolaTableFilter();
+    const input = event.target.value;
+    setMatricolaFilter(input);
+    setMatricolaFilterSubmitted(false);
+  }
+
+  async function handleMatricolaFilterSubmit(
+    e: React.FormEvent<HTMLFormElement>,
+  ) {
+    e.preventDefault();
+
+    if (matricolaFilter.length === 0) handleClearMatricolaFilter();
+    else {
+      const hash = await sha256(matricolaFilter);
+      table.getColumn("id")?.setFilterValue(hash);
+      setMatricolaFilterSubmitted(true);
+    }
+  }
 
   return (
-    <div className="flex w-full flex-wrap items-center justify-start gap-4 max-sm:items-start max-2xs:flex-col">
+    <div className="flex w-full flex-wrap items-start justify-start gap-6 max-2xs:flex-col">
       {table.getColumn("id") && (
-        <Input
-          className="w-full sm:max-w-[300px]"
-          placeholder="Filter per matricola..."
-          value={(table.getColumn("id")?.getFilterValue() as string) ?? ""}
-          onChange={(event) =>
-            table.getColumn("id")?.setFilterValue(event.target.value)
-          }
-        />
+        <div className="grid grid-cols-[auto_220px] grid-rows-[auto_auto] gap-x-4 gap-y-1">
+          <p className="self-center text-sm">Matricola</p>
+          {filteredRows.length > 0 && matricolaFilterSubmitted ? (
+            <Removable
+              onRemove={handleClearMatricolaFilter}
+              className="justify-between"
+            >
+              {matricolaFilter}
+            </Removable>
+          ) : (
+            <>
+              <form
+                className="relative w-full"
+                onSubmit={handleMatricolaFilterSubmit}
+              >
+                <Input
+                  placeholder="Inserisci la matricola..."
+                  value={matricolaFilter}
+                  onChange={handleMatricolaFilterChange}
+                />
+                {matricolaFilter.length > 0 && (
+                  <Button
+                    type="button"
+                    variant="ghost"
+                    size="icon"
+                    className="absolute right-1 top-1/2 h-7 w-7 -translate-y-1/2 text-gray-500 hover:bg-transparent hover:text-gray-900 dark:text-gray-400 dark:hover:bg-transparent dark:hover:text-gray-100"
+                    onClick={handleClearMatricolaFilter}
+                  >
+                    <LuXCircle className="h-5 w-5" />
+                    <span className="sr-only">Clear</span>
+                  </Button>
+                )}
+              </form>
+              {matricolaFilterSubmitted && (
+                <p className="col-start-2 text-sm text-red-600 dark:text-red-400">
+                  Matricola non trovata, ricontrolla.
+                </p>
+              )}
+            </>
+          )}
+        </div>
       )}
-      <div className="flex flex-1 justify-start gap-4 max-xs:flex-wrap">
+      <div className="flex flex-1 justify-start gap-6 max-xs:flex-wrap">
         {has.enrollAllowed && enrollAllowedCol && (
           <FilterBtn
             column={enrollAllowedCol}
@@ -45,7 +118,7 @@ export function Toolbar({ has, onCsvClick, table }: Props) {
           />
         )}
       </div>
-      <div className="flex justify-end ">
+      <div className="flex justify-end">
         <Button
           variant="outline"
           className="whitespace-nowrap"
diff --git a/src/routes/viewer/Table/columns.tsx b/src/routes/viewer/Table/columns.tsx
index 4b217d95..49112e51 100644
--- a/src/routes/viewer/Table/columns.tsx
+++ b/src/routes/viewer/Table/columns.tsx
@@ -60,9 +60,7 @@ class Formatter {
 
     return (
       <div className="flex items-center justify-center">
-        {option.icon && (
-          <option.icon className="text-muted-foreground mr-2 h-4 w-4" />
-        )}
+        {option.icon && <option.icon className="mr-2 h-4 w-4" />}
         <span>{option.label}</span>
       </div>
     );