Skip to content

Commit d64428a

Browse files
committed
Fix HTML license; standardize HTML code
Authors are encouraged to omit the type attribute. And the language attribute was never standardized and should not be used. We use lowercase for HTML so made the doctype lowercase as well to be consistent. https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Elements/script/type https://developer.mozilla.org/en-US/docs/Web/HTML/Reference/Elements/script#language
1 parent 5b7c3b4 commit d64428a

File tree

7 files changed

+66
-62
lines changed

7 files changed

+66
-62
lines changed

systemvm/agent/noVNC/vnc.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<!DOCTYPE html>
1+
<!doctype html>
22
<html lang="en" class="noVNC_loading">
33
<head>
44

systemvm/agent/noVNC/vnc_lite.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
<!DOCTYPE html>
1+
<!doctype html>
22
<html lang="en">
33
<head>
44

systemvm/agent/ui/viewer.ftl

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,10 @@ under the License.
1818
-->
1919
<html>
2020
<head>
21-
<script type="text/javascript" language="javascript" src="/resource/js/jquery.js"></script>
22-
<script type="text/javascript" language="javascript" src="/resource/js/ajaxviewer.js"></script>
23-
<script type="text/javascript" language="javascript" src="/resource/js/ajaxkeys.js"></script>
24-
<script type="text/javascript" language="javascript" src="/resource/js/handler.js"></script>
21+
<script src="/resource/js/jquery.js"></script>
22+
<script src="/resource/js/ajaxviewer.js"></script>
23+
<script src="/resource/js/ajaxkeys.js"></script>
24+
<script src="/resource/js/handler.js"></script>
2525
<link rel="stylesheet" type="text/css" href="/resource/css/ajaxviewer.css"></link>
2626
<title>${title}</title>
2727
</head>
@@ -44,7 +44,7 @@ under the License.
4444

4545
<div id="main_panel" tabindex="1"></div>
4646

47-
<script language="javascript">
47+
<script>
4848
4949
var tileMap = [ ${tileSequence} ];
5050
var ajaxViewer = new AjaxViewer('main_panel', '${imgUrl}', '${updateUrl}', tileMap,

tools/apidoc/generatecommands.xsl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ version="1.0">
141141
</div><!-- #BeginLibraryItem "/libraries/footer.lbi" -->
142142
<div id="footer">
143143
<div id="comments_thread">
144-
<script type="text/javascript" src="https://comments.apache.org/show_comments.lua?site=test" async="true">
144+
<script src="https://comments.apache.org/show_comments.lua?site=test" async="true">
145145
</script>
146146
<noscript>
147147
<iframe width="930" height="500" src="https://comments.apache.org/iframe.lua?site=test&amp;page=4.2.0/rootadmin"></iframe>

tools/ngui/templates/index.html

Lines changed: 55 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,23 @@
1-
#Licensed to the Apache Software Foundation (ASF) under one
2-
#or more contributor license agreements. See the NOTICE file
3-
#distributed with this work for additional information
4-
#regarding copyright ownership. The ASF licenses this file
5-
#to you under the Apache License, Version 2.0 (the
6-
#"License"); you may not use this file except in compliance
7-
#with the License. You may obtain a copy of the License at
8-
#http://www.apache.org/licenses/LICENSE-2.0
9-
#Unless required by applicable law or agreed to in writing,
10-
#software distributed under the License is distributed on an
11-
#"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
12-
#KIND, either express or implied. See the License for the
13-
#specific language governing permissions and limitations
14-
#under the License.
1+
<!--
2+
Licensed to the Apache Software Foundation (ASF) under one
3+
or more contributor license agreements. See the NOTICE file
4+
distributed with this work for additional information
5+
regarding copyright ownership. The ASF licenses this file
6+
to you under the Apache License, Version 2.0 (the
7+
"License"); you may not use this file except in compliance
8+
with the License. You may obtain a copy of the License at
159
16-
<!DOCTYPE html>
10+
http://www.apache.org/licenses/LICENSE-2.0
11+
12+
Unless required by applicable law or agreed to in writing,
13+
software distributed under the License is distributed on an
14+
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
15+
KIND, either express or implied. See the License for the
16+
specific language governing permissions and limitations
17+
under the License.
18+
-->
19+
20+
<!doctype html>
1721
<html lang="en" ng-app="cloudstack">
1822
<head>
1923
<meta charset="utf-8">
@@ -102,41 +106,41 @@ <h3>CloudStack UI using Angular.js and Twitter Bootstrap</h3>
102106
</table>
103107
</script>
104108

105-
<script type="text/javascript" src="static/js/lib/jquery-1.7.2.js"></script>
106-
<script type="text/javascript" src="static/js/lib/angular.js"></script>
107-
<script type="text/javascript" src="static/js/app/app.js"></script>
108-
<script type="text/javascript" src="static/js/common/resources/virtualmachines.js"></script>
109-
<script type="text/javascript" src="static/js/app/instances/instances.js"></script>
110-
<script type="text/javascript" src="static/js/common/resources/volumes.js"></script>
111-
<script type="text/javascript" src="static/js/common/resources/snapshots.js"></script>
112-
<script type="text/javascript" src="static/js/app/storage/storage.js"></script>
113-
<script type="text/javascript" src="static/js/common/resources/networks.js"></script>
114-
<script type="text/javascript" src="static/js/app/networks/networks.js"></script>
115-
<script type="text/javascript" src="static/js/common/resources/templates.js"></script>
116-
<script type="text/javascript" src="static/js/app/templates/templates.js"></script>
117-
<script type="text/javascript" src="static/js/common/resources/events.js"></script>
118-
<script type="text/javascript" src="static/js/app/events/events.js"></script>
119-
<script type="text/javascript" src="static/js/common/resources/accounts.js"></script>
120-
<script type="text/javascript" src="static/js/app/accounts/accounts.js"></script>
121-
<script type="text/javascript" src="static/js/common/resources/domains.js"></script>
122-
<script type="text/javascript" src="static/js/app/domains/domains.js"></script>
123-
<script type="text/javascript" src="static/js/app/globalsettings/globalsettings.js"></script>
124-
<script type="text/javascript" src="static/js/app/serviceofferings/serviceofferings.js"></script>
125-
<script type="text/javascript" src="static/js/common/resources/serviceofferings.js"></script>
126-
<script type="text/javascript" src="static/js/common/resources/projects.js"></script>
127-
<script type="text/javascript" src="static/js/app/projects/projects.js"></script>
128-
<script type="text/javascript" src="static/js/common/resources/configurations.js"></script>
129-
<script type="text/javascript" src="static/js/common/services/breadcrumbs.js"></script>
130-
<script type="text/javascript" src="static/js/common/services/helperfunctions.js"></script>
131-
<script type="text/javascript" src="static/js/common/services/requester.js"></script>
132-
<script type="text/javascript" src="static/js/common/services/notifications.js"></script>
133-
<script type="text/javascript" src="static/js/common/directives/confirm.js"></script>
134-
<script type="text/javascript" src="static/js/common/directives/modal-form.js"></script>
135-
<script type="text/javascript" src="static/js/common/directives/label.js"></script>
136-
<script type="text/javascript" src="static/js/common/directives/edit-in-place.js"></script>
137-
<script type="text/javascript" src="static/js/common/dictionary.js"></script>
138-
<script type="text/javascript" src="static/js/common/resources/zones.js"></script>
139-
<script type="text/javascript" src="static/js/common/resources/diskofferings.js"></script>
140-
<script type="text/javascript" src="static/js/lib/angular-ui.min.js"></script>
109+
<script src="static/js/lib/jquery-1.7.2.js"></script>
110+
<script src="static/js/lib/angular.js"></script>
111+
<script src="static/js/app/app.js"></script>
112+
<script src="static/js/common/resources/virtualmachines.js"></script>
113+
<script src="static/js/app/instances/instances.js"></script>
114+
<script src="static/js/common/resources/volumes.js"></script>
115+
<script src="static/js/common/resources/snapshots.js"></script>
116+
<script src="static/js/app/storage/storage.js"></script>
117+
<script src="static/js/common/resources/networks.js"></script>
118+
<script src="static/js/app/networks/networks.js"></script>
119+
<script src="static/js/common/resources/templates.js"></script>
120+
<script src="static/js/app/templates/templates.js"></script>
121+
<script src="static/js/common/resources/events.js"></script>
122+
<script src="static/js/app/events/events.js"></script>
123+
<script src="static/js/common/resources/accounts.js"></script>
124+
<script src="static/js/app/accounts/accounts.js"></script>
125+
<script src="static/js/common/resources/domains.js"></script>
126+
<script src="static/js/app/domains/domains.js"></script>
127+
<script src="static/js/app/globalsettings/globalsettings.js"></script>
128+
<script src="static/js/app/serviceofferings/serviceofferings.js"></script>
129+
<script src="static/js/common/resources/serviceofferings.js"></script>
130+
<script src="static/js/common/resources/projects.js"></script>
131+
<script src="static/js/app/projects/projects.js"></script>
132+
<script src="static/js/common/resources/configurations.js"></script>
133+
<script src="static/js/common/services/breadcrumbs.js"></script>
134+
<script src="static/js/common/services/helperfunctions.js"></script>
135+
<script src="static/js/common/services/requester.js"></script>
136+
<script src="static/js/common/services/notifications.js"></script>
137+
<script src="static/js/common/directives/confirm.js"></script>
138+
<script src="static/js/common/directives/modal-form.js"></script>
139+
<script src="static/js/common/directives/label.js"></script>
140+
<script src="static/js/common/directives/edit-in-place.js"></script>
141+
<script src="static/js/common/dictionary.js"></script>
142+
<script src="static/js/common/resources/zones.js"></script>
143+
<script src="static/js/common/resources/diskofferings.js"></script>
144+
<script src="static/js/lib/angular-ui.min.js"></script>
141145
</body>
142146
</html>

ui/public/example.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
specific language governing permissions and limitations
1717
under the License.
1818
-->
19-
<!DOCTYPE html>
19+
<!doctype html>
2020
<html lang="en-gb">
2121
<head>
2222
<meta charset="utf-8">

ui/public/index.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
specific language governing permissions and limitations
1717
under the License.
1818
-->
19-
<!DOCTYPE html>
19+
<!doctype html>
2020
<html lang="en-gb">
2121
<head>
2222
<meta charset="utf-8">
@@ -54,7 +54,7 @@
5454
<div class="loader"></div>
5555
</div>
5656
</body>
57-
<script type="text/javascript">
57+
<script>
5858
fetch('./config.json?ts=' + Date.now())
5959
.then(response => response.json())
6060
.then(data => {

0 commit comments

Comments
 (0)