Skip to content

Commit

Permalink
Fix import order in tutorials.
Browse files Browse the repository at this point in the history
geo.ext.js has to be imported before geo.js for Hammer to be used.  With this change, the tutorials work on an iPad.
  • Loading branch information
manthey committed Aug 4, 2017
1 parent 7f8590c commit 5e46023
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
5 changes: 3 additions & 2 deletions tutorials/basic/index.pug
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ block mainTutorial
<!DOCTYPE html>
<html>
<head>
<script type="text/javascript" src="../../built/geo.min.js"></script>
<script type="text/javascript" src="../../built/geo.ext.min.js"></script>
<script type="text/javascript" src="../../built/geo.min.js"></script>
</head>
<body>
<div id="map"></div>
Expand All @@ -19,7 +19,8 @@ block mainTutorial

:markdown-it
The second script, ``geo.ext.min.js``, is optional. It adds support
for svg elements and touch interaction.
for svg elements and touch interaction. It needs to be added before
``geo.min.js``.

Some CSS will make our map fill the page:

Expand Down
2 changes: 1 addition & 1 deletion tutorials/common/tutorials.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ var processBlockInfo = {
/* We could also load from the CDN:
* https://cdnjs.cloudflare.com/ajax/libs/geojs/0.12.2/geo.min.js
* or the non-minified versions to make debug easier. */
' <script type="text/javascript" src="../../built/geo.min.js"></script>\n' +
' <script type="text/javascript" src="../../built/geo.ext.min.js"></script>\n' +
' <script type="text/javascript" src="../../built/geo.min.js"></script>\n' +
'</head>\n' +
'<body>\n' +
' <div id="map"></div>\n' +
Expand Down

0 comments on commit 5e46023

Please sign in to comment.