[#WW-1381] invalid html 4.0 generated by ww:head tag

WebWork

invalid html 4.0 generated by ww:head tag

Created: 23/Nov/06 03:16 AM Updated: 23/Nov/06 03:30 PM

Component/s:

None

Affects Version/s:

2.2.4

Fix Version/s:

2.2.4

Description

« Hide

I'm using tag
<ww:head />
and this tag generates folowwing html
<link rel="stylesheet" href="/WAL/webwork/xhtml/styles.css" type="text/css"/>

but I run it through validator at http://validator.w3.org/

It complains and sais it is not according to HTML 4.0.1 standard, it should be

<link rel="stylesheet" href="/WAL/webwork/xhtml/styles.css" type="text/css">

end tag is forbidden for link.

To test
paste this in validator
------
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<title>d92ulg</title>
<link rel="stylesheet" href="/WAL/webwork/xhtml/styles.css" type="text/css"/>
</head>
<body>
</body>
</html>
-----

I assume this bug exists in struts 2, which we will move to as soon there is a stable version.

Tankyou for webwork, it's working great.

All

Comments

Change History

Sort Order:

[ Permlink | « Hide ]

Philip Luppens - [23/Nov/06 03:33 AM ]

This is a tricky one .. we also need to support xhtml - where I assume not closing a link tag would result in a similar error. If it's a real problem, override the head template (see the wiki/docs on how to do that - it's very simple), or just drop the tag, and place the html directly in your page instead of the tag.

[ Permlink | « Hide ]

Philip Luppens - [23/Nov/06 03:35 AM ]

Updated summary.

[ Permlink | « Hide ]

Urban Lindberg - [23/Nov/06 06:33 AM ]

You ar correct, xhtml would complain about missing end :)
I will fix my problem by updating the head template. or maybe change doctype, but then I will get other complaints i suppose.

I do not know if this is to ask to much, but maybe one would wan't to be able to define in some configuration file if one would like to generate html or xhtml.

What is the status of xhtml, do browser support it?

[ Permlink | « Hide ]

Philip Luppens - [23/Nov/06 07:45 AM ]

Adding support for HTML 4.0 would be simple (just add a 'html' theme), but I suspect most people try to use xhtml nowadays, so interest for it will be quite low.

xhtml: afaik, all modern browsers support it without problems. CSS otoh ..

[ Permlink | « Hide ]

Urban Lindberg - [23/Nov/06 03:13 PM ]

A html theme I suppose is the way to do it. Thank you for you're prompt answer.

I was looking for where to close this bug, but can't seem to find it

[ Permlink | « Hide ]

Rainer Hermanns - [23/Nov/06 03:30 PM ]

Closed as requested