Forum at me and my mac » AdRotate

Suggestion for AdRotate (Widget Title html tags still show when empty)

(4 posts)

Tags:

  1. topsys
    Member

    I have a small suggestion for AdRotate.

    If you leave the title of a widget empty, the outputted code will still display the heading tags, which, when empty, might not be desirable.

    Therefore I propose that future releases change the following lines of code 20 and 99 to be changed for the following,

    from
    echo $before_widget . $before_title . $options['title'] . $after_title;

    to
    echo $before_widget;
    if(!empty($options['title'])) {echo $before_title . $options['title'] . $after_title;}

    I can also see that others have talked about unlimited number of widgets, I will see if I can figure out how to add this, and if so, share the code in this forum, for those who might wish to add that hack to the plugin.

    Cheers,

    Olaf

    Posted 1 month ago #
  2. topsys
    Member

    oh sorry I forgot to write that the file which need the lines changed above is adrotate-widget.php in the adrotate plugin folder.

    Lines 20 and 99.

    sorry about that.

    Posted 1 month ago #
  3. jkirkwood
    Member

    That's a +1 from me - I have just hacked the same hack simultaneously.
    (I haven't tested your unlimited widgets, but I plan to soon).

    Posted 1 month ago #
  4. Arnan
    Kung-fu Jedi

    I didn't see this thread earlier, but it is a nice thing to include i think. I'll look into it and add it to the next version.

    Woohoo! Check out http://meandmymac.net
    Get my RSS feed! http://meandmymac.net/feed/
    Posted 1 month ago #

RSS feed for this topic

Reply

You must log in to post.