Skip to content

fix(renderer): render step timeline once after all charts#12

Merged
Garbee merged 3 commits intomainfrom
copilot/fix-duplicate-step-timeline
Feb 14, 2026
Merged

fix(renderer): render step timeline once after all charts#12
Garbee merged 3 commits intomainfrom
copilot/fix-duplicate-step-timeline

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 14, 2026

Step timeline was rendered inside the chart loop, appearing twice in the summary (once after CPU Loads, once after Memory Usages).

Changes

  • Moved step annotation generation outside chart iteration loop
  • Step timeline now renders once after all charts
  • Conditionally generate annotations only when charts contain data

Structure

Before:

### CPU Loads
[chart]
#### Step Timeline
...

### Memory Usages  
[chart]
#### Step Timeline  ← duplicate
...

After:

### CPU Loads
[chart]

### Memory Usages
[chart]

#### Step Timeline  ← single occurrence
...

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits February 14, 2026 16:43
Co-authored-by: Garbee <868301+Garbee@users.noreply.github.com>
Co-authored-by: Garbee <868301+Garbee@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix duplicate step timeline in final summary fix(renderer): render step timeline once after all charts Feb 14, 2026
Copilot AI requested a review from Garbee February 14, 2026 16:47
@Garbee Garbee marked this pull request as ready for review February 14, 2026 16:52
@Garbee Garbee merged commit 3b9e378 into main Feb 14, 2026
7 checks passed
@Garbee Garbee deleted the copilot/fix-duplicate-step-timeline branch February 14, 2026 19:24
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