suffusion_set_more_link filter breaks validation

Reports about issues that you encounter in Suffusion
Forum rules
1. No offensive language and no mocking
2. Please do a thorough search before you post something. Trust us, there is a high probability that the question you are asking has been asked previously.
3. No soliciting. You cannot post here soliciting bids for people offer you quotes, or even offer money to people for some work. You will be moderated if you do so. If you are looking for help, please post your request on http://jobs.wordpress.net or http://codepoet.com
4. Please be reasonable. You are getting software and support. For free. Complicated requests from a general purpose theme are not welcome and some volunteers might lose patience with you.
5. Please do your due diligence. If you posted a query and we answered with a link, take the trouble to go through the link contents.
6. Please post with complete information. Requests for help MUST be accompanied with your URL, particularly if you are asking something like "Why am I seeing a blank space?"

suffusion_set_more_link filter breaks validation

Postby metal450 » 13 Feb 2010, 14:42

The filter suffusion_set_more_link wraps the Read More text in a div. This works fine if <!--more--> is on a line by itself, but if it's in the middle of a paragraph, the result is that wpautop "forgets" to close the <p> tag for that paragraph. i.e. given the content:

This is a paragraph. Wanna read the rest? <!--more--> Well here it is!

Wordpress will generate:

<p>This is a paragraph. Wanna read the rest?
<div class='more-link fix'> <a href="..." class="more-link">Continue reading &raquo;</a></div>

Thus the <p> tag isn't closed, and validation breaks.
metal450
 
Posts: 85
Joined: 06 Feb 2010, 21:00

Re: suffusion_set_more_link filter breaks validation

Postby sayontan » 13 Feb 2010, 15:13

That is not within the theme developer's scope. Technically there are several things that a blog poster could break using invalid tags - you can't expect the theme developer to fix all of them.
Sayontan Sinha | http://mynethome.net/blog | http://www.aquoid.com/news
I don't do freelance work (for Suffusion or otherwise), so please don't contact me for quotes or offers.
sayontan
Site Admin
 
Posts: 10159
Joined: 15 Sep 2009, 16:39
Location: Houston, Texas

Re: suffusion_set_more_link filter breaks validation

Postby metal450 » 13 Feb 2010, 16:17

Huh? But it's the theme that's wrapping the "read more" text in a div; if that suffusion filter is removed, it validates just fine. From what I've seen there's nothing in Wordpress that stipulates "<!--more-->" must be on its own line and not within a paragraph: http://codex.wordpress.org/Customizing_the_Read_More

Furthermore, putting <!--more--> within a paragraph validates just fine on the default/classic themes. It's the addition of the <div> by suffusion_set_more_link that makes it not validate...
metal450
 
Posts: 85
Joined: 06 Feb 2010, 21:00

Re: suffusion_set_more_link filter breaks validation

Postby metal450 » 13 Feb 2010, 16:18

Also, changing <div> to <span> in that function makes it valid (regardless of where the author places their <!--more-->)
metal450
 
Posts: 85
Joined: 06 Feb 2010, 21:00

Re: suffusion_set_more_link filter breaks validation

Postby metal450 » 16 Feb 2010, 20:34

Thanx for fixing it :)
metal450
 
Posts: 85
Joined: 06 Feb 2010, 21:00


Return to Bug Reports

Who is online

Users browsing this forum: No registered users and 1 guest