Headlines in Firefox

Found a bug? Tell us!!
Post Reply
Josh
Regular
Posts: 110
Joined: Mon Jul 18, 2005 3:02 pm
Location: Berlin
Contact:

Headlines in Firefox

Post by Josh »

Internet Explorer correctly shows the headlines of all posts in a big font on my frontpage:
http://atlanticreview.org/

Firefox, however, only shows the headline of the first post in a big font on my frontpage. All following headlines are in small fonts.

I am not sure, if this is a bug or whether it is just a problem with my Firefox.
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Headlines in Firefox

Post by garvinhicking »

Hi!

I guess this is a HTML validation issue. Check http://validator.w3.org/check?verbose=1 ... iew.org%2F - it throws a lot of errors.

Tip 1: Switch your serendipity to "XHTML 1.1 compliant output". The rest is up to your template and entries/sidebar plugins. :)

Best regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Josh
Regular
Posts: 110
Joined: Mon Jul 18, 2005 3:02 pm
Location: Berlin
Contact:

Post by Josh »

Thank you, Garvin.

I have always had "Force XHTML 1.1 compliance" switched on in the configurations. Besides I am using the "Fix common XHTML errors" plugin.

Unfortunately, I don't know and understand enough HTML to fix all the errors myself.
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post by garvinhicking »

Hi!

The Fix-Plugin only helps you with most of the problems; it cannot fix unmatching tags or invalid tags.

This is what you need to check:

* Lowercase <META> tags. XHTML does not allow uppercase tags
* Wrong <ul><li> elemenets. After <ul> there must be a <li>, and all <li>s must be closed before </ul>. No texst is allowed within <ul>, only within <li>.
* Special characters must be escaped ("non SGML characters")
* All <div> tags must be properly matched by a closing </div>
* "border" is not a valid attribute, it must be removed for all tags and replaced with "style='border: 0px'".
* All <p> tags need a closing </p>
* Check all those seek="" shall="" not=""

Best regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Josh
Regular
Posts: 110
Joined: Mon Jul 18, 2005 3:02 pm
Location: Berlin
Contact:

Post by Josh »

* Wrong <ul><li> elemenets. After <ul> there must be a <li>, and all <li>s must be closed before </ul>.
It should be correct, but it isn't. The Vailidator says:
Error Line 58 column 87: document type does not allow element "div" here; assuming missing "li" start-tag.

...ner_serendipity_plugin_recententries">
This is the relevant code for the drop down menu in my index.tpl:

Code: Select all

<div id="navigation">
<ul id="nav_header_menu">

<li><a href="{$serendipityBaseURL}" accesskey="h">Home</a></li>

<li><a href="#">About</a>
	<ul>
		<li><a href="http://www.atlanticreview.org/archives/116-ABOUT-US.html">About the Atlantic Review</a></li>
		<li><a href="http://www.atlanticreview.org/archives/55-Impressum-and-Disclaimer.html">Impressum, Masthead, Disclaimer</a></li>
		<li><a href="mailto:editors@atlanticreview.org?subject=Regarding%20the%20Atlantic%20Review">Contact Us</a></li>
	</ul></li>
	
<li><a href="#">New Entries</a>
	<ul>{serendipity_showPlugin class="serendipity_plugin_recententries" side="hidden" negate="null"}</ul></li>
This continues with a few more drop down menues and then ends with

Code: Select all

</ul>
</div>
Thus I don't see where my error is. :-(
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post by garvinhicking »

Hi!

You are not allowed to use

Code: Select all

<li><a href="#">New Entries</a>
	<ul>{serendipity_showPlugin class="serendipity_plugin_recententries" side="hidden" negate="null"}</ul></li>
You must use.

Code: Select all

<li><a href="#">New Entries</a>
	<ul><li>{serendipity_showPlugin class="serendipity_plugin_recententries" side="hidden" negate="null"}</li></ul></li>
Note how to properly wrap the plugin inside <li> and not <ul>.

Regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Josh
Regular
Posts: 110
Joined: Mon Jul 18, 2005 3:02 pm
Location: Berlin
Contact:

Post by Josh »

Thank you. That worked. Now there are "only" 45 errors left. :-(

The validator says:
Error Line 230 column 47: document type does not allow element "div" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag.

<div class="serendipity_entry_body">
But that is the original S9Y stuff. I did not mess with it, I think. Why is element div not allowed? There is an /div for closure. Why does div need a start tag?

Next error:
Error Line 231 column 3835: there is no attribute "border".

...edium none ;" marginwidth="0" border="0" src="http://rcm.amazon.com/e/cm?t=at
But that is an iframe copied exactly from Amazon. I checked.

Next error:
Error Line 237 column 48: document type does not allow element "div" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag.

<div class='serendipity_entryFooter'>
But that is also normal S9Y. I did not mess with it.


I have solved my problem with the headlines, by writing that post new. However, it would be nice to have an XHTML compliment blog. Though, how important is that?

Having 45 errors is not good for google page rank, I assume. However, will users experience problems due to these errors?
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Post by garvinhicking »

Hi!

I think many of those errors stem from a previous error with an invalid <p> element enclosing stuff or so? Maybe the original creator of your template, Carl galloway, does know some help here?
...edium none ;" marginwidth="0" border="0" src="http://rcm.amazon.com/e/cm?t=at
But that is an iframe copied exactly from Amazon. I checked.
[/quote]

Amazon only gives a HTML4.0 example, but their code is not XHTML compliant, you need to remove the 'border'.
I have solved my problem with the headlines, by writing that post new. However, it would be nice to have an XHTML compliment blog. Though, how important is that?
The default template should validate as proper XHTML.

Best regards,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Josh
Regular
Posts: 110
Joined: Mon Jul 18, 2005 3:02 pm
Location: Berlin
Contact:

Post by Josh »

Thanks again, Garvin.

I have deleted all

Code: Select all

border="0" 
in the Amazon code. And Carl has fixed by entrie.tpl

Now my page Is Valid XHTML 1.0 Transitional!

The S9Y community is great!
carl_galloway
Regular
Posts: 1331
Joined: Sun Dec 04, 2005 5:43 pm
Location: Andalucia, Spain
Contact:

Post by carl_galloway »

But I didn't do very much... Pleased to see it all working for you Josh
Post Reply