Skip to content

Lite - fixed sort on columns that are formatted for seconds or milliseconds#451

Closed
dphugo wants to merge 1 commit intoerikdarlingdata:mainfrom
dphugo:fix/lite-sort-formatted-columns-by-value
Closed

Lite - fixed sort on columns that are formatted for seconds or milliseconds#451
dphugo wants to merge 1 commit intoerikdarlingdata:mainfrom
dphugo:fix/lite-sort-formatted-columns-by-value

Conversation

@dphugo
Copy link

@dphugo dphugo commented Mar 6, 2026

What does this PR do?

This PR fixes bug/issue #450.

  • Used the SortMemberPath property to point to the numeric value for the sorting of the columns that are formatted for seconds or milliseconds under the Server Tab -> Collection Health tabs.
  • Increased some column widths to ensure complete header is shown.

Which component(s) does this affect?

  • Full Dashboard
  • Lite Dashboard
  • Lite Tests
  • SQL collection scripts
  • CLI Installer
  • GUI Installer
  • Documentation

How was this tested?

Describe the testing you've done. Include:

  • SQL Server version(s) tested against:
    • SQL Server 2022
  • Steps to verify the change works:
    • Complied and ran the PerformanceMonitorLite project
    • Waited for Collection to finish
    • Browsed to the affected data grids under Server Tab -> Collection Health tabs.
    • Clicked on the affected columns to verify visually that sorting was applied correctly. Also specifically where the output could be a mix of "s" and "ms".

Checklist

  • I have read the contributing guide
  • My code builds with zero warnings (dotnet build -c Debug)
  • I have tested my changes against at least one SQL Server version
  • I have not introduced any hardcoded credentials or server names

…, miliseconds or both to order correctly.

- Used the SortMemberPath property to point to the numeric value.
- Increased some widths of columns to ensure complete header is shown.
@dphugo dphugo closed this Mar 6, 2026
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