Thread Starter
madri2
(@madri2)
also, why did you changed the post-stats class to block ?
Hi madri2,
Using $this when not in object context in /plugins/wordpress-popular-posts/wordpress-popular-posts.php on line 2939
Thanks for the notice. I’ll get on it right away.
Edit: Fixed it already, but won’t push an update just yet. I’m going to wait a little while to make sure there are no other bugs present (hopefully, there should be none).
The fix is quite simple, actually. Replacing !$this->__is_numeric($id) with !is_numeric($id) will do the job.
Why did you changed the post-stats class to block ?
Because I believe that the statistics should go on a new line. Looks better than aligning them with the post title, IMHO.
Thread Starter
madri2
(@madri2)
thanks for the fix.
In my opinion, since it wasn’t “block” until now, I was accustomed to it to be in the same line. Now it looks strange to me.
Well, you can always set it to inline to regain the former look & feel.
To do so, follow these instructions:
- Copy the wpp.css file (located at wp-content/plugins/wordpress-popular-posts/styles) into your theme’s folder, so that future upgrades won’t affect any changes made to WPP’s stylesheet.
- Open wpp.css using a text editor and find:
/* Stats tag styles */
.post-stats {
display:block;
font-size:9px;
font-weight:bold;
}
..
. and change it to:
/* Stats tag styles */
.post-stats {
/*display:block;*/
display:inline;
/* you might want to comment out the lines below, too */
font-size:9px;
font-weight:bold;
}
- Save changes.
Actually, this is the reason why I made that small change to the stylesheet. I figured the old one needed a change, so I did it hehe.
Thread Starter
madri2
(@madri2)
are you sure putting the wpp.css modified inside the /wp-content/themes/mytheme/ directory works ?
for me it looks like it’s not working.
I’m in a multisite env.