Layout fail

Resolved nicart
(@nicart)

12 years, 7 months ago

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 : http://www.vegactu.com/
Or here [URL=http://imageshack.us/photo/my-images/36/2uhj.jpg/][IMG]http://img36.imageshack.us/img36/1368/2uhj.jpg[/IMG][/URL]

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

Thank you !
ps : I use Sahifa theme

http://ww.wp.xz.cn/plugins/disqus-recent-comments-widget/

Viewing 10 replies - 1 through 10 (of 10 total)

Plugin Author Andrew Bartel
(@andrew-bartel)

12 years, 7 months ago

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)

12 years, 7 months ago

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)

12 years, 7 months ago

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

Thread Starter nicart
(@nicart)

12 years, 7 months ago

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)

12 years, 6 months ago

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)

12 years, 6 months ago

Thanks Andrew !
good luck for the job 🙂
Plugin Author Andrew Bartel
(@andrew-bartel)

12 years, 6 months ago
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)

12 years, 6 months ago

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)

12 years, 6 months ago

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.

kokocipher
(@kokocipher)

12 years, 5 months ago

fixed! thanks Nicart

Viewing 10 replies - 1 through 10 (of 10 total)

The topic ‘Layout fail’ is closed to new replies.