Skip to content
This repository has been archived by the owner on Sep 11, 2020. It is now read-only.

Add Hook Customization to Customizer #85

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Add Hook Customization to Customizer #85

wants to merge 6 commits into from

Conversation

DangitRick
Copy link
Contributor

This update allows users to add custom HTML to their site directly in the WordPress Customizer panel. Very powerful, very useful!

(Ideally, users should be able to add PHP code to their customizations too; the code is present in this pull request and works like a treat. However, it depends upon exec() which is prohibited by theme coding guidelines.)

Boom.

This is probably buggy as hell. Only highest level admin users can
access this — it provides SO MUCH POWER in customizing a site,
including allowing custom PHP!
Fixing WP coding standards
Disable processing of PHP on hooks as WordPress coding standards
prohibit `exec()` in themes so that the theme remains compatible and
safe for `wordpress.com` usage.

Dang.
@ineagu
Copy link
Contributor

ineagu commented Mar 18, 2015

Hey @BrazenlyGeek this sounds cool and useful however from my point of view is more or less plugin space, there are a tons of plugins for custom html/php.

However just to let you know some news :)

  • we acquired flat from the previous owner and since we do have our own process ( like different repo structure, not using bower/travis or grunt yet ) so for the moment we plan to keep both this repo and to create our own where we will continue the dev in our way ( not the best :D ).

I will try to keep an eye here too, also we are not used to contribution type of repo and I know is bad however for now we just wanna quickly change ownership and then figure it out :)

The new repo is : https://github.com/Codeinwp/FlatX and you can see has a more straightforward structure .

* Name change
* Dependencies brought up to date
* Minor whitespace cleanup
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants