Add custom color support w/ color scheme presets about primer HOT 10 CLOSED

I handled most of this in the new Primer_Customizer class. @jonathanbardo is now working on splitting this up into logical parts so we don't have a 1000 line file.

from primer.

@chriswallace I have updated @fjarrett's original issue to reflect changes to make

from primer.

Definitely makes more sense being in a class. Originally it was just pulled out of Twenty Sixteen and was a bit of a mess

from primer.

I know this part was just merged from Twenty Sixteen but it makes no logical sense.

It's a callback for the custom header but changing the site-title and description

https://github.com/godaddy/wp-primer-theme/blob/issue-2-customizer/inc/customizer.php#L90-L115

from primer.

@jonathanbardo by the way sanitize_hex_color isn't loaded outside the customizer

https://github.com/godaddy/wp-primer-theme/blob/issue-2-customizer/inc/customizer/colors.php#L321

from primer.

@schrapel Thanks I did this, will take a look

from primer.

@fjarrett appreciate your work on this. Can you let me know when this stuff is in a position that we can integrate with the child themes? As of right now we are commenting out the customizer stuff since it fatals on the front end.

cc @chriswallace for ref

from primer.

@schrapel Oh really. I'm running 4.6-beta2 and there are no fatals. Looks like sanitize_hex_color is being moved outside of the Customizer https://core.trac.wordpress.org/ticket/27583. I will fix this for backwards compat now.

from primer.

That's a good example of something that should have been done 2 years ago when the ticket was opened haha. Thanks

from primer.

Fixed in 8712948

from primer.