Addressing the technical review

Also making minor updates

app/assets/stylesheets/framework/contextual_sidebar_refactoring/contextual_sidebar_variant.scss

 //
 // TEMPORARY OVERRIDES
 // Needed while we serve both *_base and *_variant stylesheets
-// TODO: These have to be removed during the feature flag rollout
+// TODO: These have to be removed during the ':sidebar_refactor' flag rollout
 //
 &.gl-dark .nav-sidebar li.active {
   box-shadow: none;
@@ -19,7 +19,7 @@
 
   li.active {
     background-color: transparent;
-    box-shadow: none !important; // TODO: This should be updated in `theme_helper.scss`
+    box-shadow: none !important; // TODO: This should be updated in `theme_helper.scss` together with ':sidebar_refactor' rollout
   }
 }
 
@@ -138,8 +138,7 @@
 }
 
 @mixin context-header {
-  @include gl-pl-2;
-  @include gl-py-2;
+  @include gl-p-2;
   @include gl-mb-2;
 
   .avatar-container {

app/assets/stylesheets/framework/variables.scss

@@ -573,7 +573,7 @@ $inactive-badge-background: rgba($black, 0.08);
 $sidebar-toggle-height: 60px;
 $sidebar-toggle-width: 40px;
 $sidebar-milestone-toggle-bottom-margin: 10px;
-$sidebar-avatar-size: 32px;
+$sidebar-avatar-size: 34px;
 $sidebar-top-item-lr-margin: 4px;
 $sidebar-top-item-tb-margin: 1px;