Commit ec38133
Use documentwrapper's height of sidebar (#9)
This is sort of a more correct thing to do since the first/last element of the body may have a margin that can cause the documentwrapper to be taller than bodywrapper.
While this doesn't cause any major issues for this theme as is (due to the ample use of whitespace around the sidebar), this behaviour would help with derivative themes which remove the excessive margins on the sidebar.1 parent f87d922 commit ec38133
1 file changed
Lines changed: 4 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -28,6 +28,7 @@ $(function() { | |||
| 28 | 28 | var jwindow = $(window); | |
| 29 | 29 | var jdocument = $(document); | |
| 30 | 30 | var bodywrapper = $('.bodywrapper'); | |
| 31 | + var documentwrapper = $('.documentwrapper'); | ||
| 31 | 32 | var sidebar = $('.sphinxsidebar'); | |
| 32 | 33 | var sidebarwrapper = $('.sphinxsidebarwrapper'); | |
| 33 | 34 | ||
@@ -71,7 +72,7 @@ $(function() { | |||
| 71 | 72 | bodywrapper.css('margin-left', bw_margin_collapsed); | |
| 72 | 73 | sidebarbutton.css({ | |
| 73 | 74 | 'margin-left': '0', | |
| 74 | - 'height': bodywrapper.height(), | ||
| 75 | + 'height': documentwrapper.height(), | ||
| 75 | 76 | 'border-radius': '5px' | |
| 76 | 77 | }); | |
| 77 | 78 | sidebarbutton.find('span').text('»'); | |
@@ -85,7 +86,7 @@ $(function() { | |||
| 85 | 86 | sidebarwrapper.show(); | |
| 86 | 87 | sidebarbutton.css({ | |
| 87 | 88 | 'margin-left': ssb_width_expanded-12, | |
| 88 | - 'height': Math.max(sidebarwrapper.height(), bodywrapper.height()), | ||
| 89 | + 'height': Math.max(sidebarwrapper.height(), documentwrapper.height()), | ||
| 89 | 90 | 'border-radius': '0 5px 5px 0' | |
| 90 | 91 | }); | |
| 91 | 92 | sidebarbutton.find('span').text('«'); | |
@@ -109,7 +110,7 @@ $(function() { | |||
| 109 | 110 | // find the height of the viewport to center the '<<' in the page | |
| 110 | 111 | var viewport_height = get_viewport_height(); | |
| 111 | 112 | var sidebar_offset = sidebar.offset().top; | |
| 112 | - var sidebar_height = Math.max(bodywrapper.height(), sidebar.height()); | ||
| 113 | + var sidebar_height = Math.max(documentwrapper.height(), sidebar.height()); | ||
| 113 | 114 | sidebarbutton.find('span').css({ | |
| 114 | 115 | 'display': 'block', | |
| 115 | 116 | 'position': 'fixed', | |
0 commit comments