Merge pull request #3491 from davidfischer/alabaster-footer-formatting

Small formatting change to the Alabaster footer
readthedocs · Jan 10, 2018 · 76635e6 · 76635e6
2 parents d63caef + 9a9ba0e
commit 76635e6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/readthedocs/core/static-src/core/js/sponsorship.js b/readthedocs/core/static-src/core/js/sponsorship.js
@@ -117,14 +117,16 @@ Promo.prototype.place_promo = function (selector, promo_class) {
 Promo.prototype.get_alabaster_promo_selector = function () {
     // Return a jQuery selector where the promo goes on the Alabaster theme
     var self = this,
-        selector;
+        selector,
+        wrapper;
 
     if (self.display_type === constants.PROMO_TYPES.FOOTER) {
-        selector = $('<div />')
+        wrapper = $('<div />')
             .attr('class', 'rtd-pro-footer-wrapper body')
             .appendTo('div.bodywrapper');
-        $('<hr />').insertBefore(selector);
-        $('<hr />').insertAfter(selector);
+        $('<hr />').appendTo(wrapper);
+        selector = $('<div />').appendTo(wrapper);
+        $('<hr />').appendTo(wrapper);
     } else {
         selector = $('div.sphinxsidebar > div.sphinxsidebarwrapper');
     }