@carlonicora/nextjs-jsonapi 1.126.0 → 1.128.0

package/src/components/tables/ContentListTable.tsx

@@ -5,6 +5,7 @@ import { ExpandedState, flexRender, getCoreRowModel, getExpandedRowModel, useRea
 
 import { cn } from "@/index";
 import { ChevronLeft, ChevronRight } from "lucide-react";
+import { useTranslations } from "next-intl";
 import React, { ReactNode, memo, useMemo, useState } from "react";
 import { DataListRetriever, TableContent, useTableGenerator } from "../../hooks";
 import { ModuleWithPermissions } from "../../permissions";
@@ -48,10 +49,12 @@ type ContentListTableProps = {
   fullWidth?: boolean;
   groupBy?: string;
   emptyState?: ReactNode;
+  onRowClick?: (rowData: any) => void;
 };
 
 export const ContentListTable = memo(function ContentListTable(props: ContentListTableProps) {
-  const { data, fields, checkedIds, toggleId, allowSearch, filters: _filters, fullWidth } = props;
+  const { data, fields, checkedIds, toggleId, allowSearch, filters: _filters, fullWidth, onRowClick } = props;
+  const t = useTranslations();
   const [expanded, setExpanded] = useState<ExpandedState>(
     props.defaultExpanded === true ? true : typeof props.defaultExpanded === "object" ? props.defaultExpanded : {},
   );
@@ -190,7 +193,11 @@ export const ContentListTable = memo(function ContentListTable(props: ContentLis
                       </TableCell>
                     </TableRow>
                     {group.rows.map((row) => (
-                      <TableRow key={row.id}>
+                      <TableRow
+                        key={row.id}
+                        onClick={() => onRowClick?.(row.original.jsonApiData)}
+                        className={`group ${onRowClick ? "hover:bg-muted/50 cursor-pointer" : ""}`}
+                      >
                         {row.getVisibleCells().map((cell) => {
                           const meta = cell.column.columnDef.meta as { className?: string } | undefined;
                           return (
@@ -205,7 +212,11 @@ export const ContentListTable = memo(function ContentListTable(props: ContentLis
                 ))
               ) : (
                 rowModel.rows.map((row) => (
-                  <TableRow key={row.id}>
+                  <TableRow
+                    key={row.id}
+                    onClick={() => onRowClick?.(row.original.jsonApiData)}
+                    className={`group ${onRowClick ? "hover:bg-muted/50 cursor-pointer" : ""}`}
+                  >
                     {row.getVisibleCells().map((cell) => {
                       const meta = cell.column.columnDef.meta as { className?: string } | undefined;
                       return (
@@ -220,7 +231,8 @@ export const ContentListTable = memo(function ContentListTable(props: ContentLis
             ) : (
               <TableRow>
                 <TableCell colSpan={tableColumns.length} className="h-24 text-center">
-                  {props.emptyState ?? "No results."}
+                  {props.emptyState ??
+                    (t.has("ui.empty_states.no_results") ? t("ui.empty_states.no_results") : "No results.")}
                 </TableCell>
               </TableRow>
             )}

package/src/components/tables/__tests__/ContentListTable.test.tsx

@@ -132,7 +132,7 @@ describe("ContentListTable", () => {
 
       render(<ContentListTable data={dataRetriever} tableGeneratorType={mockModule as any} fields={["id", "title"]} />);
 
-      expect(screen.getByText("No results.")).toBeInTheDocument();
+      expect(screen.getByText("ui.empty_states.no_results")).toBeInTheDocument();
     });
   });
 
@@ -464,7 +464,7 @@ describe("ContentListTable", () => {
         />,
       );
 
-      expect(screen.getByText("No results.")).toBeInTheDocument();
+      expect(screen.getByText("ui.empty_states.no_results")).toBeInTheDocument();
     });
 
     it("should group by to-many relationship, duplicating rows across groups", () => {
@@ -512,4 +512,40 @@ describe("ContentListTable", () => {
       expect(article3Count).toBe(2);
     });
   });
+
+  describe("onRowClick", () => {
+    it("calls onRowClick with the row's jsonApiData when a row is clicked", () => {
+      const onRowClick = vi.fn();
+      const dataRetriever = createMockDataRetriever({
+        data: [{ id: "1", title: "First Article" }],
+      });
+
+      render(
+        <ContentListTable
+          data={dataRetriever}
+          tableGeneratorType={mockModule as any}
+          fields={["id", "title"]}
+          onRowClick={onRowClick}
+        />,
+      );
+
+      fireEvent.click(screen.getByText("First Article"));
+      expect(onRowClick).toHaveBeenCalledTimes(1);
+      expect(onRowClick).toHaveBeenCalledWith({ id: "1", title: "First Article" });
+    });
+
+    it("does not attach a row click handler when onRowClick is absent", () => {
+      const dataRetriever = createMockDataRetriever({
+        data: [{ id: "1", title: "First Article" }],
+      });
+
+      render(<ContentListTable data={dataRetriever} tableGeneratorType={mockModule as any} fields={["id", "title"]} />);
+
+      // No throw on click, and the row has no cursor-pointer affordance class.
+      const cell = screen.getByText("First Article");
+      const row = cell.closest("tr")!;
+      expect(row.className).not.toContain("cursor-pointer");
+      fireEvent.click(cell); // must not throw
+    });
+  });
 });

package/src/features/company/components/lists/CompaniesList.tsx

@@ -23,7 +23,7 @@ export function CompaniesList() {
   return (
     <ContentListTable
       data={data}
-      fields={[CompanyFields.name, CompanyFields.createdAt]}
+      fields={[CompanyFields.name]}
       tableGeneratorType={Modules.Company}
       functions={functions}
       title={t(`entities.companies`, { count: 2 })}

package/src/features/content/components/lists/ContentsListById.tsx

@@ -23,7 +23,7 @@ export function ContentsListById({ contentIds }: ContentsListByIdProps) {
   return (
     <ContentListTable
       data={data}
-      fields={[ContentFields.name, ContentFields.authors, ContentFields.updatedAt]}
+      fields={[ContentFields.name, ContentFields.authors]}
       tableGeneratorType={Modules.Content}
       title={t(`common.relevant`)}
     />

package/src/features/content/components/lists/RelevantContentsList.tsx

@@ -23,7 +23,7 @@ export function RelevantContentsList({ id }: RelevantContentsListProps) {
   return (
     <ContentListTable
       data={data}
-      fields={[ContentFields.name, ContentFields.authors, ContentFields.relevance, ContentFields.updatedAt]}
+      fields={[ContentFields.name, ContentFields.authors, ContentFields.relevance]}
       tableGeneratorType={Modules.Content}
       title={t(`common.relevant`)}
     />

package/src/features/user/components/lists/AdminUsersList.tsx

@@ -27,7 +27,7 @@ function AdminUsersListInternal({ company }: AdminUsersListProps) {
     <ContentListTable
       title={t(`entities.users`, { count: 2 })}
       data={data}
-      fields={[UserFields.name, UserFields.email, UserFields.createdAt]}
+      fields={[UserFields.name, UserFields.email]}
       tableGeneratorType={Modules.User}
       functions={<UserEditor propagateChanges={data.refresh} adminCreated />}
     />