Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 8, 2025

Actions column in report tables was not pinned right because dynamic columns were appended after it in the merged headers array.

Changes

  • Moved actions column definition to merge logic: Removed actions column from static tableHeaders and adminTableHeaders arrays in both report-table components
  • Append actions as final column: Actions column now added as last element after dynamic itemHeaders in mergedTableHeaders

Affected Components

  • eform-client/src/app/plugins/modules/backend-configuration-pn/components/reports/report-table/report-table.component.ts
  • eform-client/src/app/plugins/modules/backend-configuration-pn/modules/reports/components/report-table/report-table.component.ts

Before/After

// Before: Actions buried in middle of final columns
mergedTableHeaders = [...tableHeaders, ...itemHeaders];
// where tableHeaders already contained actions column, followed by dynamic columns

// After: Actions always last
const actionsColumn = { field: 'actions', pinned: 'right', ... };
mergedTableHeaders = [...tableHeaders, ...itemHeaders, actionsColumn];

This ensures pinned: 'right' property works correctly regardless of dynamic column count.

Original prompt

This section details on the original issue you should resolve

<issue_title>Pin actions to the right of the tables</issue_title>
<issue_description>the tables we need chaged are:

eform-client/src/app/plugins/modules/backend-configuration-pn/modules/reports/components/report-table
eform-client/src/app/plugins/modules/backend-configuration-pn/components/reports/report-table</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Pin actions to the right of the tables in report components Pin actions column to right in report tables Nov 8, 2025
Copilot AI requested a review from renemadsen November 8, 2025 04:36
Copilot finished work on behalf of renemadsen November 8, 2025 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pin actions to the right of the tables

2 participants