Fixed theming issues with Grid's Sidebar (#17412)

- if columns hidable when grid rendered, sidebar button had invalid height
- removed border radius from grid sidebar
- inherit font color for grid sidebar button

Change-Id: Ib35d917b35e701bd736a28f19b34dbece7fcbe71

1 files changed, 3 insertions, 2 deletions

diff --git a/WebContent/VAADIN/themes/base/grid/grid.scss b/WebContent/VAADIN/themes/base/grid/grid.scss
index 838c0de502..ee503822b9 100644
--- a/WebContent/VAADIN/themes/base/grid/grid.scss
+++ b/WebContent/VAADIN/themes/base/grid/grid.scss
@@ -90,6 +90,7 @@ $v-grid-details-border-bottom-stripe: 1px solid darken($v-grid-row-background-co
 
   .#{$primaryStyleName}-sidebar.v-contextmenu {
     @include box-shadow(none);
+    border-radius: 0;
     position: absolute;
     top: 0;
     right: 0;
@@ -102,6 +103,7 @@ $v-grid-details-border-bottom-stripe: 1px solid darken($v-grid-row-background-co
     .#{$primaryStyleName}-sidebar-button {
       background: transparent;
       border: none;
+      color: inherit;
       cursor: pointer;
       outline: none;
       padding: 0 4px;
@@ -133,7 +135,7 @@ $v-grid-details-border-bottom-stripe: 1px solid darken($v-grid-row-background-co
           line-height: 1;
         }
       }
-    }      
+    }
 
     .v-ie &.opened .#{$primaryStyleName}-sidebar-button {
       vertical-align: middle;
@@ -144,7 +146,6 @@ $v-grid-details-border-bottom-stripe: 1px solid darken($v-grid-row-background-co
     }
 
     .#{$primaryStyleName}-sidebar-content {
-      background: #fff;
       border-top: $v-grid-border;
       padding: 4px 0;