WordPress.org

Make WordPress Core

Opened 7 years ago

Last modified 2 years ago

#19742 new enhancement

WP_Styles should pass more info to the filter

Reported by: ptahdunbar Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Script Loader Keywords: has-patch needs-refresh close
Focuses: Cc:

Description

Currently, if your working with wp_enqueue_style(), it's not easy to customize the output of the link element.

The current filter passes the full html along with the style handle. This isn't enough information if you're trying to rebuild the link element.

I'm currently changing rel="stylesheet" to rel="stylesheet/less" for lesscss suport. I'm currently able to achieve this, but I have to know the style handle.

This patch adds a third param which contains all of the style data available for that style.

Attachments (1)

ticket.19742.diff (1.2 KB) - added by ptahdunbar 7 years ago.

Download all attachments as: .zip

Change History (6)

#1 @kovshenin
6 years ago

Hey, just stumbled across this. You have the handle which is a unique ID, and you can get all the info you need by calling $wp_styles->query() or ->get_data() if you need the extras. Does that solve your problem?

Would it be much easier if the rel attribute was filterable, say style_loader_rel or maybe even style_loader_rel-$handle so you could:

add_filter( 'style_loader_rel-my-script', function() { return 'stylesheet/less'; } );
wp_enqueue_script( 'my-script', ... );

Thoughts?

#2 @bpetty
6 years ago

Related: #22249

#3 @nacin
5 years ago

  • Component changed from General to Script Loader

#4 @chriscct7
4 years ago

  • Keywords needs-refresh added

#5 @swissspidy
2 years ago

  • Keywords close added
Note: See TracTickets for help on using tickets.