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

Use Atom Packages Config Settings as default options #22

Closed
Glavin001 opened this issue Jun 14, 2014 · 2 comments · Fixed by #30
Closed

Use Atom Packages Config Settings as default options #22

Glavin001 opened this issue Jun 14, 2014 · 2 comments · Fixed by #30

Comments

@Glavin001
Copy link
Owner

See https://github.com/taronfoxworth/atom-beautifier/blob/master/lib/beautifier.coffee#L70-L94

plugin.configDefaults =
  autoSave: false
  JsIndentSize: 2
  JsIndentChar: " "
  JsIndentLevel: 0
  JsIndentWithTabs: false
  JsPreserveNewlines: true
  JsMaxPreserveNewlines: 10
  JsJslintHappy: false
  JsBraceStyle: "collapse"
  JsKeepArrayIndentation: false
  JsKeepFunctionIndentation: false
  JsSpaceBeforeConditional: true
  JsBreakChainedMethods: false
  JsEvalCode: false
  JsUnescapeStrings: false
  JsWrapLineLength: 0
  CssIndentSize: 2
  CssIndentChar: " "
  HtmlIndentInnerHtml: false
  HtmlIndentSize: 2
  HtmlIndentChar: " "
  HtmlBraceStyle: "collapse"
  HtmlIndentScripts: "normal"
  HtmlWrapLineLength: 250

I know I liked atom-beautifier a lot more because I could easily edit settings in the Atom package settings panel. And so did this user: anaptfox/atom-beautifier#3 (comment)

@karolyi , thoughts?

@karolyi
Copy link
Collaborator

karolyi commented Jun 14, 2014

If you'd like to implement it, just go on :) Every feature makes the product better 👍

@Glavin001
Copy link
Owner Author

Implemented all of the above from atom-beautifier.

image

@Glavin001 Glavin001 mentioned this issue Jun 15, 2014
4 tasks
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 a pull request may close this issue.

2 participants