WordPress.org

Ready to get started?Download WordPress

Forums

Content Headings Tree
[resolved] Make title optional (4 posts)

  1. Colm Doyle
    Member
    Posted 1 year ago #

    Loving the plugin.

    Two suggestions based on your latest upload.

    Having a title as an option is nice, but if the site chooses to not have one, you're currently rendering an empty div, which leaves some awkward looking whitespace. It might be no harm to check the inputted title before rendering?

    This is what I've hacked in - https://gist.github.com/4142002.

    Also, and this could just be my theme, but your $before_widget at the very start of widget.php is doing this crazy nested widget thing. Removing that echo statement seems to clear it, but just thought I'd give you a heads up.

    http://cl.ly/image/3P0k2Y0j3W3W

    http://wordpress.org/extend/plugins/content-headings-tree/

  2. Colm Doyle
    Member
    Posted 1 year ago #

    Ah, I think I've figured out the weird nesting thing anyway.

    You're calling $before_widget and $after_widget in the widget function of plugin.php and inside views/widget.php.

    removing one of the call sites solves it.

  3. Konstantinos Kouratoras
    Member
    Plugin Author

    Posted 1 year ago #

    Thank you very much for your feedback! Both issues solved and released an update of my plugin. Go get it! :)

  4. Colm Doyle
    Member
    Posted 1 year ago #

    Nice, thanks!

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic

Tags

No tags yet.