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

Break class attributes at line width instead of using one class per line #33

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

elia
Copy link
Member

@elia elia commented Dec 1, 2023

Thanks to @Azeem838 for the pairing session on this feature 🙏

erb-formatter:elia+azeem/single-class-per-line ⤑ be exe/erb-format test/fixtures/multiline_attributes.html.erb                                                                                   ~/C/N/erb-formatter
<button
  class="
    text-gray-700 bg-transparent hover:bg-gray-50 active:bg-gray-100
    focus:bg-gray-50 focus:ring-gray-300 focus:ring-2 disabled:text-gray-300
    disabled:bg-transparent disabled:border-gray-300 disabled:cursor-not-allowed
    aria-disabled:text-gray-300 aria-disabled:bg-transparent
    aria-disabled:border-gray-300 aria-disabled:cursor-not-allowed
  "
>Button</button>
erb-formatter:elia+azeem/single-class-per-line ⤑ be exe/erb-format test/fixtures/multiline_attributes.html.erb --single-class-per-line                                                           ~/C/N/erb-formatter
<button
  class="
    text-gray-700
    bg-transparent
    hover:bg-gray-50
    active:bg-gray-100
    focus:bg-gray-50
    focus:ring-gray-300
    focus:ring-2
    disabled:text-gray-300
    disabled:bg-transparent
    disabled:border-gray-300
    disabled:cursor-not-allowed
    aria-disabled:text-gray-300
    aria-disabled:bg-transparent
    aria-disabled:border-gray-300
    aria-disabled:cursor-not-allowed
  "
>Button</button>
erb-formatter:elia+azeem/single-class-per-line ⤑                                                                                                                                                 ~/C/N/erb-formatter

@elia elia self-assigned this Dec 1, 2023
@elia elia force-pushed the elia+azeem/single-class-per-line branch from e51f356 to ca0c16e Compare December 1, 2023 10:22
@elia elia merged commit 4edff99 into main Dec 1, 2023
8 checks passed
@elia elia deleted the elia+azeem/single-class-per-line branch December 1, 2023 10:23
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.

1 participant