feat: Add buttons to toggle between legacy and new views in Workflow History #1133
Merged
adhityamamallan merged 10 commits intocadence-workflow:masterfrom Jan 16, 2026
Merged
Conversation
adhityamamallan
force-pushed
the
history-v2-toggle-buttons
branch
2 times, most recently
from
bb9eaa5 to
2b51d20
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds toggle buttons to switch between legacy and new Workflow History views based on a feature flag configuration. The buttons appear next to the "Workflow History" heading and include informative tooltips explaining the differences and providing feedback links.
Changes:
- Created a reusable
WorkflowHistoryViewToggleButtoncomponent with tooltip functionality - Added switch buttons to both legacy (V1) and new (V2) history views
- Updated header layouts to accommodate the new toggle buttons with responsive design
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| workflow-history-view-toggle-button.tsx | New reusable button component with tooltip for view switching |
| workflow-history-view-toggle-button.types.ts | Type definitions for the toggle button props |
| workflow-history-view-toggle-button.styles.ts | Styling for button variants and tooltip |
| workflow-history-switch-to-v2-button.tsx | V1-to-V2 switch button with conditional rendering based on config |
| workflow-history-switch-to-v1-button.tsx | V2-to-V1 switch button with conditional rendering based on config |
| workflow-history-switch-to-v2-button-tooltip-content.config.ts | Tooltip content configuration for the V2 switch button |
| workflow-history-switch-to-v1-button-tooltip-content.config.ts | Tooltip content configuration for the V1 switch button with feedback link |
| workflow-history-header.tsx (both V1 and V2) | Updated headers to include switch buttons and replaced HeadingXSmall with styled Heading component |
| workflow-history-header.styles.ts (both V1 and V2) | Added responsive Heading styled component with flex layout |
| Test files | Comprehensive unit tests for all new components |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+6
to
+9
| `The new Workflow History UI provides a table-based layout with a more compact | ||
| overview of your workflow's history, including aggregated event summaries (shown | ||
| inline and in a Summary tab), color codes for different event types, a floating | ||
| navigation bar for quick event discovery, and a sticky header.`, |
There was a problem hiding this comment.
The tooltip content contains excessive leading whitespace on line 7 onwards. Consider using a template literal without indentation or using array join to maintain clean formatting without extra whitespace.
Comment on lines
+8
to
+12
| `The new History view provides a more compact overview with a table-based | ||
| format designed for information density and easier scanning, helping you | ||
| find important events faster.`, | ||
| `Please feel free to share any feedback if you encounter anything | ||
| that seems suboptimal in the new History view.`, |
There was a problem hiding this comment.
The tooltip content contains excessive leading whitespace. Consider using a template literal without indentation or using array join to maintain clean formatting without extra whitespace.
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
Signed-off-by: Adhitya Mamallan <adhitya.mamallan@uber.com>
adhityamamallan
force-pushed
the
history-v2-toggle-buttons
branch
from
2b51d20 to
6f0b9b5
Compare
Assem-Uber
approved these changes
Jan 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
Added unit tests + ran locally.
When feature flag is
DISABLEDWhen feature flag is
ENABLEDWhen feature flag is
OPT_INNote that the last-used history view is remembered.
Screen.Recording.2026-01-08.at.15.23.04.mov
When feature flag is
OPT_OUTNote that the history page always defaults to the new view.
Screen.Recording.2026-01-08.at.15.24.22.mov
Tooltip content
On narrow screens