Title wrongly rendered

  • Resolved sierramike

    (@sierramike)


    9 years, 6 months ago

    Hello,

    The title of the widget is not properly placed in the html markup so it doesn’t get redered correctly depending on themes.

    It should be put immediately after the $before_widget.

    In sfsi_widget.php, lines 20-29 should be this way:
    echo $before_widget;
    /* Display the widget title */
    if ( $title ) echo $before_title . $title . $after_title;
    ?>
    <div class=”sfsi_widget” data-position=”widget”>
    <div id=’sfsi_wDiv’></div>
    <?php
    /* Link the main icons function */
    echo sfsi_check_visiblity(0);
    ?>

    https://www.ads-software.com/plugins/ultimate-social-media-icons/

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author socialdude

    (@socialdude)

    Thanks Mike, we’ll have a look. Rgds, John

    Plugin Author socialdude

    (@socialdude)

    Hi Mike, we looked at it, but it doesn’t really make sense to us. Could you please explain more?

    Thread Starter sierramike

    (@sierramike)

    Look at my blog : https://mmm16.ztb.fr
    On the right sidebar, your plugin is the first widget. It doesn’t render the title correcty like all other widgets, because the <h3> tag is nested into a <div> one level too low.

    If you look at the source code of the page, the html code of the rendered widget is :
    <div id=”sfsi-widget-2″ class=”widget sfsi”>
    <div class=”sfsi_widget” data-position=”widget”>
    <div id=’sfsi_wDiv’></div>
    <h3>Restez connectés / Stay in touch</h3>
    <div class=”norm_row sfsi_wDiv” style=”width:225px;text-align:left;position:absolute;”>

    Where the next standard wordpress widget renders like this :
    <div id=”recent-posts-2″ class=”widget widget_recent_entries”>
    <h3>Articles récents</h3>

    • As you see, the <h3> tag is immediately at the start of the widget, in the <div> of the widget.
      If you take my correction, it will render correctly.
      Cheers

    Plugin Author socialdude

    (@socialdude)

    Fixed, will be included in new plugin we’ll upload later this week. If issues remain please let me know.

    Thread Starter sierramike

    (@sierramike)

    Thanks for your help, sure I’ll let you know as soon as I update!

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘Title wrongly rendered’ is closed to new replies.