Ticket #4830 (closed defect: fixed)

Opened 1 year ago

Last modified 1 year ago

Not all references to wp-admin.css get cache busted

Reported by: mdawaffe Assigned to: anonymous
Priority: normal Milestone: 2.3
Component: General Version:
Severity: normal Keywords: has-patch 2nd-opinion
Cc:

Description

Most everywhere, wp-admin.css is cache busted by appending a version parameter to the query string.

But not everywhere.

Additionally, we don't include rtl stylesheets in all places.

Attached:

  1. wp_admin_css_uri( $file = 'wp-admin' ) returns the absolute URL of the requested admin css file. $file is the name of the file (with path relative to /wp-admin) and without the .css extension.
  2. wp_admin_css( $file = 'wp-admin' ) echoes <link rel="stylesheet" /> markup for the requested admin css file and it's rtl equivalent if necessary.

Complete with filters.

Or we could just go through and make sure every css file is cache busted appropriately.

Attachments

4830.diff (9.6 kB) - added by mdawaffe on 08/28/07 00:51:20.

Change History

08/28/07 00:51:20 changed by mdawaffe

  • attachment 4830.diff added.

08/28/07 23:23:27 changed by ryan

  • milestone changed from 2.4 (next) to 2.3.

08/28/07 23:23:38 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

(In [5965]) Make css cache busting and rtl inclusion uniform across admin. Props mdawaffe. fixes #4830