]> git.ipfire.org Git - thirdparty/foundation/foundation-sites.git/commitdiff
Reveal modal: Fix confusing AJAX example 9299/head
authorDavid Reese <david@whatcould.com>
Wed, 26 Oct 2016 12:31:51 +0000 (08:31 -0400)
committerDavid Reese <david@whatcould.com>
Wed, 26 Oct 2016 12:51:11 +0000 (08:51 -0400)
The .done() callback receives the data from the $.ajax request; if the request format is HTML, that `resp` will just be a string, not an object. So `resp.html` returns undefined and is non-sensical. This only makes sense if the request format is JSON.

Since my impression is most people are going to be hitting a URL that returns HTML, I'd suggest just changing the example to `$modal.html(resp)` as I've done in this pull request. If you expect most people to be making a request that returns JSON, I would at least imply that in the URL, eg `$.ajax('/api/url.json')`.

docs/pages/reveal.md

index 68572f489d771bb9f27391881c3f954fceb6fcfd..a1a6f0365c83e4d72471cffd305d06bf35612644 100644 (file)
@@ -183,7 +183,7 @@ var $modal = $('#modal');
 
 $.ajax('/url')
   .done(function(resp){
-    $modal.html(resp.html).foundation('open');
+    $modal.html(resp).foundation('open');
 });
 ```