cabrerahector / wordpress-popular-posts

WordPress Popular Posts - A highly customizable WordPress widget that displays your most popular posts.
https://wordpress.org/plugins/wordpress-popular-posts/
GNU General Public License v2.0
279 stars 83 forks source link

List of links now in the H3 Tag - Change of CSS with the latest update? #291

Closed fintrader closed 3 years ago

fintrader commented 3 years ago

With the latest update of wordpress-popular-posts (5.3.4), we noticed a significant change in the styling.

Digging into it, it looks like the list of URLs is now in the h3 tag: image

Example here: https://graceandlightness.com/

I do not believe this was the case in the prior version (5.3.3). Is this a potential bug?

cabrerahector commented 3 years ago

Hi @Yoshi9143,

Is this a potential bug?

Possibly, yes, as there have been some changes to the way the HTML output is built now to enhance security that might affect some users (see 5.3.4 Release Notes) but since I can't reproduce that -the HTML is being rendered normally for me, otherwise I would have noticed it haha- I'll need more details.

For example:

koenige commented 3 years ago

It looks like the closing tag is not appended directly behind () if you set a value in “Title“. Instead, it is appended at the end of the

cabrerahector commented 3 years ago

I partially reverted some of the security enhancements changes just now, please let me know if that fixes the issue.

fintrader commented 3 years ago

Thank you for the quick fix! This does appear to fix the issue. Please let me know if I can provide you with any more details for reproducibility.

Here is the widget setting screenshot: image image

koenige commented 3 years ago

Thanks, Hector. Works like before. Thank you for the quick fix (especially on a Sunday!).

cabrerahector commented 3 years ago

Alright, so it was those changes then. How odd, I tested that many times using different combinations way before releasing version 5.3.4 -precisely thinking that it was a major change- and never had any issues with it.

I guess there's at least an edge use case not directly related to the widget (sidebar settings? something else?) that I didn't take into consideration that caused this mess.

I'll probably leave that alone, at least for now, as these security enhacements were meant to help clueless users who grant admin privileges to people who could misuse them, but I guess that's not really the plugin's responsibility.

Anyways, this comment became a rant haha. Thanks for reporting back guys, glad to know everything's back to normal.