-
Notifications
You must be signed in to change notification settings - Fork 148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Only load front end CSS when it is needed #1455
Conversation
This checks to make sure that a Coblock is being used before requiring the CSS to be loaded on the front end of the site. This is to address godaddy-wordpress#1454 In my basic tests this worked fine, however I'd like to see it get broader testing to make sure I haven't missed other conditions.
Good stuff, we'll run it though some testing. I'll hand this to you @jrtashjian. Should work find, though we'll need to make sure the assets load on index/archive views. |
@richtabor / @jrtashjian - any thoughts on when this could be reviewed and included in a new release? |
Hey @josephscott, I’m planning to review this during the current release cycle. If all looks/works well (which at first glance it does) then it’ll make it into the release next Thursday. |
Sounds good! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works great! I made a couple fixes around global usage and added automated tests.
Closes #1454 |
This checks to make sure that a Coblock is being used before requiring the CSS to be loaded on the front end of the site. This is to address #1454
In my basic tests this worked fine, however I'd like to see it get broader testing to make sure I haven't missed other conditions.