Allow to define a default action when clicking a grid row

Author: javiereguiluz

AGENTS.md

@@ -82,9 +82,9 @@ yarn install                  # Install JS dependencies
 Before submitting changes, run these commands to verify them:
 
 If PHP code changed:
-- [ ] `./vendor/bin/phpstan analyse` passes with no errors
-- [ ] `php-cs-fixer fix --dry-run` shows no issues
-- [ ] Run tests with:
+- `./vendor/bin/phpstan analyse` passes with no errors
+- `php-cs-fixer fix --dry-run` shows no issues
+- Run tests with:
   ```bash
   ./vendor/bin/simple-phpunit                    # All tests
   ./vendor/bin/simple-phpunit tests/Field/       # Specific directory
@@ -93,18 +93,18 @@ If PHP code changed:
   ```
 
 If JS/CSS changed:
-- [ ] `yarn ci` passes with no errors
-- [ ] `yarn biome check --write` shows no issues
-- [ ] `make build-assets` completed successfully
+- `yarn ci` passes with no errors
+- `yarn biome check --write` shows no issues
+- `make build-assets` completed successfully
 
 If Twig templates changed:
-- [ ] `./vendor/bin/twig-cs-fixer lint templates/` passes
+- `./vendor/bin/twig-cs-fixer lint templates/` passes
 
 If documentation changed:
-- [ ] `make linter-docs` to validate RST syntax
+- `make linter-docs` to validate RST syntax
 
 If translations changed:
-- [ ] all locales updated consistently; use English as placeholder if unsure
+- all locales updated consistently; use English as placeholder if unsure
 
 ## Git and Pull Requests
 
@@ -167,6 +167,7 @@ make checks-before-pr
 - Error messages: concise but precise and actionable (e.g. include class names, file paths)
 - Handle exceptions explicitly (no silent catches)
 - Config files in PHP format (`config/services.php`, `translations/*.php`)
+- Prefer project constants (Action::EDIT, EA::QUERY) over hardcoded strings
 
 ### PHPDoc
 - No `@return` for void methods
@@ -214,6 +215,7 @@ make checks-before-pr
 - Use simple names: 'Action 1', 'Field 1', not realistic data
 - Add `void` return type to all test methods
 - Name tests descriptively without `test` prefix duplication
+- Use `@testWith` and data providers when possible to avoid duplicated tests
 
 ### Test Fixtures
 - Entity fixtures in `tests/TestApplication/src/Entity/`

assets/css/easyadmin-theme/datagrids.css

@@ -165,6 +165,21 @@ table.datagrid:not(.datagrid-empty) tr:not(.empty-row) td.actions.actions-as-dro
 .datagrid tbody tr.selected-row td ::-moz-selection {
     background: transparent;
 }
+
+/* clickable rows for default row action */
+.datagrid tr.ea-clickable-row {
+    cursor: pointer;
+}
+.datagrid tr.ea-clickable-row td.actions,
+.datagrid tr.ea-clickable-row td.batch-actions-selector {
+    cursor: default;
+}
+.datagrid tr.ea-clickable-row td.actions a,
+.datagrid tr.ea-clickable-row td.actions button,
+.datagrid tr.ea-clickable-row td.batch-actions-selector .form-check {
+    cursor: pointer;
+}
+
 .datagrid td.actions {
     text-align: right;
 }

assets/js/app.js

@@ -30,6 +30,7 @@ class App {
         this.#createAutoCompleteFields();
         this.#createBatchActions();
         this.#createActionConfirmationModals();
+        this.#createDefaultRowAction();
         this.#createPopovers();
         this.#createTooltips();
         this.#createActionHandlers();
@@ -447,6 +448,85 @@ class App {
         });
     }
 
+    #createDefaultRowAction() {
+        const clickableRows = document.querySelectorAll('tr.ea-clickable-row[data-default-action-url]');
+
+        const interactiveSelectors = [
+            'a',
+            'button',
+            'input',
+            'select',
+            'textarea',
+            '.form-check',
+            '.dropdown',
+            '.actions',
+            '[data-bs-toggle]',
+            '.btn',
+        ];
+
+        const isInteractiveElement = (element) => {
+            // walk up the DOM tree to check if any ancestor is interactive
+            // this also handles elements with pointer-events: none whose clicks bubble to parents
+            let current = element;
+            while (current && current !== document.body) {
+                if (interactiveSelectors.some((selector) => current.matches(selector))) {
+                    return true;
+                }
+                current = current.parentElement;
+            }
+
+            return false;
+        };
+
+        const navigateToUrl = (url) => {
+            // create a temporary link and click it to let Turbo (or other libraries) intercept the navigation
+            const link = document.createElement('a');
+            link.href = url;
+            link.style.display = 'none';
+            document.body.appendChild(link);
+            link.click();
+            document.body.removeChild(link);
+        };
+
+        const handleRowActivation = (row, event) => {
+            // don't navigate if rows are selected (batch mode)
+            if (row.classList.contains('selected-row')) {
+                return;
+            }
+
+            const url = row.dataset.defaultActionUrl;
+            if (url) {
+                navigateToUrl(url);
+            }
+        };
+
+        clickableRows.forEach((row) => {
+            // handle mouse clicks
+            row.addEventListener('click', (event) => {
+                if (isInteractiveElement(event.target)) {
+                    return;
+                }
+
+                handleRowActivation(row, event);
+            });
+
+            // handle keyboard navigation (Enter and Space)
+            row.addEventListener('keydown', (event) => {
+                if ('Enter' !== event.key && ' ' !== event.key) {
+                    return;
+                }
+
+                // don't activate if focus is on an interactive child element
+                if (isInteractiveElement(event.target) && event.target !== row) {
+                    return;
+                }
+
+                event.preventDefault();
+                handleRowActivation(row, event);
+            });
+        });
+    }
+
     #createPopovers() {
         document.querySelectorAll('[data-bs-toggle="popover"]').forEach((popoverElement) => {
             new bootstrap.Popover(popoverElement);

doc/crud.rst

@@ -509,6 +509,72 @@ Templates and Form Options
         ;
     }
 
+Default Row Action
+~~~~~~~~~~~~~~~~~~
+
+By default, when you click on any row of the ``index`` page, you navigate to
+the ``edit`` page of that entity. If the ``edit`` action is not available, it
+falls back to the ``detail`` action. This behavior is called the "default row action"
+and you can configure it with the ``setDefaultRowAction()`` method::
+
+    public function configureCrud(Crud $crud): Crud
+    {
+        return $crud
+            // this is the default behavior: first try 'edit', then fallback to 'detail'
+            ->setDefaultRowAction([Action::EDIT, Action::DETAIL])
+
+            // use a single action (no fallback)
+            ->setDefaultRowAction(Action::EDIT)
+
+            // navigate to 'detail' only
+            ->setDefaultRowAction(Action::DETAIL)
+
+            // use any action name, including custom actions
+            ->setDefaultRowAction('review')
+
+            // define a custom fallback chain (first available action wins)
+            ->setDefaultRowAction([Action::DETAIL, 'preview', Action::EDIT])
+
+            // pass null to disable the row click behavior entirely
+            ->setDefaultRowAction(null)
+        ;
+    }
+
+.. note::
+
+    If none of the configured actions (in the fallback chain) are available for
+    some entity (disabled action, no permission, or condition not met), the row
+    won't be clickable for that entity. This also applies to actions defined
+    inside action groups.
+
+.. tip::
+
+    The default row action can be configured globally in your dashboard (so it
+    applies to all CRUD controllers) and overridden in specific CRUD controllers::
+
+        // in your Dashboard
+        public function configureCrud(): Crud
+        {
+            return Crud::new()
+                // all CRUD controllers will navigate to 'detail' by default
+                ->setDefaultRowAction(Action::DETAIL)
+            ;
+        }
+
+        // in a specific CRUD controller
+        public function configureCrud(Crud $crud): Crud
+        {
+            return $crud
+                // only this CRUD controller will navigate to 'edit'
+                ->setDefaultRowAction(Action::EDIT)
+            ;
+        }
+
+The row click behavior is fully accessible via keyboard (using Enter or Space keys).
+Clicks on checkboxes, buttons, links, or any action elements within the row won't
+trigger the navigation to preserve the expected behavior of those elements.
+Also, rows selected in batch mode won't navigate when clicked.
+
 Other Options
 ~~~~~~~~~~~~~
 

public/app.e409b63d.js public/app.4cd174dc.jspublic/app.e409b63d.js renamed to public/app.4cd174dc.js

@@ -2,10 +2,10 @@
   "entrypoints": {
     "app": {
       "css": [
-        "/app.897c5adb.css"
+        "/app.6fe1b7f1.css"
       ],
       "js": [
-        "/app.e409b63d.js"
+        "/app.4cd174dc.js"
       ]
     },
     "form": {

public/app.e409b63d.js.LICENSE.txt public/app.4cd174dc.js.LICENSE.txtpublic/app.e409b63d.js.LICENSE.txt renamed to public/app.4cd174dc.js.LICENSE.txt

@@ -1,6 +1,6 @@
 {
-  "app.css": "app.897c5adb.css",
-  "app.js": "app.e409b63d.js",
+  "app.css": "app.6fe1b7f1.css",
+  "app.js": "app.4cd174dc.js",
   "form.js": "form.5bccac01.js",
   "page-layout.js": "page-layout.6e9fe55d.js",
   "page-color-scheme.js": "page-color-scheme.30cb23c2.js",

public/app.897c5adb.css public/app.6fe1b7f1.csspublic/app.897c5adb.css renamed to public/app.6fe1b7f1.css

@@ -450,6 +450,34 @@ public function askConfirmationOnBatchActions(bool|string|TranslatableInterface
         return $this;
     }
 
+    /**
+     * Sets the action to execute when clicking on a row in the index page.
+     * Pass a string for a single action, an array for a fallback chain (first available wins),
+     * or null to disable. By default, it tries [Action::EDIT, Action::DETAIL].
+     * The action will only work if it's enabled for the CRUD and the user has permission.
+     *
+     * @param string|string[]|null $actionName
+     */
+    public function setDefaultRowAction(string|array|null $actionName): self
+    {
+        if (\is_string($actionName) && '' === trim($actionName)) {
+            throw new \InvalidArgumentException('The default row action cannot be an empty string. Use null to disable it.');
+        }
+
+        if (\is_array($actionName)) {
+            foreach ($actionName as $action) {
+                /** @phpstan-ignore function.alreadyNarrowedType */
+                if (!\is_string($action) || '' === trim($action)) {
+                    throw new \InvalidArgumentException('All actions in the fallback chain must be non-empty strings.');
+                }
+            }
+        }
+
+        $this->dto->setDefaultRowAction($actionName);
+
+        return $this;
+    }
+
     public function getAsDto(): CrudDto
     {
         $this->dto->setPaginator(new PaginatorDto($this->paginatorPageSize, $this->paginatorRangeSize, 1, $this->paginatorFetchJoinCollection, $this->paginatorUseOutputWalkers));