[wp-trac] [WordPress Trac] #41870: Code Editor: Add grunt task for building new CodeMirror bundles from external dependencies

WordPress Trac noreply at wordpress.org
Wed Sep 13 18:21:38 UTC 2017


#41870: Code Editor: Add grunt task for building new CodeMirror bundles from
external dependencies
--------------------------------+------------------
 Reporter:  westonruter         |       Owner:
     Type:  enhancement         |      Status:  new
 Priority:  normal              |   Milestone:  4.9
Component:  External Libraries  |     Version:
 Severity:  normal              |  Resolution:
 Keywords:  needs-patch         |     Focuses:
--------------------------------+------------------
Description changed by westonruter:

Old description:

> After [41376] for #12423, it would be ideal if the CodeMirror bundling
> logic (via browserify) could be added as part of core. In this way, the
> external dependencies (CodeMirror and the linter linraries) can be
> managed by NPM. For now, just the minified bundles are being included.
>
> The logic used in the Better Code Editing plugin for creating the bundles
> can be seen here: https://github.com/WordPress/better-code-
> editing/pull/92/files
>
> Depends on #40894 which replaces Browserify with Webpack in core.
>
> Additionally, the linters are currently being included as separate
> assets. They are not being minified, other than they are already minified
> as distributed. These could either benefit from additional uglification
> or bundling with codemirror if they are not going to be enqueued
> separately.

New description:

 After [41376] for #12423, it would be ideal if the CodeMirror bundling
 logic (via browserify) could be added as part of core. In this way, the
 external dependencies (CodeMirror and the linter linraries) can be managed
 by NPM. For now, just the minified bundles are being included.

 The logic used in the Better Code Editing plugin for creating the bundles
 can be seen here: https://github.com/WordPress/better-code-
 editing/pull/92/files

 Depends on #40894 which replaces Browserify with Webpack in core.

 Additionally, the linters are currently being included as separate assets.
 They are not being minified, other than they are already minified as
 distributed. These could either benefit from additional uglification or
 bundling with codemirror if they are not going to be enqueued separately.
 The same goes for the `htmlhint-kses` script.

--

--
Ticket URL: <https://core.trac.wordpress.org/ticket/41870#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list