Skip to content

Conversation

@DavidWiseman
Copy link
Collaborator

Added support for column tooltips in custom reports.
Refactoring of custom reports
Updated tooltips on various reports.
#1582

Refactor custom reports with a single dictionary mapping metadata to each column.  Add description. trimble-oss#1582
Use new format instead of backward compatibility layer, simplifying report definitions.
Update column ToolTips for QS, Summary & Database Finder. trimble-oss#1582
Copilot AI review requested due to automatic review settings October 9, 2025 18:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request adds comprehensive support for column tooltips in custom reports and refactors the custom reports system to use a unified metadata structure. The change consolidates multiple separate dictionaries into a single ColumnMetadata class for better organization and maintainability.

  • Introduces tooltip support for column headers in custom reports
  • Refactors custom reports to use a unified ColumnMetadata structure instead of separate dictionaries
  • Updates various system reports to use the new structure while maintaining backward compatibility

Reviewed Changes

Copilot reviewed 25 out of 28 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
DBADashGUI/CustomReports/ColumnMetadata.cs New class defining unified column metadata structure with tooltip support
DBADashGUI/CustomReports/CustomReportResults.cs Major refactoring to use new structure with backward compatibility
DBADashGUI/ExtensionMethods.cs Updated to use new metadata structure and set column tooltips
DBADashGUI/CustomReports/CustomReportView.cs Added tooltip editing functionality
Multiple report files Converted to use new ColumnMetadata structure
Files not reviewed (2)
  • DBADashGUI/Checks/Summary.Designer.cs: Language not supported
  • DBADashGUI/Properties/Resources.Designer.cs: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@DavidWiseman DavidWiseman requested a review from Copilot October 9, 2025 18:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 25 out of 28 changed files in this pull request and generated no new comments.

Files not reviewed (2)
  • DBADashGUI/Checks/Summary.Designer.cs: Language not supported
  • DBADashGUI/Properties/Resources.Designer.cs: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@DavidWiseman DavidWiseman merged commit 537896e into trimble-oss:main Oct 10, 2025
@DavidWiseman DavidWiseman deleted the 1582_ColumnDescription branch October 21, 2025 08:25
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.

1 participant