onsale class – want to add a ribbon but needs DIV wrapper

You can use this filter woocommerce_sale_flash to change it to your preferred HTML.

Hello Roy. Thanks! Would you kindly elaborate? I am still learning the ways of the WordPress ninja.

I did note that both versions of sale-flash.php were applying the same filter “woocommerce_sale_flash”. However, since that PHP code contained the HTML output that I wanted to alter, I did not think it possible to change the filter external to the PHP file itself. Hence, I copied them from the aforementioned woocommerce dirs to my child theme:
\childtheme\woocommerce\loop\sale-flash.php
\childtheme\woocommerce\single-product\sale-flash.php
Which I learned about from articles such as this.

I then modified them as such, even renaming the SPAN’s original Classname, so that the theme’s CSS for “onsale” no longer applied:

Before:
echo apply_filters( 'woocommerce_sale_flash', '<span class="onsale">' . __( 'Sale!', 'woocommerce' ) . '</span>', $post, $product );

After:
echo apply_filters( 'woocommerce_sale_flash', '<div class="onsale-ribbon-wrapper"><span class="onsale-ribbon">' . __( 'Sale!', 'woocommerce' ) . '</span></div>', $post, $product );

If I understood you correctly, I am able to redefine the output portion of this filter .. which I can do externally (either modify my child’s functions.php .. or use a plugin such as “Code Snippets”) and, in effect, alter the 2nd argument of the filter as desired:

'<div class="onsale-ribbon-wrapper"><span class="onsale-ribbon">' . __( 'Sale!', 'woocommerce' ) . '</span></div>'

Please do elaborate!

add_filter( 'woocommerce_sale_flash', 'wpwc_change_on_sale' );

function wpwc_change_on_sale() {
     return '<div class="onsale-ribbon-wrapper"><span class="onsale-ribbon">Sale!</span></div>';
}

Indeed; that is SO much simpler! Thank you kindly for the shared wisdom.

Is there a way to retain the theme-defined value for “Sale!”, using your approach? I noted it doesn’t reference __( ‘Sale!’, ‘woocommerce’ )

Perhaps as such:

add_filter( 'woocommerce_sale_flash', 'wpwc_change_on_sale' );

function wpwc_change_on_sale() {
    return '<div class="onsale-ribbon-wrapper"><span class="onsale-ribbon">' . __( 'Sale!', 'woocommerce' ) . '</span></div>';
}

No use..that is WC’s namespace which will not apply even if you added that.

Understood. Since I am customising this functionality, it matters not for me to hardcode the desired value. Just asking, in case the custom wishes to alter the value thereafter.

Thanks, again!

then you would need to use a namespace within your plugin or theme depending on where you put it…