Skip to content

Commit 4b7a5df

Browse files
committed
For HTML5 script tags you can omit the type attribute when its value is JavaScript. From MDN -> https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script - Omitted or a JavaScript MIME type: For HTML5-compliant browsers this indicates the script is JavaScript. HTML5 specification urges authors to omit the attribute rather than provide a redundant MIME type.
1 parent 5e16992 commit 4b7a5df

File tree

5 files changed

+17
-17
lines changed

5 files changed

+17
-17
lines changed

README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,13 @@ and require plotly.js using CommonJS as `var Plotly = require('plotly.js');` or
5454
#### Use the plotly.js CDN hosted by Fastly
5555
```html
5656
<!-- Latest compiled and minified plotly.js JavaScript -->
57-
<script type="text/javascript" src="https://cdn.plot.ly/plotly-latest.min.js"></script>
57+
<script src="https://cdn.plot.ly/plotly-latest.min.js"></script>
5858

5959
<!-- OR use a specific plotly.js release (e.g. version 1.5.0) -->
60-
<script type="text/javascript" src="https://cdn.plot.ly/plotly-1.5.0.min.js"></script>
60+
<script src="https://cdn.plot.ly/plotly-1.5.0.min.js"></script>
6161

6262
<!-- OR an un-minified version is also available -->
63-
<script type="text/javascript" src="https://cdn.plot.ly/plotly-latest.js"></script>
63+
<script src="https://cdn.plot.ly/plotly-latest.js"></script>
6464
```
6565

6666
and use the `Plotly` object in the window scope.
@@ -99,7 +99,7 @@ To learn more about the plotly.js module architecture, refer to our [modularizin
9999
Important: the plotly.js code base contains some non-ascii characters. Therefore, please make sure to set the `charset` attribute to `"utf-8"` in the script tag that imports your plotly.js bundle. For example:
100100

101101
```html
102-
<script type="text/javascript" src="my-plotly-bundle.js" charset="utf-8"></script>
102+
<script src="my-plotly-bundle.js" charset="utf-8"></script>
103103
```
104104

105105

devtools/image_viewer/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,6 @@
77
<div id="plot-images"></div>
88
<pre id="plot-mock"></pre>
99

10-
<script type="text/javascript" src="../../build/image_viewer-bundle.js"></script>
10+
<script src="../../build/image_viewer-bundle.js"></script>
1111
</body>
1212
</html>

devtools/test_dashboard/index.html

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@
2323
<!-- <script>if(typeof window.Int16Array !== 'function')document.write("<scri"+"pt src='../../dist/extras/typedarray.min.js'></scr"+"ipt>");</script>
2424
<script>document.write("<scri"+"pt src='../../dist/extras/request_animation_frame.js'></scr"+"ipt>");</script> -->
2525

26-
<script type="text/javascript" src="../../dist/extras/mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>
27-
<script id="source" type="text/javascript" src="../../build/plotly.js"></script>
28-
<script type="text/javascript" src="../../build/test_dashboard-bundle.js"></script>
26+
<script src="../../dist/extras/mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>
27+
<script id="source" src="../../build/plotly.js"></script>
28+
<script src="../../build/test_dashboard-bundle.js"></script>
2929
</body>
3030
</html>

tasks/stats.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -51,13 +51,13 @@ function getInfoContent() {
5151
'Import plotly.js as:',
5252
'',
5353
'```html',
54-
'<script type="text/javascript" src="plotly.min.js"></script>',
54+
'<script src="plotly.min.js"></script>',
5555
'```',
5656
'',
5757
'or the un-minified version as:',
5858
'',
5959
'```html',
60-
'<script type="text/javascript" src="plotly.js" charset="utf-8"></script>',
60+
'<script src="plotly.js" charset="utf-8"></script>',
6161
'```',
6262
'',
6363
'### To support IE9',
@@ -74,7 +74,7 @@ function getInfoContent() {
7474
'*Before* the plotly.js script tag, add:',
7575
'',
7676
'```html',
77-
'<script type="text/javascript" src="mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>',
77+
'<script src="mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>',
7878
'```',
7979
'',
8080
'You can grab the relevant MathJax files in `./dist/extras/mathjax/`.',
@@ -90,8 +90,8 @@ function getInfoContent() {
9090
'*After* the plotly.js script tag, add:',
9191
'',
9292
'```html',
93-
'<script type="text/javascript" src="plotly-locale-de-ch.js"></script>',
94-
'<script type="text/javascript">Plotly.setPlotConfig({locale: \'de-CH\'})</script>',
93+
'<script src="plotly-locale-de-ch.js"></script>',
94+
'<script>Plotly.setPlotConfig({locale: \'de-CH\'})</script>',
9595
'```',
9696
'',
9797
'The first line loads and registers the locale definition with plotly.js, the second sets it as the default for all Plotly plots.',

test/image/index.html

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
<html>
33
<body>
44
<!-- this index file gets copied in to the image server docker -->
5-
<script type="text/javascript" src="../plotly.js/dist/extras/mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>
6-
<script type="text/javascript" src="../plotly.js/build/plotly.js" charset="utf-8"></script>
7-
<script type="text/javascript" src="../plotly.js/dist/plotly-geo-assets.js" charset="utf-8"></script>
8-
<script type="text/javascript" src="./main.js"></script>
5+
<script src="../plotly.js/dist/extras/mathjax/MathJax.js?config=TeX-AMS-MML_SVG"></script>
6+
<script src="../plotly.js/build/plotly.js" charset="utf-8"></script>
7+
<script src="../plotly.js/dist/plotly-geo-assets.js" charset="utf-8"></script>
8+
<script src="./main.js"></script>
99
</body>
1010
</html>

0 commit comments

Comments
 (0)