Skip to content

Commit 0dcbf28

Browse files
Build Tools: Consolidate CodeMirror CSS concatenation and minification.
Updates the CodeMirror CSS build process to perform concatenation and minification in a single `cssmin` task. This eliminates the need for an intermediate unminified file and ensures the license banner is applied correctly to the final minified bundle without an extra newline. Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
1 parent 404c7fa commit 0dcbf28

1 file changed

Lines changed: 17 additions & 31 deletions

File tree

Gruntfile.js

Lines changed: 17 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ module.exports = function(grunt) {
179179
},
180180
codemirror: {
181181
options: {
182-
linebreak: true,
182+
linebreak: false,
183183
banner: `/*! This file is auto-generated from CodeMirror - v${ pkg.dependencies.codemirror }\n` +
184184
`\n` +
185185
`CodeMirror, copyright (c) by Marijn Haverbeke and others\n` +
@@ -194,7 +194,7 @@ module.exports = function(grunt) {
194194
},
195195
files: {
196196
src: [
197-
WORKING_DIR + 'wp-includes/js/codemirror/codemirror.css'
197+
WORKING_DIR + 'wp-includes/js/codemirror/codemirror.min.css'
198198
]
199199
}
200200
},
@@ -605,13 +605,20 @@ module.exports = function(grunt) {
605605
compatibility: 'ie11'
606606
},
607607
codemirror: {
608-
expand: true,
609-
cwd: WORKING_DIR,
610-
dest: WORKING_DIR,
611-
ext: '.min.css',
612-
src: [
613-
'wp-includes/js/codemirror/codemirror.css',
614-
]
608+
files: {
609+
[ WORKING_DIR + 'wp-includes/js/codemirror/codemirror.min.css' ]: [
610+
'node_modules/codemirror/lib/codemirror.css',
611+
'node_modules/codemirror/addon/hint/show-hint.css',
612+
'node_modules/codemirror/addon/lint/lint.css',
613+
'node_modules/codemirror/addon/dialog/dialog.css',
614+
'node_modules/codemirror/addon/display/fullscreen.css',
615+
'node_modules/codemirror/addon/fold/foldgutter.css',
616+
'node_modules/codemirror/addon/merge/merge.css',
617+
'node_modules/codemirror/addon/scroll/simplescrollbars.css',
618+
'node_modules/codemirror/addon/search/matchesonscrollbar.css',
619+
'node_modules/codemirror/addon/tern/tern.css'
620+
]
621+
}
615622
},
616623
core: {
617624
expand: true,
@@ -976,26 +983,6 @@ module.exports = function(grunt) {
976983
codemirror: require( './tools/webpack/codemirror.config.js' ),
977984
},
978985
concat: {
979-
codemirror: {
980-
options: {
981-
process: function( src, filepath ) {
982-
return '/* Source: ' + filepath.replace( 'node_modules/', '' ) + '*/\n' + src;
983-
}
984-
},
985-
src: [
986-
'node_modules/codemirror/lib/codemirror.css',
987-
'node_modules/codemirror/addon/hint/show-hint.css',
988-
'node_modules/codemirror/addon/lint/lint.css',
989-
'node_modules/codemirror/addon/dialog/dialog.css',
990-
'node_modules/codemirror/addon/display/fullscreen.css',
991-
'node_modules/codemirror/addon/fold/foldgutter.css',
992-
'node_modules/codemirror/addon/merge/merge.css',
993-
'node_modules/codemirror/addon/scroll/simplescrollbars.css',
994-
'node_modules/codemirror/addon/search/matchesonscrollbar.css',
995-
'node_modules/codemirror/addon/tern/tern.css',
996-
],
997-
dest: WORKING_DIR + 'wp-includes/js/codemirror/codemirror.css'
998-
},
999986
tinymce: {
1000987
options: {
1001988
separator: '\n',
@@ -1726,9 +1713,8 @@ module.exports = function(grunt) {
17261713

17271714
grunt.registerTask( 'build:codemirror', [
17281715
'webpack:codemirror',
1729-
'concat:codemirror',
1730-
'usebanner:codemirror',
17311716
'cssmin:codemirror',
1717+
'usebanner:codemirror',
17321718
'copy:codemirror'
17331719
] );
17341720

0 commit comments

Comments
 (0)