Skip to content
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

Theme style.css not propagating #1

Open
antonioOrtiz opened this issue Oct 22, 2015 · 1 comment
Open

Theme style.css not propagating #1

antonioOrtiz opened this issue Oct 22, 2015 · 1 comment

Comments

@antonioOrtiz
Copy link

Thanks for making a cool build tool.
It appears though, the style.css is not connecting to the index.php is there something I am missing? Do you have to configure that as well? As a background, I am trying to get started pretty quickly and just wanted to see what the theme was like. All I have changed was the proxy: property in the server task.

@bstaruk
Copy link

bstaruk commented Dec 2, 2015

I don't see anything in the theme files that supports adding theme information, so I made the following modifications:

  • Change the gulp.dest in the scss action as such:

From: .pipe( gulp.dest( '.' ) )
To: .pipe( gulp.dest( 'css' ) )

This changes the gulp scss process to put the styles into css/style.css instead of style.css. The purpose of this is to free up the root style.css so we can add theme information.

  • Change the style enqueue in functions.php to the new stylesheet as such:

From: wp_enqueue_style( 'sip-style', get_stylesheet_uri() );
To: wp_enqueue_style( 'sip-style', get_template_directory_uri() . '/css/style.css' );

Now, your site will use the new css/style.css stylesheet, and you can nuke the root style.css to your liking (delete all the old style, so all that remains will be your theme info).

@tomazzaman - I could submit a pull request for this, if you'd like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants