Skip to content

Issue 2108 - added name of the selected analysis item in account and facility#2112

Merged
rmroot merged 2 commits intodevelopfrom
issue-2108
Oct 17, 2025
Merged

Issue 2108 - added name of the selected analysis item in account and facility#2112
rmroot merged 2 commits intodevelopfrom
issue-2108

Conversation

@jamlokim
Copy link
Copy Markdown
Contributor

connects #2108

This pull request enhances the "Energy Reduction Goal" and "Water Reduction Goal" cards for both account and facility dashboards by displaying the name of the analysis used and allowing users to navigate directly to the relevant analysis details. Additionally, styling improvements are made for consistency. The main changes are grouped below:

Feature Enhancements:

  • Added a new UI section to each goal card (account-energy-reduction-goal, account-water-reduction-goal, facility-energy-reduction-goal, facility-water-reduction-goal) to display the name of the analysis item used, with a clickable link for navigation.
  • Implemented a goToAnalysisItem() method in each component to set the selected analysis item and navigate the user to either the analysis setup or results page, depending on the presence of setup errors.

State Management and Data Handling:

  • Updated each component to track the latest analysis item in a dedicated property (latestAnalysisItem) instead of a local variable, ensuring the UI and navigation logic use the correct analysis reference.

Styling Improvements:

  • Added a new CSS class .item-name to standardize the font size of the analysis item display across all affected components.

These changes collectively improve user experience by making it clearer which analysis is being used for each goal and providing a direct way to access further analysis details or resolve setup issues.

@jamlokim jamlokim requested a review from rmroot October 17, 2025 15:29
@jamlokim jamlokim requested a review from rmroot October 17, 2025 16:00
@rmroot rmroot merged commit 9965f22 into develop Oct 17, 2025
3 checks passed
@jamlokim jamlokim deleted the issue-2108 branch October 20, 2025 18:38
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