From: Jani Nikula <jani.nikula@intel.com>
To: intel-gfx@lists.freedesktop.org
Cc: jani.nikula@intel.com
Subject: [maintainer-tools PATCH 6/5] doc: load WaveDrom scripts directly from CDN instead of bundling
Date: Thu, 10 Aug 2017 00:33:58 +0300 [thread overview]
Message-ID: <20170809213358.18973-1-jani.nikula@intel.com> (raw)
In-Reply-To: <cover.1502312576.git.jani.nikula@intel.com>
Way back when the WaveDrom stuff was added, the scripts could only be
accessed over http. This caused issues with sites served over https and
modern browsers rightly complaining about mixed content. This was worked
around by downloading the WaveDrom scripts over http at build time, and
bundling them inline into the http.
Now that WaveDrom is available over https, simplify the hackery, and let
the user's browser load the scripts directly at page load time.
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
---
drm-intel-timeline.rst | 20 +++-----------------
drm-misc-timeline.rst | 21 +++------------------
2 files changed, 6 insertions(+), 35 deletions(-)
diff --git a/drm-intel-timeline.rst b/drm-intel-timeline.rst
index 3ab39afd5788..fe69fd374cf6 100644
--- a/drm-intel-timeline.rst
+++ b/drm-intel-timeline.rst
@@ -1,22 +1,8 @@
.. raw:: html
- <script type="text/javascript">
- /* Embedded WaveDrom skin from http://wavedrom.com/skins/default.js */
-
-.. raw:: html
- :url: http://wavedrom.com/skins/default.js
-
-.. raw:: html
-
- </script>
- <script type="text/javascript">
- /* Embedded WaveDrom engine from http://wavedrom.com/WaveDrom.js */
-
-.. raw:: html
- :url: http://wavedrom.com/WaveDrom.js
-
-.. raw:: html
-
+ <script src="https://cdnjs.cloudflare.com/ajax/libs/wavedrom/1.4.1/skins/default.js" type="text/javascript"></script>
+ <script src="https://cdnjs.cloudflare.com/ajax/libs/wavedrom/1.4.1/wavedrom.min.js" type="text/javascript"></script>
+ <script>
function init() {
WaveDrom.ProcessAll();
}
diff --git a/drm-misc-timeline.rst b/drm-misc-timeline.rst
index a9a80d6a4cfb..76eb780ebd0a 100644
--- a/drm-misc-timeline.rst
+++ b/drm-misc-timeline.rst
@@ -1,27 +1,13 @@
.. raw:: html
- <script type="text/javascript">
- /* Embedded WaveDrom skin from http://wavedrom.com/skins/default.js */
-
-.. raw:: html
- :url: http://wavedrom.com/skins/default.js
-
-.. raw:: html
-
+ <script src="https://cdnjs.cloudflare.com/ajax/libs/wavedrom/1.4.1/skins/default.js" type="text/javascript"></script>
+ <script src="https://cdnjs.cloudflare.com/ajax/libs/wavedrom/1.4.1/wavedrom.min.js" type="text/javascript"></script>
+ <script>
function init() {
WaveDrom.ProcessAll();
}
window.onload = init;
</script>
- <script type="text/javascript">
- /* Embedded WaveDrom engine from http://wavedrom.com/WaveDrom.js */
-
-.. raw:: html
- :url: http://wavedrom.com/WaveDrom.js
-
-.. raw:: html
-
- </script>
<script type="WaveDrom">
.. raw:: html
@@ -30,4 +16,3 @@
.. raw:: html
</script>
-
--
2.11.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2017-08-09 21:34 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-09 21:08 [maintainer-tools PATCH 0/5] add Sphinx doc build to maintainer tools Jani Nikula
2017-08-09 21:08 ` [maintainer-tools PATCH 1/5] mancheck: run dim.rst through rst2man Jani Nikula
2017-08-09 22:45 ` Rodrigo Vivi
2017-08-09 21:08 ` [maintainer-tools PATCH 2/5] build: add drm-misc-commit-flow.svg to clean target Jani Nikula
2017-08-09 22:45 ` Rodrigo Vivi
2017-08-09 21:08 ` [maintainer-tools PATCH 3/5] doc: use window.onload to call WaveDrom.ProcessAll() Jani Nikula
2017-08-09 22:49 ` Rodrigo Vivi
2017-08-10 8:32 ` Jani Nikula
2017-08-09 21:08 ` [maintainer-tools PATCH 4/5] doc: build documentation using Sphinx Jani Nikula
2017-08-09 22:56 ` Rodrigo Vivi
2017-08-10 6:49 ` Jani Nikula
2017-08-09 21:08 ` [maintainer-tools PATCH 5/5] doc: use Sphinx bizstyle builtin html theme Jani Nikula
2017-08-09 23:09 ` Rodrigo Vivi
2017-08-10 7:22 ` Jani Nikula
2017-08-10 15:11 ` Rodrigo Vivi
2017-08-09 21:33 ` Jani Nikula [this message]
2017-08-09 23:00 ` [maintainer-tools PATCH 6/5] doc: load WaveDrom scripts directly from CDN instead of bundling Rodrigo Vivi
2017-08-10 6:43 ` Jani Nikula
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170809213358.18973-1-jani.nikula@intel.com \
--to=jani.nikula@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox