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

Improve default content function #40

Open
GaryJones opened this issue Feb 10, 2014 · 1 comment
Open

Improve default content function #40

GaryJones opened this issue Feb 10, 2014 · 1 comment

Comments

@GaryJones
Copy link
Member

While expand_content() can be dropped by developers in favour of a custom output, the default function should still be improved.

It is currently very focused towards how functions look, and not classes and hooks. For instance, the prototype doesn't make a lot of sense of classes or hooks.

The Arguments heading is output regardless of whether there are any argument or not.

The function name itself is also a little too generic - wpfuncref_content() might be better for instance.

@Rarst
Copy link
Contributor

Rarst commented Apr 20, 2014

This was refactored out into WP_Parser\the_content() template tag, filtering approach was just too full of a trouble.

I hadn't looked much into what it does though. DevHub theme currently doesn't use these bits actually.

@Rarst Rarst changed the title Improve default expand_content() Improve default content function Apr 20, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants