[ACS-10280]: Keyboard Navigation: Search: Unnecessary stops in the results table using the Tab key

[rmnvch]

Please check if the PR fulfills these requirements

• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• Documentation
• Other... Please describe:

What is the current behaviour? (You can also link to an open issue here)
https://hyland.atlassian.net/browse/ACS-10280

What is the new behaviour?

Does this PR introduce a breaking change? (check one with "x")

• Yes
• No

!!!this PR removes decorative icon/image cells from
adf-datatable's accessibility tree. That change breaks

'XAT-16631 Should be able to correctly display data table with nested object and special characters in JSON keys for Path to JSON property'

on the hxp project where empty cell is expected and reserved for icon/Image cell.

suggested fix:

[Copilot]

Pull request overview

This PR targets an accessibility/keyboard-navigation issue in the DataTable used by search results, aiming to reduce unnecessary Tab stops and improve discoverability of hover-only row controls during keyboard interaction.

Changes:

• Removes row-focus configuration (allowFocusOnRows / setAllowFocusOnTableRows) from the DataTableAdapter API and from ShareDataTableAdapter.
• Updates the DataTable template to stop passing a disabled binding to rows (impacting row focus/tabindex behavior).
• Adds a CSS rule to reveal “hover-only” controls when the table has focus within, and removes related unit tests.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
lib/core/src/lib/datatable/data/datatable-adapter.ts	Removes exported adapter members previously used to control row focus behavior.
lib/core/src/lib/datatable/components/datatable/datatable.component.html	Stops binding disabled into adf-datatable-row, changing row focusability/tab order.
lib/core/src/lib/datatable/components/datatable/datatable.component.scss	Shows hover-only controls when the datatable contains focus (keyboard support).
lib/core/src/lib/datatable/components/datatable/datatable.component.spec.ts	Removes tests related to adapter-controlled row focus.
lib/content-services/src/lib/document-list/data/share-datatable-adapter.ts	Removes public focus-control property/method from the Share adapter.
lib/content-services/src/lib/document-list/data/share-datatable-adapter.spec.ts	Removes unit tests for the deleted Share adapter members.

Comments suppressed due to low confidence (1)

lib/core/src/lib/datatable/components/datatable/datatable.component.spec.ts:1823

• The PR removes the only tests that asserted row tabbability could be controlled via an adapter, but it doesn’t add/adjust tests to cover the new expected keyboard behavior (Tab stops, ArrowUp/ArrowDown navigation, Shift+Arrow drag&drop). Please add/update accessibility tests to lock in the intended focus order and prevent regressions.

    it('should create focus trap on main menu open', () => {
        dataTable.showHeader = ShowHeaderMode.Always;
        dataTable.showMainDatatableActions = true;
        dataTable.mainActionTemplate = columnCustomTemplate;

---

You can also share your feedback on Copilot code review. Take the survey.

[DenysVuika]

can this be converted to @for angular flow syntax?

[MichalKinas]

Done, I've refactored the whole file with new control flow syntax

[sonarqubecloud]

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud