@primer/css 20.0.0-rc.b5fee637 → 20.0.0-rc.c0ed9d90

package/forms/form-validation.scss

@@ -235,15 +235,20 @@ dl.form-group > dd, // TODO: Deprecate
   }
 
   &.focused {
+    border-color: var(--color-accent-fg);
     border-radius: $border-radius;
-    box-shadow: var(--color-primer-shadow-inset), var(--color-primer-shadow-focus);
+    outline: none;
+    // stylelint-disable-next-line primer/box-shadow
+    box-shadow: 0 0 0 2px var(--color-accent-fg);
 
     .form-control {
+      border-color: transparent;
+      border-bottom-color: var(--color-accent-fg);
       box-shadow: none;
     }
 
     .drag-and-drop {
-      border-color: var(--color-accent-emphasis);
+      border-color: transparent;
     }
   }
 }
@@ -272,7 +277,9 @@ dl.form-group > dd, // TODO: Deprecate
     border: $border-width $border-style var(--color-border-default);
   }
 
-  .comment-form-error { margin-bottom: $spacer-2; }
+  .comment-form-error {
+    margin-bottom: $spacer-2;
+  }
 
   .write-content,
   .preview-content {
@@ -295,7 +302,7 @@ dl.form-group > dd, // TODO: Deprecate
   }
 }
 
-// Used in our boxed-group-less form styles. Give the sumbit button enough space
+// Used in our boxed-group-less form styles. Give the submit button enough space
 // to breathe without the need for the extra hr.
 .form-action-spacious {
   // stylelint-disable-next-line primer/spacing
@@ -317,8 +324,10 @@ div.composer {
   min-height: 200px;
 }
 
-// stylelint-disable-next-line primer/spacing
-.composer .tabnav { margin: 0 0 10px; }
+.composer .tabnav {
+  // stylelint-disable-next-line primer/spacing
+  margin: 0 0 10px;
+}
 
 // Misc CSS
 //

package/forms/input-group.scss

@@ -18,6 +18,20 @@
   &.inline {
     display: inline-table;
   }
+
+  // within input group, if button exists change focus styles to match input (no offset)
+  &:focus-within {
+    // stylelint-disable-next-line selector-max-type
+    button {
+      outline-offset: 0;
+    }
+  }
+
+  // Autocomplete with embedded icon
+  .form-control.autocomplete-embedded-icon-wrap {
+    display: inline-flex;
+    padding: $spacer-1 * 1.25 $spacer-2;
+  }
 }
 
 .input-group .form-control,
@@ -30,6 +44,10 @@
   vertical-align: middle; // Match the inputs
 }
 
+.input-group-button--autocomplete-embedded-icon {
+  vertical-align: bottom;
+}
+
 .input-group .form-control:first-child,
 .input-group-button:first-child .btn {
   border-top-right-radius: 0;

package/labels/index.scss

@@ -1,6 +1,5 @@
 @import '../support/index.scss';
 @import './mixins.scss';
-
 @import './issue-labels.scss';
 @import './labels.scss';
 @import './states.scss';

package/labels/labels.scss

@@ -7,9 +7,10 @@
 
 // Default label
 
-.label, // TODO: Deprecte
+.label, // TODO: Deprecate
 .Label {
   @include labels-base;
+
   border-color: var(--color-border-default);
 
   &:hover {
@@ -70,6 +71,16 @@
   border-color: var(--color-danger-emphasis);
 }
 
+.Label--open {
+  color: var(--color-open-fg);
+  border-color: var(--color-open-emphasis);
+}
+
+.Label--closed {
+  color: var(--color-closed-fg);
+  border-color: var(--color-closed-emphasis);
+}
+
 .Label--done {
   color: var(--color-done-fg);
   border-color: var(--color-done-emphasis);

package/labels/mixins.scss

@@ -6,16 +6,20 @@
 
 @mixin labels-base {
   display: inline-block;
+  // stylelint-disable-next-line primer/spacing
   padding: 0 7px;
   font-size: $font-size-small;
   font-weight: $font-weight-semibold;
   line-height: 18px;
+  white-space: nowrap;
   border: $border-width $border-style transparent;
   border-radius: 2em;
 }
 
 @mixin labels-large {
+  // stylelint-disable-next-line primer/spacing
   padding-right: 10px;
+  // stylelint-disable-next-line primer/spacing
   padding-left: 10px;
   line-height: 22px;
 }
@@ -27,6 +31,7 @@
 
 @mixin labels--inline {
   display: inline;
-  padding: 0.1667em 0.5em;
-  font-size: 0.9em;
+  // stylelint-disable-next-line primer/spacing
+  padding: 0.12em $em-spacer-5;
+  font-size: 85%;
 }

package/labels/states.scss

@@ -28,7 +28,7 @@
 
 .State--open {
   color: var(--color-fg-on-emphasis);
-  background-color: var(--color-success-emphasis);
+  background-color: var(--color-open-emphasis);
 }
 
 .State--merged {
@@ -38,7 +38,7 @@
 
 .State--closed {
   color: var(--color-fg-on-emphasis);
-  background-color: var(--color-danger-emphasis);
+  background-color: var(--color-closed-emphasis);
 }
 
 // Small

package/layout/grid-offset.scss

@@ -2,6 +2,7 @@
 
 // Offset Columns
 
+// stylelint-disable primer/spacing
 @each $breakpoint, $variant in $responsive-variants {
   @include breakpoint($breakpoint) {
     .offset#{$variant}-1 { margin-left: (1 * 0.0833333333 * 100%) !important; }

package/layout/grid.scss

@@ -33,6 +33,7 @@
 
 // Gutters
 // Apply padding and a negative margin to the outside of the container
+// stylelint-disable primer/spacing
 @mixin gutters ($gutter-width: $spacer-3) {
   margin-right: -$gutter-width;
   margin-left: -$gutter-width;

package/layout/index.scss

@@ -1,7 +1,7 @@
 @import '../support/index.scss';
 @import './mixins.scss';
-
 @import './container.scss';
 @import './grid.scss';
 @import './grid-offset.scss';
 @import './layout.scss';
+@import './page-layout.scss';

package/layout/layout.scss

@@ -2,21 +2,22 @@
 
 .Layout {
   display: grid;
+
   --Layout-sidebar-width: #{map-get($sidebar-width, 'sm')};
   --Layout-gutter: 16px;
 
-  @media (max-width: calc(#{$width-sm} - 1px)) {
+  @media (max-width: calc(#{$width-sm} - 0.02px)) {
     @include flow-as-row;
   }
 
   &.Layout--flowRow-until-md {
-    @media (max-width: calc(#{$width-md} - 1px)) {
+    @media (max-width: calc(#{$width-md} - 0.02px)) {
       @include flow-as-row;
     }
   }
 
   &.Layout--flowRow-until-lg {
-    @media (max-width: calc(#{$width-lg} - 1px)) {
+    @media (max-width: calc(#{$width-lg} - 0.02px)) {
       @include flow-as-row;
     }
   }
@@ -24,8 +25,7 @@
   // Flow as column
 
   grid-auto-flow: column;
-  grid-template-columns: auto 0 1fr; // sidebar column, separator, main column
-  // stylelint-disable-next-line primer/no-undefined-vars
+  grid-template-columns: auto 0 minmax(0, calc(100% - var(--Layout-sidebar-width) - var(--Layout-gutter))); // sidebar column, separator, main column
   grid-gap: var(--Layout-gutter);
 
   .Layout-sidebar {
@@ -47,7 +47,8 @@
   }
 
   &.Layout--gutter-none {
-    --Layout-gutter: 0;
+    // stylelint-disable-next-line length-zero-no-unit
+    --Layout-gutter: 0px;
   }
 
   &.Layout--gutter-condensed {
@@ -101,7 +102,7 @@
   }
 
   &.Layout--sidebarPosition-end {
-    grid-template-columns: 1fr 0 auto;
+    grid-template-columns: minmax(0, calc(100% - var(--Layout-sidebar-width) - var(--Layout-gutter))) 0 auto;
 
     .Layout-main {
       grid-column: 1;
@@ -146,7 +147,6 @@
 }
 
 .Layout-sidebar {
-  // stylelint-disable-next-line primer/no-undefined-vars
   width: var(--Layout-sidebar-width);
 }
 
@@ -169,17 +169,14 @@
   }
 
   .Layout-main-centered-md {
-    // stylelint-disable-next-line primer/no-undefined-vars
     max-width: calc(#{$container-md} + var(--Layout-sidebar-width) + var(--Layout-gutter));
   }
 
   .Layout-main-centered-lg {
-    // stylelint-disable-next-line primer/no-undefined-vars
     max-width: calc(#{$container-lg} + var(--Layout-sidebar-width) + var(--Layout-gutter));
   }
 
   .Layout-main-centered-xl {
-    // stylelint-disable-next-line primer/no-undefined-vars
     max-width: calc(#{$container-xl} + var(--Layout-sidebar-width) + var(--Layout-gutter));
   }
 }

package/layout/mixins.scss

@@ -1,4 +1,4 @@
-// Layout mixins
+// Layout alpha mixins
 
 @mixin flow-as-row {
   grid-auto-flow: row;
@@ -7,9 +7,9 @@
   .Layout-sidebar,
   .Layout-divider,
   .Layout-main {
-      width: 100% !important;
-      grid-column: 1 !important;
-    }
+    width: 100% !important;
+    grid-column: 1 !important;
+  }
 
   &.Layout--sidebarPosition-flowRow-start {
     .Layout-sidebar {
@@ -77,3 +77,29 @@
     }
   }
 }
+
+// Layout beta mixins
+
+// responsive region dividers
+
+@mixin Layout-line-divider {
+  position: absolute;
+  left: calc(var(--Layout-outer-spacing-x) * -1);
+  display: block;
+  width: calc(100% + (var(--Layout-outer-spacing-x) * 2));
+  height: 1px;
+  content: '';
+  background-color: $Layout-divider-color;
+}
+
+@mixin Layout-filled-divider {
+  position: absolute;
+  bottom: calc(#{$spacer-2} * -1); // -8px
+  left: calc(var(--Layout-outer-spacing-x) * -1);
+  display: block;
+  width: calc(100% + (var(--Layout-outer-spacing-x) * 2));
+  height: #{$spacer-2}; // 8px
+  content: '';
+  background-color: var(--color-canvas-inset);
+  box-shadow: inset 0 1px $Layout-divider-color, inset 0 -1px $Layout-divider-color;
+}