The Support Forums will be in read-only mode for a scheduled maintenance window on 01 September 2016 14:00 UTC - 20:00 UTC. More information.

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

  1. Colm Doyle
    Posted 3 years 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.



  2. Colm Doyle
    Posted 3 years 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
    Plugin Author

    Posted 3 years ago #

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

  4. Colm Doyle
    Posted 3 years ago #

    Nice, thanks!

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic


No tags yet.