Skip to content

Issue 2153 - changed account analysis dashboard UI#2156

Merged
rmroot merged 1 commit intodevelopfrom
issue-2153
Nov 17, 2025
Merged

Issue 2153 - changed account analysis dashboard UI#2156
rmroot merged 1 commit intodevelopfrom
issue-2153

Conversation

@jamlokim
Copy link
Copy Markdown
Contributor

connects #2153

This pull request refactors the account analysis dashboard tabs and introduces a new, feature-rich analysis details table for improved user experience and maintainability. The main changes include removing the old dashboard tabs component, updating dashboard layout and navigation, and adding a new details table component with filtering, sorting, and error display capabilities.

Dashboard Tabs Removal and Layout Updates

  • Removed the AccountAnalysisDashboardTabsComponent and its related files, including its HTML, CSS, TypeScript, and spec files, simplifying the dashboard and removing tab navigation.
  • Updated the dashboard layout in account-analysis-dashboard.component.html by removing the tabs reference and moving the "selected analysis for report year" indicator to the new details table.

New Analysis Details Table Component

  • Added a new AccountAnalysisDetailsTableComponent with comprehensive table features: filtering by category and report year, sorting, pagination, selection indicators, error handling, and delete confirmation modal.
  • Introduced new CSS for the details table, including styling for table headers, icons, filters, and error states for improved visual clarity.
  • Added a corresponding spec file for the new component to ensure proper testing and maintainability.

Minor Logic and Navigation Fix

  • Fixed the dashboard detection logic in AccountAnalysisBannerComponent to use the correct route for analysis, ensuring proper banner behavior.

@jamlokim jamlokim requested a review from rmroot November 17, 2025 14:10
@rmroot rmroot merged commit 95dcdc6 into develop Nov 17, 2025
3 checks passed
@jamlokim jamlokim deleted the issue-2153 branch January 12, 2026 16:09
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.

2 participants