Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Refactor JSLint into an extension #3094

Closed
DennisKehrig opened this issue Mar 11, 2013 · 1 comment
Closed

Refactor JSLint into an extension #3094

DennisKehrig opened this issue Mar 11, 2013 · 1 comment

Comments

@DennisKehrig
Copy link
Contributor

Places in the code referring to JSLint

  • base-config/keyboard.json
    • Defines keyboard shortcuts for navigate.gotoJSLintError
  • brackets.config.json
    • Sets config.enable_jslint to true
  • brackets.js
    • Loads JSLintUtils - only needed for the menu entries and the point below
    • Exports JSLintUtils for tests - affects Performance-test.js
  • command/Commands.js
    • Defines Commands.TOGGLE_JSLINT
    • Defines Commands.NAVIGATE_GOTO_JSLINT_ERROR
  • command/DefaultMenus.js
    • Adds menu item for Commands.TOGGLE_JSLINT
    • Adds menu item for Commands.NAVIGATE_GOTO_JSLINT_ERROR
  • config.json - automatically generated by tasks/write-config.js
    • Sets config.enable_jslint to true (inherited from brackets.config.json)
  • htmlContent/main-view.html
    • Adds toolbar jslint-results
  • language/JSLintUtils
    • Main code for JSLint support
  • nls/*/strings.js
    • Defines various translations for JSLint related strings - can probably stay there
  • search/QuickOpen.js
    • References JSLintUtils in two comments
  • styles/brackets.less
    • Defines the CSS classes jslint-disabled, jslint-errors and jslint-valid
  • thirdparty/jslint/
    • JSLint library
  • widgets/StatusBar.html
    • Adds the gold star for JSLint's status - should be possible to initially movie this elsewhere (e.g. JSLintUtils) since StatusBar.addIndicator is called for this already
@jasonsanjose
Copy link
Member

Marking move to backlog

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

No branches or pull requests

2 participants