@autobusal/common 1.26.0 → 1.26.1

package/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 1.26.1
+
+### Fixed
+
+- **`Table` drew an empty "Options" column when a table had only
+  top-of-table controls.** `create`, `search` and `extra` all live *above* the
+  table — `Actions/Get` already returns null for each — but `Header` and
+  `Rows` decided whether to draw the options column from `actions.length > 0`,
+  which counted them. A table using the component purely for sorting, search
+  and paging therefore got an "Options" heading with an empty cell under every
+  row.
+
+  Invisible on the admin screens, where every table has row actions anyway;
+  visible the moment the public bus-company directory used it. The row/top
+  split is one shared list now (`Table/rowActions.ts`) rather than a condition
+  repeated in three files that could drift.
+
 ## 1.26.0
 
 ### Added

package/Table/Header/Header.tsx

@@ -1,6 +1,7 @@
 import { TFunction } from 'i18next';
 import Sort from './Sort';
 import { getUrl } from '../browseUrl';
+import { rowActions } from '../rowActions';
 import { Container, Td, Column, Text } from './styles';
 import { ColumnData, SortingData } from '../types';
 
@@ -31,7 +32,7 @@ const Header = ({ url, data, sorting, t, actions }: Props): JSX.Element => {
       <tr>
         { columns }
 
-        { (actions !== undefined && actions.length > 0) && (
+        { rowActions(actions).length > 0 && (
           <Td $width={ 10 }>
             <Column>
               <Text>{ t('table.options', { ns: 'common' }) }</Text>

package/Table/Rows/Rows.tsx

@@ -3,6 +3,7 @@ import { NavigateFunction } from 'react-router-dom';
 import Loading from './Loading';
 import NotFound from './NotFound';
 import Actions from '../Actions/Actions';
+import { rowActions } from '../rowActions';
 import { Tbody, Tr } from './styles';
 
 interface Props {
@@ -39,7 +40,7 @@ const Rows = ({ url, urlWith, data, loading, fetching, colLength, t, actions, ha
     <Tr key={ index } $viewable={ isViewable } onClick={ () => onClick(item.props.id) }>
       { item }
 
-      { (actions !== undefined && actions.length > 0) && (
+      { rowActions(actions).length > 0 && (
         <Actions
           id={ item.props.id }
           url={ url }

package/Table/rowActions.ts

@@ -0,0 +1,22 @@
+/**
+ * Which of a table's actions belong to a ROW rather than to the top of the
+ * table.
+ *
+ * Ferjolt Ozuni - Date: 2026-08-06
+ *
+ * `create`, `search` and `extra` are all controls that live ABOVE the table -
+ * Top renders them and Actions/Get already returns null for each. But Header
+ * and Rows both decided whether to draw the options column from
+ * `actions.length > 0`, counting those three, so a table with only
+ * top-of-table controls got an "Options" heading with an empty cell under
+ * every row. Harmless on the admin screens, where every table has row actions
+ * anyway - visible the moment a PUBLIC page used the component for its
+ * sorting and paging alone.
+ *
+ * One list, shared by the three places that need it, so it cannot drift.
+ */
+export const TOP_ACTIONS = ['create', 'search', 'extra'];
+
+export const rowActions = (actions?: string[]): string[] => (
+  (actions ?? []).filter(action => !TOP_ACTIONS.includes(action))
+);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@autobusal/common",
-  "version": "1.26.0",
+  "version": "1.26.1",
   "author": "Ferjolt Ozuni",
   "type": "module",
   "main": "index.ts"