Skip to content

format cli for better ui#4693

Closed
stefanbinoj wants to merge 7 commits into
webpack:mainfrom
stefanbinoj:refactor/format-cli
Closed

format cli for better ui#4693
stefanbinoj wants to merge 7 commits into
webpack:mainfrom
stefanbinoj:refactor/format-cli

Conversation

@stefanbinoj

@stefanbinoj stefanbinoj commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

ref #1481 (comment)

Before (help) After (help)
Screenshot 2026-03-09 at 8 01 36 PM Screenshot 2026-03-10 at 8 12 31 AM
Before (bulild help) After (build help)
Screenshot 2026-03-09 at 8 01 51 PM Screenshot 2026-03-10 at 8 12 48 AM

Video (for different width):

Screen.Recording.2026-03-10.at.8.14.01.AM.mov

Summary

While going thorough major PR's found this one not sure if it is still relevant. Added a nice formatting as in the comment image

What kind of change does this PR introduce?

refactor: improve cli ui

Did you add tests for your changes?

yes added test for making sure that the header banner is placed in center irrespective of width of terminal

Does this PR introduce a breaking change?

NO

If relevant, what needs to be documented once your changes are merged or what have you already documented?

NO

Use of AI

Was used for better understanding of codebase and for a initial draft.

Tests

Screenshot 2026-03-09 at 7 57 01 PM

@changeset-bot

changeset-bot Bot commented Mar 9, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6c97df2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@stefanbinoj

Copy link
Copy Markdown
Contributor Author

Hi @alexander-akait could you take a look at this and tell if its relevant and a ui improvement?

@stefanbinoj stefanbinoj changed the title refactor: format cli for better ui format cli for better ui Mar 9, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

already present in header so removved

@evenstensberg

Copy link
Copy Markdown
Member

Looks good, will review later

@alexander-akait

Copy link
Copy Markdown
Member

Sorry, we don't need to make it, centering takes up a lot of space and makes it look fashionable rather than functional, I am fine to improve colors in our help, but don't want to waste space for fashion, if you really using terminal in your IDE you will find what this outputs take most of place

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.

3 participants