[wp-trac] [WordPress Trac] #25947: jshint shouldn't throw errors: wp-admin/js/editor.js
WordPress Trac
noreply at wordpress.org
Fri Nov 15 20:29:31 UTC 2013
#25947: jshint shouldn't throw errors: wp-admin/js/editor.js
--------------------------+------------------------------
Reporter: dougwollison | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Build Tools | Version:
Severity: normal | Resolution:
Keywords: has-patch |
--------------------------+------------------------------
Comment (by azaozz):
Some more notes:
- Don't think defining the globals for JSHint needs the `:false` part.
This is the [http://www.jshint.com/docs/ default].
- The globals should probably be comma separated on the same line,
following the same standard as `var`.
- `tinymce` is a superset of `tinyMCE` (tinyMCE exists for back-compat),
but `tinyMCEPreInit` is another global that cannot be changed. Please run
some tests with the patched file to confirm all is still working.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/25947#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list