<div dir="ltr">In general, *enhancing* core functionality is fine, while *circumventing* it is not. So, if the core function can't be modified as needed via parameters or filters, then one legitimate implementation is to provide a Theme option for an "enhanced" version of the function.*<div>
<br></div><div style>I don't think the default value for the option is of any particular importance.</div><div style><br></div><div style>* The best-of-both-worlds action would be to submit the improvement/enhancement to Trac as a patch to the core function/functionality. Any time a Theme developer finds a way to build upon/enhance a core function, I strongly recommend trying to push that enhancement back upstream to core.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 21, 2013 at 6:24 PM, Daniel Fenn <span dir="ltr"><<a href="mailto:danielx386@gmail.com" target="_blank">danielx386@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi there,<br>
<br>
In the guidelines it says that we are required to use<br>
paginate_comments_links (and other hooks), is it ok to include our own<br>
custon one (the paginate_comments_links(); would have been fine but<br>
the class needs to be changed and some extra html needs to go in) and<br>
then put a setting in under theme options allowing users to chose what<br>
they want to use?<br>
<br>
Same with paginate_links(); as well.<br>
<br>
And if the answer is yes, what must be the default setting?<br>
<br>
Regards,<br>
Daniel Fenn<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</blockquote></div><br></div>