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

Add support for theme customizer #1573

Merged
merged 1 commit into from
Nov 9, 2015
Merged

Add support for theme customizer #1573

merged 1 commit into from
Nov 9, 2015

Conversation

retlehs
Copy link
Sponsor Member

@retlehs retlehs commented Nov 6, 2015

two new files:

  • lib/customizer.php — add postmessage support and enqueue customizer js
  • assets/scripts/customizer.js — js for the theme customizer. currently changes the site name in the header when it's updated from the customizer

@JulienMelissas
Copy link
Sponsor Contributor

i'm weird

@retlehs
Copy link
Sponsor Member Author

retlehs commented Nov 6, 2015

what this PR does:

sage-customizer

use it as a start for getting your theme customizer setup how you want. here's a quick example of something i'm working on — using the customizer to change up some colors:

customizer-carrot

@ciromattia
Copy link
Contributor

+1
Great, I use customizer for all the site-wide options and at the moment I'm putting the whole bunch of code into setup.php.
Is the JS really needed?

@retlehs
Copy link
Sponsor Member Author

retlehs commented Nov 9, 2015

@ciromattia yep, the JS is needed to make use of the real time preview

retlehs added a commit that referenced this pull request Nov 9, 2015
@retlehs retlehs merged commit 32dddb0 into master Nov 9, 2015
@retlehs retlehs deleted the add-customizer-support branch November 9, 2015 19:35
@jo4nny8
Copy link

jo4nny8 commented Jun 3, 2017

Does anyone have a working example of being able to change the theme colours using the customizer?

I've been trying to achieve it with leafo scssphp compiler but I'm struggling.

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

Successfully merging this pull request may close these issues.

7 participants