Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Javascript fixes / "performance" #17

Open
nfreear opened this issue Oct 28, 2015 · 0 comments
Open

Javascript fixes / "performance" #17

nfreear opened this issue Oct 28, 2015 · 0 comments
Assignees

Comments

@nfreear
Copy link
Owner

nfreear commented Oct 28, 2015

Nick@~/workspace/nfreear.github.io (master>)$ git diff
diff --git a/_includes/footer-javascript.html b/_includes/footer-javascript.html
index 04a191f..092b2e9 100644
--- a/_includes/footer-javascript.html
+++ b/_includes/footer-javascript.html
@@ -3,15 +3,19 @@
  id="js-config"
 {% for cfg in site.js_config %} data-{{ cfg[0] }}={{ cfg[1] | jsonify }}{% endfor %}
  data-post_title={{ page.title | jsonify }}
- data-post_id={{ page.id | jsonify }}
+ data-post_id="{{ page.id | slugify }}"
  data-post_count="{{ site.posts | size | jsonify }}"
  data-site_time={{ site.time | jsonify }}
 ></div>

 <script src="//ajax.googleapis.com/ajax/libs/jquery/1.11.3/jquery.min.js"></script>
-{% if page.url != '/index.html' %}
+{% case page.url %}
+{% when '/index.html' or '/tag/' or '/search/' %}
+<!--no-embed-->
+{% else %}
 <script src="http://embed.open.ac.uk/scripts/jquery.oembed.js"></script>
-{% endif %}
+{% endcase %}
+<!--page.url: {{ page.url }} -->
 <script src="/js/main.js"></script>

 {% if site.ex_comments and page.excerpt %}
diff
@nfreear nfreear self-assigned this Oct 28, 2015
nfreear added a commit that referenced this issue Oct 30, 2015
nfreear added a commit that referenced this issue Nov 2, 2015
* Bug #17 - generalise the "big-me" Javascript + CSS function.
nfreear added a commit that referenced this issue Nov 5, 2015
…n" JS

* Javascript: `window.external.AddSearchProvider()`
nfreear added a commit that referenced this issue Nov 15, 2015
* Other style/CSS/SASS & Javascript fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant