Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Better handling for long config names #60

Merged
merged 2 commits into from
Jun 11, 2024

Conversation

clemyan
Copy link
Contributor

@clemyan clemyan commented Jun 8, 2024

The config inspector does not handle long config names particularly well when there isn't enough screen width

Configs page

In the configs page, the config name breaks in weird ways and, more importantly, misaligns the stats.

image

This PR makes it so that when there isn't enough screen width, the config name breaks first. Then if there still isn't enough, the stats wraps.

image

image

Files page

In the files page, the config name breaks but is centered.

image

The PR makes the config name aligned

image

@eslint-github-bot
Copy link

Hi @clemyan!, thanks for the Pull Request

The pull request title isn't properly formatted. We ask that you update the pull request title to match this format, as we use it to generate changelogs and automate releases.

  • The commit message tag wasn't recognized. Did you mean "docs", "fix", or "feat"?
  • There should be a space following the initial tag and colon, for example 'feat: Message'.
  • The first letter of the tag should be in lowercase

To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page.

Read more about contributing to ESLint here

Copy link

linux-foundation-easycla bot commented Jun 8, 2024

CLA Signed


The committers listed above are authorized under a signed CLA.

@clemyan clemyan changed the title Better handling for long config names fix: Better handling for long config names Jun 8, 2024
@eslint-github-bot eslint-github-bot bot added the bug Something isn't working label Jun 8, 2024
@antfu antfu merged commit 055a667 into eslint:main Jun 11, 2024
7 checks passed
@github-actions github-actions bot mentioned this pull request Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants