Opened 10 years ago
Closed 10 years ago
#31912 closed defect (bug) (fixed)
Media splitting: don't copy the media folder to the build directory
Reported by: | iseulde | Owned by: | wonderboymusic |
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | normal | Version: | 4.2 |
Component: | Build/Test Tools | Keywords: | has-patch |
Focuses: | javascript | Cc: |
Description
These are not used when SCRIPT_DEBUG is on.
Attachments (2)
Change History (10)
#2
@
10 years ago
- Component changed from Media to Build/Test Tools
- Focuses javascript added
Moving this ticket also to the Build Tools component as I did for #31911, these tickets are more about how we build and compile assets rather than the media component functionality :)
This ticket was mentioned in Slack in #core by drew. View the logs.
10 years ago
#7
@
10 years ago
We won't delete the files, rather copy/move the existing ones so the history remains. Based on the patch this seems to be the follow changes:
- /src/wp-includes/js/media/audio-video.manifest.js => /src/wp-includes/js/media/audiovideo.manifest.js
- /src/wp-includes/js/media/audio-video.js => /src/wp-includes/js/media-audiovideo.js
- /src/wp-includes/js/media/grid.js => /src/wp-includes/js/media-grid.js
- /src/wp-includes/js/media/models.js => /src/wp-includes/js/media-models.js
- /src/wp-includes/js/media/views.js => /src/wp-includes/js/media-views.js
31912.2.patch includes only the changes for the build task and the script loader.
Note: See
TracTickets for help on using
tickets.
Maybe better to keep the old file structure? Not sure. Makes the tasks a bit simpler as well.