Skip to content

Conversation

@laundmo
Copy link

@laundmo laundmo commented Jan 31, 2025

Description

With this config:

break_chained_methods = true
wrap_line_length = 80

for method chains, it formatted like this:

document.getElementById('some_id')
  .appendChild(document.getElementById('child_to_append')
    .content.cloneNode(true));

Now, after this change, it will format method chains like this, breaking on each dot instead of only some:

document
  .getElementById('some_id')
  .appendChild(document
    .getElementById('child_to_append')
    .content
    .cloneNode(true));

For various reasons i dont have the time, ability, or desire to fully get this PR to completion and merged (along with all the below checks). Im perfectly fine if anyone else wants to finish, rework, or expand it, please, go ahead.

  • Source branch in your fork has meaningful name (not main)

Before Merge Checklist

These items can be completed after PR is created.

(Check any items that are not applicable (NA) for this PR)

  • JavaScript implementation
  • Python implementation (NA if HTML beautifier)
  • Added Tests to data file(s)
  • Added command-line option(s) (NA if
  • README.md documents new feature/option(s)

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