Merge pull request #99 from mrliptontea/fix-sidebar-toggler-in-fixed-…

…layout

Fix sidebar toggler in fixed layout

README.md

@@ -56,6 +56,27 @@ If you need to customize styles for [Redmine Backlogs][redmine_backlogs] remembe
 
 ## Changelog
 
+Latest:
+
+* Fixed sidebar toggler style when fixed layout is enabled
+* Fixed contextual dropdown padding in mobile view
+
+v1.11.0 (2019-03-22):
+
+* Backported fixes from v2:
+  * Fixed #89: [RM+ custom menu](http://rmplus.pro/en/redmine/plugins/custom_menu) breaking the layout
+  * Fixed #90: regression in some sidebar layouts
+  * Fixed #93: Agile chart expanding indefinitely when Additional "Go to top" link is enabled
+  * Fixed #94: subtasks indentation
+  * Fixed styling of some flash messages
+  * Fixed horizontal scrollbar appearing when sidebar is on the right
+  * Fixed footer being mispositioned in Agile charts
+  * Fixed positioning of admin menu icons for some plugins
+  * Improved support for [Redmine Tags](https://www.redmineup.com/pages/plugins/tags) plugin
+  * Added separator line between news on the news list
+  * Improved Redmine 4.0 compatibility
+  * Updated Font Awesome icons to 4.7.0
+
 v2.1.0 (2019-03-22):
 
 * Added CSS grid layout support (off by default because of IE support)

src/sass/components/_elements.scss

@@ -217,7 +217,6 @@ a.collapsible.collapsed,
       }
 
       img.gravatar {
-        position: relative;
         margin-right: .5em;
       }
     }

src/sass/components/_flash.scss

@@ -70,16 +70,16 @@
   padding-left: $padding-large-horizontal;
   text-align: center;
 
-  &::before {
-    margin-left: 0;
-    float: none;
-  }
-
   &.nodata-left {
     &::before {
       float: left;
     }
   }
+
+  &::before {
+    margin-left: 0;
+    float: none;
+  }
 }
 
 #errorExplanation {

src/sass/components/_issue.scss

@@ -246,10 +246,16 @@
   }
 
   div.attachments {
-    margin: -$issue-padding;
-    padding: $issue-padding;
+    margin: 0;
+    padding: 0;
     border-top: 0;
   }
+
+  div.description + div.attachments {
+    margin: $issue-padding (-$issue-padding) (-$issue-padding);
+    padding: $issue-padding;
+    border-top: 1px solid $issue-border;
+  }
 }
 
 div.fileover {
@@ -329,6 +335,10 @@ div.thumbnails {
   .issues {
     margin: ($line-height-computed / 2) 0 0;
 
+    tr {
+      background-color: transparent;
+    }
+
     tr:last-child > td {
       border-bottom: 0 none;
     }

src/sass/components/_plugins.scss

@@ -258,7 +258,6 @@
 }
 
 
-
 //
 // Redmine custom menu
 // http://rmplus.pro/en/redmine/plugins/custom_menu
@@ -319,7 +318,6 @@
 }
 
 
-
 //
 // Redmine CRM People
 // http://www.redminecrm.com/projects/people/

src/sass/components/_responsive.scss

@@ -233,7 +233,7 @@
       border: 0;
 
       .drdn-trigger {
-        padding: ($btn-padding-vertical * 2) $btn-padding-horizontal;
+        padding: $btn-padding-vertical $btn-padding-horizontal;
       }
 
       .drdn-items {

src/sass/javascripts/_sidebar-toggler.scss

@@ -54,16 +54,22 @@
 
     @if $sidebar-position == "left" {
       left: 0;
-      border-left: 0;
-      border-top-left-radius: 0;
-      border-bottom-left-radius: 0;
       background-image: inline-svg("chevron-left.svg", (path: (fill: $gray-700)));
+
+      @if not $fixed-layout {
+        border-left: 0;
+        border-top-left-radius: 0;
+        border-bottom-left-radius: 0;
+      }
     } @else {
       right: 0;
-      border-right: 0;
-      border-top-right-radius: 0;
-      border-bottom-right-radius: 0;
       background-image: inline-svg("chevron-right.svg", (path: (fill: $gray-700)));
+
+      @if not $fixed-layout {
+        border-right: 0;
+        border-top-right-radius: 0;
+        border-bottom-right-radius: 0;
+      }
     }
 
     &.sidebar-hidden {