• Resolved nicart

    (@nicart)


    Hello,
    Thanks for your plugin, a must have when you use Disqus !
    But I have one problem : the widget don’t go in the layout in classic markup. (In HTML5 no style, don’t manage to apply style… don’t know why, but it’s inside the widget box!).
    You can check it here : https://www.vegactu.com/
    Or here [URL=https://imageshack.us/photo/my-images/36/2uhj.jpg/][IMG]https://img36.imageshack.us/img36/1368/2uhj.jpg[/IMG][/URL]

    Uploaded with [URL=https://imageshack.us]ImageShack.us[/URL]

    Thank you !
    ps : I use Sahifa theme

    https://www.ads-software.com/plugins/disqus-recent-comments-widget/

Viewing 10 replies - 1 through 10 (of 10 total)
  • Plugin Author Andrew Bartel

    (@andrew-bartel)

    Hey Nicart,

    Sahifa is a premium theme so unfortunately I can’t dig into the code and figure out what’s going on. Does this happen with the standard WordPress recent comments widget as well? Would it be possible for you to paste in the part of the theme code that is calling register_sidebar()?

    Thread Starter nicart

    (@nicart)

    Hello,
    This does not happen with the standard WordPress Recent comment widget.
    Here is the register_sidebar() in Sahifa :

    register_sidebar( array(
    ‘name’ => __( ‘Primary Widget Area’, ‘tie’ ),
    ‘id’ => ‘primary-widget-area’,
    ‘description’ => __( ‘The Primary widget area’, ‘tie’ ),
    ‘before_widget’ => $before_widget , ‘after_widget’ => $after_widget , ‘before_title’ => $before_title , ‘after_title’ => $after_title ,
    ) );

    Thank you Andrew

    Plugin Author Andrew Bartel

    (@andrew-bartel)

    Can you post the code that sets $before_widget, $after_widget, $before_title and $after_title please?

    Thread Starter nicart

    (@nicart)

    Hello Andrew, here it is

    add_action( ‘widgets_init’, ‘tie_widgets_init’ );
    function tie_widgets_init() {
    $before_widget = ‘<div id=”%1$s” class=”widget %2$s”>’;
    $after_widget = ‘</div></div><!– .widget /–>’;
    $before_title = ‘<div class=”widget-top”><h4>’;
    $after_title = ‘</h4><div class=”stripe-line”></div></div>
    <div class=”widget-container”>’;

    Thanks

    Plugin Author Andrew Bartel

    (@andrew-bartel)

    Hi Nicart,

    I’m going to ask the theme author for a development copy of the theme and see if I can track this down. Sorry for the delay, busy time of the year at the job. Thanks for your patience.

    Thread Starter nicart

    (@nicart)

    Thanks Andrew !
    good luck for the job ??

    Plugin Author Andrew Bartel

    (@andrew-bartel)

    Hi Nicart,

    Fouad was nice enough to give me a copy to test on. Can you switch to the html5 layout and try adding:

    .disqus_rcw_message {
        font-style: italic;
    }
    
    .disqus_rcw_single li {
        background-image: none;
    }
    
    .widget-container li {
        background-image: none;
    }

    into the global css in the theme settings? Let me know how that looks.

    Thread Starter nicart

    (@nicart)

    Hi,
    thanks !
    I had to change

    } elseif($style_params[‘markup_style’] == ‘html5’) {
    $recent_comments .= ‘<aside id=”disqus_rcw_title” class=”widget”>’;

    to

    } elseif($style_params[‘markup_style’] == ‘html5’) {
    $recent_comments .= ‘<div id=”disqus_rcw_title” class=”widget”>’;

    and line301 :

    if($style_params[‘markup_style’] == ‘html5’) $recent_comments .= ‘</aside>’;
    by
    if($style_params[‘markup_style’] == ‘html5’) $recent_comments .= ‘</div>’;

    Or there was a bug on the sidebar with next widget (they appear below the site).
    Thanks

    Plugin Author Andrew Bartel

    (@andrew-bartel)

    Hey Nicart, glad you got it working! Not sure why it would matter if it had an aside vs a div, but I’ll take a look before the next release. Thanks for your help in troubleshooting.

    fixed! thanks Nicart

Viewing 10 replies - 1 through 10 (of 10 total)
  • The topic ‘Layout fail’ is closed to new replies.