Customizer custom colors preview overriden by style.css about newspack-theme

[laurelfulford]

This might be related to #52 -- the issue hangs around even after you've switched back to the default style pack, as long as it's been changed at some point in the past -- it's due to my clunky code for the Customizer.

@claudiulodro do you remember if you switched the style pack at any point on this test site?

[claudiulodro]

Not sure. I should be getting my new computer today, so I'll give it a test on a clean slate and see.

[laurelfulford]

Thanks! (And yay new computer!! 🎉 )

[claudiulodro]

Just tested on the cleanest of new sites, and the issue still happens. It's not because I had a style pack and then removed it.

[laurelfulford]

Thanks for double-checking!

Dang, this could be a bigger issue than I thought.

[claudiulodro]

Did a little more digging into this, and it seems to def. be related to the style packs. If I remove add_action( 'customize_preview_init', array( $this, 'customize_preview' ) ); from Newspack_Style_Packs_Core the customizer colors work great.

The underlying issue to me looks like style.css is getting enqueued twice:

• Once normally
• Once by the style pack JS

The second enqueue (the one by style pack) overrides the customizer styles, since the styles all have the same specificity but the style pack loads style.css after the customizer styles have been applied.

Closing this issue since it seems to be the same as #52. Hope this info helps! :)

[laurelfulford]

Yes -- thanks @claudiulodro!!