Skip to:

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#4455 closed enhancement (fixed)

Switch to min.js convention

Reported by: nacin's profile nacin Owned by:
Milestone: 1.7 Priority: normal
Severity: normal Version:
Component: Core Keywords:


Per #WP21633, core is likely to switch from .dev.js and .js, to .js and .min.js.

When this occurs, bumpbot will be updated to use the .min.js standard. So BuddyPress should probably follow suit.

Attachments (1)

4455.diff (14.9 KB) - added by nacin 11 years ago.

Download all attachments as: .zip

Change History (5)

#1 @DJPaul
11 years ago

  • Milestone changed from Awaiting Review to 1.7

#2 @nacin
11 years ago

Okay, this is ready for testing. First, a committer will need to run these commands:

svn mv bp-activity/admin/css/admin.css bp-activity/admin/css/admin.min.css; svn mv bp-activity/admin/css/ bp-activity/admin/css/admin.css; svn mv bp-core/admin/css/common.css bp-core/admin/css/common.min.css; svn mv bp-core/admin/css/ bp-core/admin/css/common.css; svn mv bp-core/admin/css/wizard.css bp-core/admin/css/wizard.min.css; svn mv bp-core/admin/css/ bp-core/admin/css/wizard.css; svn mv bp-core/css/admin-bar-rtl.css bp-core/css/admin-bar-rtl.min.css; svn mv bp-core/css/ bp-core/css/admin-bar-rtl.css; svn mv bp-core/css/admin-bar.css bp-core/css/admin-bar.min.css; svn mv bp-core/css/ bp-core/css/admin-bar.css; svn mv bp-core/css/buddybar-rtl.css bp-core/css/buddybar-rtl.min.css; svn mv bp-core/css/ bp-core/css/buddybar-rtl.css; svn mv bp-core/css/buddybar.css bp-core/css/buddybar.min.css; svn mv bp-core/css/ bp-core/css/buddybar.css; svn mv bp-messages/css/autocomplete/jquery.autocompletefb.css bp-messages/css/autocomplete/jquery.autocompletefb.min.css; svn mv bp-messages/css/autocomplete/ bp-messages/css/autocomplete/jquery.autocompletefb.css; svn mv bp-xprofile/admin/css/admin.css bp-xprofile/admin/css/admin.min.css; svn mv bp-xprofile/admin/css/ bp-xprofile/admin/css/admin.css; svn mv bp-activity/admin/js/admin.js bp-activity/admin/js/admin.min.js; svn mv bp-activity/admin/js/ bp-activity/admin/js/admin.js; svn mv bp-core/admin/js/wizard.js bp-core/admin/js/wizard.min.js; svn mv bp-core/admin/js/ bp-core/admin/js/wizard.js; svn mv bp-core/js/widget-members.js bp-core/js/widget-members.min.js; svn mv bp-core/js/ bp-core/js/widget-members.js; svn mv bp-groups/js/widget-groups.js bp-groups/js/widget-groups.min.js; svn mv bp-groups/js/ bp-groups/js/widget-groups.js; svn mv bp-messages/js/autocomplete/jquery.autocomplete.js bp-messages/js/autocomplete/jquery.autocomplete.min.js; svn mv bp-messages/js/autocomplete/ bp-messages/js/autocomplete/jquery.autocomplete.js; svn mv bp-messages/js/autocomplete/jquery.autocompletefb.js bp-messages/js/autocomplete/jquery.autocompletefb.min.js; svn mv bp-messages/js/autocomplete/ bp-messages/js/autocomplete/jquery.autocompletefb.js; svn mv bp-messages/js/autocomplete/jquery.bgiframe.js bp-messages/js/autocomplete/jquery.bgiframe.min.js; svn mv bp-messages/js/autocomplete/ bp-messages/js/autocomplete/jquery.bgiframe.js; svn mv bp-messages/js/autocomplete/jquery.dimensions.js bp-messages/js/autocomplete/jquery.dimensions.min.js; svn mv bp-messages/js/autocomplete/ bp-messages/js/autocomplete/jquery.dimensions.js; svn mv bp-xprofile/admin/js/admin.js bp-xprofile/admin/js/admin.min.js; svn mv bp-xprofile/admin/js/ bp-xprofile/admin/js/admin.js

Then, apply the attached patch.

11 years ago

#3 @johnjamesjacoby
11 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [6264]) Switch to .min for compressed JS and CSS files.

  • This moves our "development" versions from .dev.js to .js (same for css).
  • The compressed version then moves from .js to .min.js (same for css).
  • By switching to the standard .min convention, it sets expectations for developers, and works nicely with existing tools such as ack.
  • Fixes #4455.
  • Props nacin.

#4 @johnjamesjacoby
11 years ago

(In [6265]) Some deleted files were missed in r6264. See #4455.

Note: See TracTickets for help on using tickets.