Skip to content

Commit

Permalink
Fix sidebar height in styles.
Browse files Browse the repository at this point in the history
There was some inconsistency with sidebar height definition in styles,
it is fixed now.

Also, #content gained overflow hidden to force every element inside it
that has clear:both set to be present next to the sidebar, not below
it.
  • Loading branch information
gbdlin committed Nov 15, 2018
1 parent 1554ac8 commit 800a9b5
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 8 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

14 changes: 10 additions & 4 deletions admin_toolbox/sidebar/static/admin_sidebar/css/admin-sidebar.scss
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
$sidebar_width: 220px;

.submit-row {
position: fixed;
bottom: 0;
right: 0;
left: 220px;
left: $sidebar_width;
margin-bottom: 0 !important;
}

Expand All @@ -25,7 +27,7 @@ html.w-su-sidebar {
}

#su-content {
padding-left: 200px;
padding-left: $sidebar_width;
overflow: hidden;
padding-bottom: 70px;

Expand All @@ -38,9 +40,9 @@ html.w-su-sidebar {

#su-sidebar {
background: #79aec8;
width: 220px;
width: $sidebar_width;
float: left;
margin-left: -200px;
margin-left: -$sidebar_width;
top: 60px;
left: 0;
bottom: 0;
Expand Down Expand Up @@ -206,6 +208,10 @@ html.w-su-sidebar {
}
}

#content {
overflow: hidden;
}

.object-tools a.addlink {
padding-right: 32px;
}
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
setup(
name='django-admin-toolbox',

version='1.0.0.dev13',
version='1.0.0.dev14',

description='Django admin toolbox - bunch of improvements for default django admin',
long_description=long_description,
Expand Down

0 comments on commit 800a9b5

Please sign in to comment.