[GitHub] [tomcat] jbampton opened a new pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] jbampton opened a new pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox

jbampton opened a new pull request #316:
URL: https://github.com/apache/tomcat/pull/316


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] kkolinko commented on pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox

kkolinko commented on pull request #316:
URL: https://github.com/apache/tomcat/pull/316#issuecomment-653954164


   error.html:
   1. Shouldn't it better be <form method="GET"?
   2. Options shall be closed with </option>. The <br>s there are out of place and ignored. (validator.w3.org complains)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] jbampton commented on pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox
In reply to this post by GitBox

jbampton commented on pull request #316:
URL: https://github.com/apache/tomcat/pull/316#issuecomment-653964411


   Hey @kkolinko I removed the `br` tags and also closed the `option` tags now.
   
   I also validated the 'error.html' by direct upload and it validates now.
   
    https://validator.w3.org/#validate_by_input 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] martin-g commented on a change in pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox
In reply to this post by GitBox

martin-g commented on a change in pull request #316:
URL: https://github.com/apache/tomcat/pull/316#discussion_r450018806



##########
File path: webapps/ROOT/index.jsp
##########
@@ -25,11 +25,11 @@ request.setAttribute("tomcatExamplesUrl", "/examples/");
 <!DOCTYPE html>
 <html lang="en">
     <head>
-        <meta charset="UTF-8" />
+        <meta charset="UTF-8">

Review comment:
       Why did you remove the auto-closing ? It doesn't do any harm. I, personally, prefer the style it was before.

##########
File path: webapps/examples/jsp/cal/cal2.jsp
##########
@@ -15,31 +15,30 @@
   limitations under the License.
 --%>
 <%@page contentType="text/html; charset=UTF-8" %>
-<HTML>
-<HEAD><TITLE>
-    Calendar: A JSP APPLICATION
-</TITLE></HEAD>
+<html>
+<head>
+<title>Calendar: A JSP APPLICATION</title>
+</head>
 
-
-<BODY BGCOLOR="white">
+<body bgcolor="white">
 <jsp:useBean id="table" scope="session" class="cal.TableBean" />
 
 <%
     String time = request.getParameter ("time");
 %>
 
-<FONT SIZE=5> Please add the following event:
-<BR> <h3> Date <%= table.getDate() %>
-<BR> Time <%= util.HTMLFilter.filter(time) %> </h3>
-</FONT>
-<FORM METHOD=POST ACTION=cal1.jsp>
-<BR>
-<BR> <INPUT NAME="date" TYPE=HIDDEN VALUE="current">
-<BR> <INPUT NAME="time" TYPE=HIDDEN VALUE="<%= util.HTMLFilter.filter(time) %>">
-<BR> <h2> Description of the event <INPUT NAME="description" TYPE=TEXT SIZE=20> </h2>
-<BR> <INPUT TYPE=SUBMIT VALUE="submit">
-</FORM>
-
-</BODY>
-</HTML>
+<font size="5"> Please add the following event:
+<br> <h3> Date <%= table.getDate() %>

Review comment:
       indentation

##########
File path: webapps/examples/jsp/cal/cal1.jsp
##########
@@ -32,63 +31,60 @@
 %>
 
 <!-- html table goes here -->
-<CENTER>
-<TABLE WIDTH=60% BGCOLOR=yellow CELLPADDING=15>
-<TR>
-<TD ALIGN=CENTER> <A HREF=cal1.jsp?date=prev> prev </A>
-<TD ALIGN=CENTER> Calendar:<%= table.getDate() %></TD>
-<TD ALIGN=CENTER> <A HREF=cal1.jsp?date=next> next </A>
-</TR>
-</TABLE>
+<center>
+<table width="60%" bgcolor="yellow" cellpadding="15">
+<tr>
+<td align="center"> <a href="cal1.jsp?date=prev"> prev </a>
+<td align="center"> Calendar:<%= table.getDate() %></td>
+<td align="center"> <a href="cal1.jsp?date=next"> next </a>

Review comment:
       no indentation

##########
File path: webapps/examples/jsp/cal/cal1.jsp
##########
@@ -15,13 +15,12 @@
   limitations under the License.
 --%>
 <%@page contentType="text/html; charset=UTF-8" %>
-<HTML>
-<HEAD><TITLE>
-    Calendar: A JSP APPLICATION
-</TITLE></HEAD>
+<html>
+<head>
+<title>Calendar: A JSP APPLICATION</title>

Review comment:
       There is no indentation here.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] jbampton commented on a change in pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox
In reply to this post by GitBox

jbampton commented on a change in pull request #316:
URL: https://github.com/apache/tomcat/pull/316#discussion_r450021555



##########
File path: webapps/ROOT/index.jsp
##########
@@ -25,11 +25,11 @@ request.setAttribute("tomcatExamplesUrl", "/examples/");
 <!DOCTYPE html>
 <html lang="en">
     <head>
-        <meta charset="UTF-8" />
+        <meta charset="UTF-8">

Review comment:
       In HTML5 you don't need the auto-closing.
   I got the style from MDN.
   https://developer.mozilla.org/en-US/docs/Web/HTML/Element/meta




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] martin-g commented on a change in pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox
In reply to this post by GitBox

martin-g commented on a change in pull request #316:
URL: https://github.com/apache/tomcat/pull/316#discussion_r450023003



##########
File path: webapps/ROOT/index.jsp
##########
@@ -25,11 +25,11 @@ request.setAttribute("tomcatExamplesUrl", "/examples/");
 <!DOCTYPE html>
 <html lang="en">
     <head>
-        <meta charset="UTF-8" />
+        <meta charset="UTF-8">

Review comment:
       I know, but it does not do any harm either.
   This change makes the diff bigger and harder to back-port to the other branches.
   I prefer the XHTML style because it is easier to spot errors in the IDE/text editor without copying to W3C validator in the web.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] stokito commented on a change in pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox
In reply to this post by GitBox

stokito commented on a change in pull request #316:
URL: https://github.com/apache/tomcat/pull/316#discussion_r457980335



##########
File path: webapps/ROOT/index.jsp
##########
@@ -25,11 +25,11 @@ request.setAttribute("tomcatExamplesUrl", "/examples/");
 <!DOCTYPE html>
 <html lang="en">
     <head>
-        <meta charset="UTF-8" />
+        <meta charset="UTF-8">
         <title><%=request.getServletContext().getServerInfo() %></title>
-        <link href="favicon.ico" rel="icon" type="image/x-icon" />
-        <link href="favicon.ico" rel="shortcut icon" type="image/x-icon" />
-        <link href="tomcat.css" rel="stylesheet" type="text/css" />
+        <link href="favicon.ico" rel="icon" type="image/x-icon">

Review comment:
       in fact we can remove the line but the favicon.ico will be anyway requested by a browser. The tag makes sense when the file name is another




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[GitHub] [tomcat] jbampton commented on pull request #316: Use lowercase HTML tags and attributes. Add quotes around attributes.

GitBox
In reply to this post by GitBox

jbampton commented on pull request #316:
URL: https://github.com/apache/tomcat/pull/316#issuecomment-667315584


   > error.html:
   >
   > 1. Shouldn't it better be <form method="GET"?
   > 2. Options shall be closed with . The s there are out of place and ignored. (validator.w3.org complains)
   
   MDN says -> Possible (case insensitive) values
   https://developer.mozilla.org/en-US/docs/Web/HTML/Element/form
   
   ![Screen Shot 2020-08-01 at 5 31 33 am](https://user-images.githubusercontent.com/418747/89070645-9b3b0100-d3b8-11ea-8c0a-748db69c5c81.png)
   
   
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]



---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]