Bladeren bron

Change sidebar mobile close button

Denis K 8 jaren geleden
bovenliggende
commit
416c2b5310

+ 4 - 0
jet/static/jet/css/_header.scss

@@ -68,6 +68,10 @@
       body.scroll-to-bottom & {
         transform: translate3d(0, -100%, 0);
       }
+
+      &.sidebar-opened {
+        transform: translate3d(100%, 0, 0);
+      }
     }
 
     &.opened {

+ 48 - 43
jet/static/jet/css/_sidebar.scss

@@ -13,14 +13,16 @@
   padding-bottom: 32px;
 
   @include for-mobile {
-    min-width: 280px;
-    max-width: 360px;
-    width: 80%;
+    width: 360px;
     padding-bottom: 0;
     transform: translate3d(-100%, 0, 0);
     transition: transform $transitions-duration cubic-bezier(0, 0.5, 0.5, 1);
   }
 
+  @include for-phone {
+    width: 80%;
+  }
+
   &.sidebar-opened {
     @include for-mobile {
       transform: none;
@@ -29,23 +31,41 @@
   }
 
   &-header {
-    display: none;
-    background-color: $sidebar-background-color;
-    color: $sidebar-text-color;
     height: $sidebar-header-height;
     line-height: $sidebar-header-height;
-    position: fixed;
-    top: 0;
-    right: 0;
-    left: 0;
-    z-index: 3;
-    transition: background-color $transitions-duration, transform $transitions-duration;
+    transition: transform $transitions-duration;
 
-    @include for-mobile {
-      display: block;
+    &.sidebar-opened {
+      @include for-mobile {
+        transform: translate3d(360px, 0, 0);
+      }
+
+      @include for-phone {
+        transform: translate3d(80%, 0, 0);
+      }
+    }
+
+    &-wrapper {
+      display: none;
+      background-color: $sidebar-background-color;
+      color: $sidebar-text-color;
+      position: fixed;
+      top: 0;
+      right: 0;
+      left: 0;
+      z-index: 3;
+      transition: background-color $transitions-duration, transform $transitions-duration;
+
+      @include for-mobile {
+        display: block;
+
+        body.scroll-to-bottom & {
+          transform: translate3d(0, -100%, 0);
+        }
+      }
 
-      body.scroll-to-bottom & {
-        transform: translate3d(0, -100%, 0);
+      &.sidebar-opened {
+        background-color: $sidebar-contrast-background-color;
       }
     }
 
@@ -63,37 +83,22 @@
       &-icon {
         font-size: 16px;
         vertical-align: middle;
-      }
-    }
-  }
-
-  &-close {
-    display: none;
-    position: absolute;
-    right: 0;
-    padding: 4px;
-    margin: 12px 18px 0 12px;
-    background: $sidebar-contrast-background-color;
-    border-radius: 5px;
-    z-index: 1;
-
-    @include for-mobile {
-      display: inline-block;
-    }
 
-    &.popup {
-      background-color: $sidebar-popup-search-input-background-color;
+        &.icon-cross {
+          display: none;
+          font-size: 20px;
+        }
+      }
     }
 
-    &-icon {
-      color: $sidebar-action-color;
-      font-size: 28px;
-      font-weight: bold;
-      vertical-align: middle;
-    }
+    &.sidebar-opened &-menu-icon {
+      &.icon-menu {
+        display: none;
+      }
 
-    &.popup &-icon {
-      color: $sidebar-popup-search-input-text-color;
+      &.icon-cross {
+        display: inline;
+      }
     }
   }
 

+ 1 - 1
jet/static/jet/js/src/layout-updaters/user-tools.js

@@ -9,7 +9,7 @@ var UserToolsUpdater = function($usertools) {
 
 UserToolsUpdater.prototype = {
     updateUserTools: function($usertools) {
-        var $list = $('<ul>');
+        var $list = $('<ul>').addClass('sidebar-dependent');
         var user = $usertools.find('strong').first().text();
 
         $('<li>')

+ 7 - 7
jet/templates/admin/base.html

@@ -135,16 +135,16 @@
             <span class="icon-refresh loading-indicator"></span>
         </div>
 
-        <div class="sidebar-header sidebar-dependent">
-            <a href="#" class="sidebar-header-menu sidebar-toggle">
-                <span class="sidebar-header-menu-icon icon-menu"></span>
-            </a>
+        <div class="sidebar-header-wrapper sidebar-dependent">
+            <div class="sidebar-header sidebar-dependent">
+                <a href="#" class="sidebar-header-menu sidebar-toggle">
+                    <span class="sidebar-header-menu-icon icon-menu"></span>
+                    <span class="sidebar-header-menu-icon icon-cross"></span>
+                </a>
+            </div>
         </div>
         <div class="sidebar sidebar-dependent">
             <div class="sidebar-wrapper scrollable">
-                <a href="#" class="sidebar-close sidebar-toggle">
-                    <span class="sidebar-close-icon icon-arrow-left"></span>
-                </a>
                 <div class="sidebar-section">
                     {% if user.is_active and user.is_staff %}
                         <a href="{% url 'admin:index' %}" class="sidebar-link icon">