Skip to content

fix: Match VPC metrics toolbar to the portal - #908

Merged
mattdjenkinson merged 1 commit into
mainfrom
fix/vpc-metrics-toolbar-button-group
Nov 12, 2026
Merged

mattdjenkinson merged 1 commit into
mainfrom
fix/vpc-metrics-toolbar-button-group

Conversation

@mattdjenkinson

@mattdjenkinson mattdjenkinson commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The step and refresh controls on the VPC metrics page were hand-built, so they looked different from the same controls on the ALB metrics page. They now use the shared datum-ui button group, with the same styling the portal uses for its own metrics controls. The region, instance and time range pickers now come from the portal instead of an older bundled copy, so they follow the current field styling too. The plugin's datum-ui dependency moves from 3.8 to 3.12.

Test plan

  • Typecheck, unit tests and production build pass
  • Compared before and after toolbars on a page styled only with classes the portal generates
  • Loaded the plugin in a local portal and checked the VPC metrics tab
  • Check the VPC metrics toolbar against the ALB one in staging

The VPC metrics step and refresh controls were hand-built with a plain
border div and borderless select, so they looked different from the ALB
metrics page. The region, instance and time pickers also bundled
datum-ui 3.8, which predates the shared field trigger spec.

Key changes:
- build step and refresh from datum-ui ButtonGroup, ButtonGroupText and
  Button, with the same classes as the portal's metrics controls
- take multi-select and picker from the host instead of bundling them
- bump datum-ui to ^3.12.0
@mattdjenkinson
mattdjenkinson requested a review from a team as a code owner November 12, 2026 14:46
@mattdjenkinson mattdjenkinson self-assigned this Nov 12, 2026
@mattdjenkinson
mattdjenkinson merged commit beb2dcb into main Nov 12, 2026
10 checks passed
@mattdjenkinson
mattdjenkinson deleted the fix/vpc-metrics-toolbar-button-group branch November 12, 2026 15:30
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