[ 
http://www.stripesframework.org/jira/browse/STS-556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=11926#action_11926
 ] 

Nikolaos commented on STS-556:
------------------------------

Timothy,

Unless something has changed and HTML5 has gone "main stream" / beyond 
"expiremental" I think the HTML5 argument of this is the next upcoming thing 
and we should implement for it now is irrelevant.  There are ample Developers 
using HTML 4.01 and your argument would best stick there IMHO....

Also as you speak to the W3C considering XHTML dropped / dead then it ONLY 
naturally makes sense to consider the W3C Validator (http://validator.w3.org/) 
in presenting your case and NOT the Validator at http://validator.nu ... which 
in the simple test cases you provided DOES NOT provide the same results.

With that said, I agree, it would be a good idea for Stripes to allow 
configurable support for HTML 4.01+ vs. XHTML doc type.

However what you have TODAY with your example and Stripes 1.5.X GENERATES:
<input name="website.url" type="text" size="30"/>

1) HTML 5:  "This document was successfully checked as HTML5!"
2) XHTML 1.0 Strict:  "This document was successfully checked as XHTML 1.0 
Strict!"
3) HTML 4.01 Strict:  "This document was successfully checked as HTML 4.01 
Strict!" with 1 WARNING... YES... a WARNING not an ERROR it still successfully 
validates... and true a WARNING is annoying indeed... .

However the snippet of code you provide IS NOT configurable and instead 
GENERATES:
<input name="website.url" type="text" size="30">

1) HTML 5:  "This document was successfully checked as HTML5!"
2) XHTML 1.0 Strict:  "Errors found while checking this document as XHTML 1.0 
Strict!" with 6 ERRORS!!!!!!!!!!!!!!!
3) HTML 4.01 Strict:  "This document was successfully checked as HTML 4.01 
Strict!"

So you propose incorporating a patch that results in an Unsuccessful validation 
and 6 Errors for Developers using XHTML VS. the status quo which is Successful 
validation in ALL cases save 1 warning  for those strictly using HTML 4.01 
AND... YES... NO ERRORS OR WARNINGS FOR THOSE USING HTML5 which is your main 
argument!!!

I too think Stripes should be smarter about this BUT not by using the code you 
propose BUT some other logic AND this is more of an annoyance for those using 
HTML 4.01 Strict which is a large group indeed BUT clearly not for anyone 
wanting to ride the edge with HTML5.

So I would like to see this solved too but not for the arguments / reasons or 
the manner you propose... .

--Nikolaos

> Some stripes tags generate invalid html
> ---------------------------------------
>
>                 Key: STS-556
>                 URL: http://www.stripesframework.org/jira/browse/STS-556
>             Project: Stripes
>          Issue Type: Bug
>    Affects Versions: Release 1.5.1
>            Reporter: Dan Kaplan
>            Priority: Minor
>
> Hello, If I use an input tag like this:
> <stripes:text name="website.url" size="30"></stripes:text>
> It'll generate some html like this:
> <input name="website.url" type="text" size="30" />
> But, if your html is using a doctype like this:
> <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" 
> "http://www.w3.org/TR/html4/strict.dtd";>
> That form is actually invalid.  It should instead generate HTML like this: 
> <input name="website.url" type="text" size="30">
> Or this:
> <input name="website.url" type="text" size="30" ></input>
> Here is a validator that you can test this with:  http://validator.nu
> Paste this html into it to see for yourself:
> <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" 
> "http://www.w3.org/TR/html4/strict.dtd";>
> <html>
> <head><title>title</title></head>
> <body>
> <form action="na">
> <div>
> <input name="website.url" type="text" size="30"/>
> </div>
> </form>
> </body>
> </html>
> This HTML will validate after you remove the trailing / on the input tag.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://www.stripesframework.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------

_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to