-
-
Notifications
You must be signed in to change notification settings - Fork 90
Added column tooltips & support for tooltips in custom reports #1589
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added column tooltips & support for tooltips in custom reports #1589
Conversation
Refactor custom reports with a single dictionary mapping metadata to each column. Add description. trimble-oss#1582
Easier to read & maintain.
Use new format instead of backward compatibility layer, simplifying report definitions.
Update column ToolTips for QS, Summary & Database Finder. trimble-oss#1582
There was a problem hiding this 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
ColumnMetadatastructure 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.
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this 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.
Added support for column tooltips in custom reports.
Refactoring of custom reports
Updated tooltips on various reports.
#1582