Skip to content

Prettier deployment plans#12

Open
mclasmeier wants to merge 1 commit into
mainfrom
mc/wider-deployment-plan-box
Open

Prettier deployment plans#12
mclasmeier wants to merge 1 commit into
mainfrom
mc/wider-deployment-plan-box

Conversation

@mclasmeier
Copy link
Copy Markdown
Collaborator

@mclasmeier mclasmeier commented Dec 5, 2025

  • Wider boxes to account for the long infra default cluster names.
  • Fix the coloring bug for the right-hand-side borders.

Before:
2C1B1AED-33C6-498A-8F1C-FB5FE3C5E11D

After:
57CC7CFE-7FE9-47F7-A716-9291F141429C

defaultExposure = "loadbalancer"

// Pick a reasonable box width, suitable for infra default cluster names.
deploymentPlanBoxWidth = 90
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A default terminal is 80x25, so I guess it's going to be ugly by default. :-/
I guess there should be a way to determine current terminal width?
How about reducing the padding before and text of keys? e.g. k8s instead of kubernetes...

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