Skip to content

Redesign build summary page charts#3794

Draft
williamjallen wants to merge 1 commit into
Kitware:masterfrom
williamjallen:build-summary-charts
Draft

Redesign build summary page charts#3794
williamjallen wants to merge 1 commit into
Kitware:masterfrom
williamjallen:build-summary-charts

Conversation

@williamjallen

Copy link
Copy Markdown
Collaborator

This PR completely redesigns the charts on the build summary page, switching from a series of charts with time on the x-axis, to a single stacked bar chart showing relative configure, build, and test times, as well as passing/failing status via a colored background. I intend to follow this with a series of PRs introducing new charts, moving the comments feature to a dedicated page, and rethinking the previous/current/next tables.

New UI:
image

@josephsnyder josephsnyder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

When looking at a subproject build, echarts is unable to render data. The error message there also appears on pages that do render data, so I think it's a red herring for this problem, but worth looking into.
Image
Is that expected?

It also seems that the filter display generated by the Show History button doesn't seem to return any valid data to the screen, but no errors are found in the console or network tabs. Shouldn't we expect to see at least the build we just left in that data?

@williamjallen williamjallen marked this pull request as draft June 15, 2026 17:35
@williamjallen

Copy link
Copy Markdown
Collaborator Author

@josephsnyder Good catch--I got a little ahead of myself here. Parent builds work, but child builds do not because they don't appear in the list of builds for a project. Converting to draft until I fix some blockers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants