[jira] Commented: (TAPESTRY-2111) Remove HTML comments around JavaScript.

["Kalle Korhonen (JIRA)" <de...@tapestry.apache.org>]

    [ https://issues.apache.org/jira/browse/TAPESTRY-2111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12565018#action_12565018 ] 

Kalle Korhonen commented on TAPESTRY-2111:
------------------------------------------

Agree. It's not needed and can be harmful because in XHTML mode browser should actually *ignore* Javascript. Some resources:
http://javascript.about.com/library/blxhtml.htm
http://www.netmechanic.com/news/vol6/javascript_no12.htm
http://javascript.about.com/od/reference/a/accessibility_2.htm

Note that the latest browsers that didn't understand script tags at all were Netscape1 and IE 2 browsers, and nobody has been using those for years.


> Remove HTML comments around JavaScript.
> ---------------------------------------
>
>                 Key: TAPESTRY-2111
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2111
>             Project: Tapestry
>          Issue Type: Improvement
>    Affects Versions: 4.1.4
>            Reporter: Kevin Menard
>            Priority: Minor
>
> Tapestry's generated JavaScript generates HTML comments around its JavaScript.  While this served a purpose with older browsers, it really doesn't do much with anything semi-recent.  These browsers, if they don't support JavaScript, know not to render the text inside the script tag anyway.  My guess is that the only browsers that this would be useful for are also those that look for the "language" attribute, rather than the "text" attribute of the script tag, so they're dead in the water anyway.
> On top of all this, there have been unconfirmed reports that the comments can mess up IE7 in certain situations.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org