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

Better behavior for javascript --wrap-line-length #284

Closed
dnarvaez opened this issue Jun 8, 2013 · 3 comments
Closed

Better behavior for javascript --wrap-line-length #284

dnarvaez opened this issue Jun 8, 2013 · 3 comments

Comments

@dnarvaez
Copy link

dnarvaez commented Jun 8, 2013

The documentation of javascript --wrap-line-length says:

"Wrap lines at next opportunity after N characters"

And the code behaves like that. Though I think that's not usually what you want. Most projects code style requires a maximum line length which has to be respected exactly.

(I also noticed that css/html behave as I expect, according to the docs at least).

@bitwiseman
Copy link
Member

Thanks for filing this. This is an interesting problem, which I hope someone will fix. And I'm pretty sure the docs are incorrect regarding css/html. 😄 They are all in the same boat, best effort.

In the meanwhile, as a workaround set your wrap length slightly shorter than your guidelines require. Also when #286 goes in this will be less of an issue as, due to the later removal of redundant indents, the js beautifier will tend to appear to wrap too early rather than too late.

@schickling
Copy link

👍

@mietek
Copy link

mietek commented Feb 2, 2016

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants