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

Integrate CodeMirror with linting #2

Open
1 of 4 tasks
helen opened this issue Dec 7, 2017 · 0 comments
Open
1 of 4 tasks

Integrate CodeMirror with linting #2

helen opened this issue Dec 7, 2017 · 0 comments
Labels
help wanted type:enhancement New feature or request.

Comments

@helen
Copy link
Contributor

helen commented Dec 7, 2017

WordPress core file editors now use CodeMirror with real-time linting and a pre-save "are you sure" checkbox, which allows users to save things with suspected errors if they really really want or need to. Ads.txt would be a great place to also use this functionality as well as UI.

  • Implement CodeMirror for the editor (Add codemirror, first pass #10)
  • Linting - parser and errors (if this can be easily kept in sync with PHP that would be best)
  • Pre-save AYS
  • Use more of core's built-in code editor wrappers as sensible (JS and PHP, e.g. wp_enqueue_code_editor()).

Does syntax highlighting make any sense? Seems like overkill, probably.

@helen helen added the type:enhancement New feature or request. label Feb 20, 2018
@jeffpaul jeffpaul added this to the Future Release milestone Apr 25, 2019
@jeffpaul jeffpaul modified the milestones: Future Release, 1.3.0 Jan 13, 2020
@peterwilsoncc peterwilsoncc modified the milestones: 1.4.0, Future Release Apr 14, 2022
@jeffpaul jeffpaul moved this from Incoming to Backlog in Open Source Practice Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted type:enhancement New feature or request.
Projects
Status: Backlog
Development

No branches or pull requests

3 participants