Skip to content
This repository was archived by the owner on Jul 31, 2026. It is now read-only.

fix(cli-wizard): vertically center cli wizard steps - #327

Merged
bradcypert merged 3 commits into
mainfrom
bradcypert/vertically-center-cli-wizard-steps
Jan 9, 2026
Merged

fix(cli-wizard): vertically center cli wizard steps#327
bradcypert merged 3 commits into
mainfrom
bradcypert/vertically-center-cli-wizard-steps

Conversation

@bradcypert

Copy link
Copy Markdown
Contributor

Adjust the CLI Wizard component to handle cases where step values smaller than 3 are still centered. Currently, three step items are centered simply due to the fact that the steps are generally a specific height and the terminal simulator is also a specific height given the illusion of generally-centered steps. This is fine, but if we choose to deviate from three steps, there is no illusion of vertical centering and the component looks weird (specifically so with TWO steps).

Before:

Screenshot 2026-01-08 at 14-22-48 components _ __beta__ _ CLIWizard - Default ⋅ Storybook Screenshot 2026-01-08 at 14-27-35 components _ __beta__ _ CLIWizard - Default ⋅ Storybook

After:

Screenshot 2026-01-08 at 14-22-08 components _ __beta__ _ CLIWizard - Default ⋅ Storybook Screenshot 2026-01-08 at 14-27-15 components _ __beta__ _ CLIWizard - Default ⋅ Storybook

@danielkov danielkov left a comment

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.

fixed lint errors + API change from react-resizable-panels

@bradcypert
bradcypert merged commit b99546d into main Jan 9, 2026
9 checks passed
@bradcypert
bradcypert deleted the bradcypert/vertically-center-cli-wizard-steps branch January 9, 2026 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants