@primer/css 0.0.0-2021102331850 → 0.0.0-2021102434749

package/labels/labels.scss

@@ -10,7 +10,6 @@
 .label, // TODO: Deprecte
 .Label {
   @include labels-base;
-  background-color: transparent !important; // TODO: Remove again
   border-color: var(--color-border-default);
 
   &:hover {

package/layout/index.scss

@@ -5,4 +5,3 @@
 @import './grid.scss';
 @import './grid-offset.scss';
 @import './layout.scss';
-@import './layout-beta.scss';

package/marketing/buttons/button.scss

@@ -2,95 +2,137 @@
   position: relative;
   z-index: 1;
   display: inline-block;
-  appearance: none !important;
   // stylelint-disable-next-line primer/spacing
   padding: 0.9rem 1.5rem 1.1rem;
   // stylelint-disable-next-line primer/typography
   font-size: 1rem;
   font-weight: $font-weight-bold;
   line-height: 1;
+  color: var(--color-canvas-default);
+  text-align: center;
   white-space: nowrap;
   vertical-align: middle;
   user-select: none;
+  background: linear-gradient(180deg, rgba(255 255 255 / 0.15) 0%, rgba(255 255 255 / 0) 100%), var(--color-mktg-btn-bg) !important;
   border: 0;
   // stylelint-disable-next-line primer/borders
   border-radius: 0.375rem;
-
-  @include btn-solid-mktg(
-    var(--color-mktg-btn-text),
-    var(--color-mktg-btn-bg-top),
-    var(--color-mktg-btn-bg-bottom),
-    var(--color-mktg-btn-bg-overlay-top),
-    var(--color-mktg-btn-bg-overlay-bottom),
-  );
+  transition: box-shadow 0.2s;
+  appearance: none !important;
 
   &::before {
-    background-blend-mode: overlay, normal;
     position: absolute;
     top: 0;
     right: 0;
     bottom: 0;
     left: 0;
     z-index: -1;
-    content: "";
-    // stylelint-disable-next-line primer/borders
-    border-radius: 0.375rem;
+    content: '';
+    // stylelint-disable-next-line primer/colors
+    background: linear-gradient(180deg, rgba(255 255 255 / 0.15) 0%, rgba(255 255 255 / 0) 100%) !important;
+    border-radius: inherit;
     opacity: 0;
-    transition: opacity 0.4s;
+    opacity: 0;
+    transition: opacity 0.2s;
+    background-blend-mode: normal;
   }
 
   &:hover {
-    text-decoration: none;
+    box-shadow: var(--color-mktg-btn-shadow-hover) !important;
+  }
 
+  &:focus,
+  &.focus {
+    outline: 0;
+    box-shadow: var(--color-mktg-btn-shadow-focus), var(--color-mktg-btn-shadow-hover) !important;
+  }
+
+  &:hover,
+  &:focus,
+  &.focus {
     &::before {
       opacity: 1;
-      transition: opacity 0.4s;
+    }
+
+    &:disabled {
+      box-shadow: none !important;
     }
   }
-}
 
-.btn-primary-mktg {
-  @include btn-solid-mktg(
-    var(--color-mktg-btn-primary-text),
-    var(--color-mktg-btn-primary-bg-top),
-    var(--color-mktg-btn-primary-bg-bottom),
-    var(--color-mktg-btn-primary-bg-overlay-top),
-    var(--color-mktg-btn-primary-bg-overlay-bottom),
-  );
+  &:active {
+    outline: none;
+    // stylelint-disable-next-line primer/box-shadow
+    box-shadow: 0 0 0 transparent;
+
+    &::before {
+      opacity: 0.5 !important;
+    }
+  }
 }
 
-.btn-enterprise-mktg {
-  @include btn-solid-mktg(
-    var(--color-mktg-btn-enterprise-text),
-    var(--color-mktg-btn-enterprise-bg-top),
-    var(--color-mktg-btn-enterprise-bg-bottom),
-    var(--color-mktg-btn-enterprise-bg-overlay-top),
-    var(--color-mktg-btn-enterprise-bg-overlay-bottom),
-  );
+.btn-muted-mktg {
+  color: var(--color-fg-default) !important;
+  background: none !important;
+  box-shadow: var(--color-mktg-btn-shadow-outline);
+
+  &::before {
+    display: none;
+  }
+
+  &:hover {
+    box-shadow: var(--color-mktg-btn-shadow-hover-muted) !important;
+  }
+
+  &:focus,
+  &.focus {
+    box-shadow: var(--color-mktg-btn-shadow-hover-muted), var(--color-mktg-btn-shadow-focus) !important;
+  }
+
+  &:active {
+    // stylelint-disable-next-line primer/box-shadow
+    box-shadow: var(--color-fg-default) 0 0 0 3px inset !important;
+  }
+
+  &:disabled {
+    // stylelint-disable-next-line primer/box-shadow
+    box-shadow: var(--color-fg-subtle) 0 0 0 1px inset !important;
+  }
 }
 
-.btn-outline-mktg {
-  @include btn-outline-mktg(
-    var(--color-mktg-btn-outline-text),
-    var(--color-mktg-btn-outline-hover-text),
-    var(--color-mktg-btn-outline-border),
-    var(--color-mktg-btn-outline-hover-border),
-    var(--color-mktg-btn-outline-focus-border),
-    var(--color-mktg-btn-outline-focus-border-inset)
-  );
-  transition: box-shadow 0.4s, color 0.4s;
+.btn-subtle-mktg {
+  color: var(--color-fg-default) !important;
+  background: none !important;
+  box-shadow: none !important;
+
+  &::before {
+    background: none !important;
+  }
+
+  &:hover {
+    box-shadow: var(--color-mktg-btn-shadow-hover-muted) !important;
+  }
+
+  &:focus,
+  .focus {
+    box-shadow: var(--color-mktg-btn-shadow-hover-muted), var(--color-mktg-btn-shadow-focus) !important;
+  }
 }
 
-.btn-transparent {
-  @include btn-outline-mktg(
-    var(--color-mktg-btn-dark-text),
-    var(--color-mktg-btn-dark-hover-text),
-    var(--color-mktg-btn-dark-border),
-    var(--color-mktg-btn-dark-hover-border),
-    var(--color-mktg-btn-dark-focus-border),
-    var(--color-mktg-btn-dark-focus-border-inset)
-  );
-  transition: box-shadow 0.4s, color 0.4s;
+.btn-signup-mktg {
+  // stylelint-disable-next-line primer/colors
+  color: #fff;
+  // stylelint-disable-next-line primer/colors
+  background: linear-gradient(180deg, rgba(52, 183, 89, 0.15) 0%, rgba(46, 164, 79, 0) 100%), rgb(46, 164, 79) !important;
+
+  &::before {
+    // stylelint-disable-next-line primer/colors
+    background: linear-gradient(180deg, rgba(52, 183, 89, 0.15) 0%, rgba(46, 164, 79, 0) 100%) !important;
+  }
+
+  &:focus {
+    // stylelint-disable-next-line primer/box-shadow
+    box-shadow: rgba(46, 164, 79, 0.45) 0 0 0 4px !important;
+  }
 }
 
 // Size modifiers

package/marketing/buttons/index.scss

@@ -1,3 +1,3 @@
 // support files
-@import "../support/index.scss";
-@import "./button.scss";
+@import '../support/index.scss';
+@import './button.scss';

package/marketing/support/index.scss

@@ -1,3 +1,2 @@
-@import "../../support/index.scss";
-@import "./mixins.scss";
-@import "./variables.scss";
+@import '../../support/index.scss';
+@import './variables.scss';

package/marketing/support/variables.scss

@@ -1,6 +1,5 @@
-// stylelint-disable unit-no-unknown
 // Typography
-$marketing-font-path: "/fonts/" !default;
+$marketing-font-path: '/fonts/' !default;
 
 $font-mktg: $body-font !default;
 $font-weight-medium: 450 !default;
@@ -122,7 +121,7 @@ $transition-time: 0.4s !default;
 $ease-mktg: cubic-bezier(0.16, 1, 0.3, 1) !default;
 
 $marketing-position-variants: (
-  "": "",
-  md: "-md",
-  lg: "-lg",
+  '': '',
+  md: '-md',
+  lg: '-lg',
 ) !default;

package/marketing/type/index.scss

@@ -1,3 +1,3 @@
 // support files
-@import "../support/index.scss";
-@import "./typography.scss";
+@import '../support/index.scss';
+@import './typography.scss';

package/marketing/type/typography.scss

@@ -19,20 +19,20 @@
     $pairing-md: map-get($mktg-header-pairings, nth($sizes, 2));
     $pairing-lg: map-get($mktg-header-pairings, nth($sizes, 3));
 
-    font-size: map-get($pairing, "size") !important;
-    line-height: map-get($pairing, "lh") !important;
-    @if (map-get($pairing, "size") >= $mktg-header-weight-threshold) { font-weight: $mktg-header-weight-large !important; }
+    font-size: map-get($pairing, 'size') !important;
+    line-height: map-get($pairing, 'lh') !important;
+    @if (map-get($pairing, 'size') >= $mktg-header-weight-threshold) { font-weight: $mktg-header-weight-large !important; }
 
     @if (nth($sizes, 1) != nth($sizes, 2)) {
       @include breakpoint(md) {
-        font-size: map-get($pairing-md, "size") !important;
-        line-height: map-get($pairing-md, "lh") !important;
+        font-size: map-get($pairing-md, 'size') !important;
+        line-height: map-get($pairing-md, 'lh') !important;
 
-        @if (map-get($pairing-md, "size") >= $mktg-header-spacing-threshold and map-get($pairing, "size") < $mktg-header-spacing-threshold) {
+        @if (map-get($pairing-md, 'size') >= $mktg-header-spacing-threshold and map-get($pairing, 'size') < $mktg-header-spacing-threshold) {
           letter-spacing: $mktg-header-spacing-large !important;
         }
 
-        @if (map-get($pairing-md, "size") >= $mktg-header-weight-threshold and map-get($pairing, "size") < $mktg-header-weight-threshold) {
+        @if (map-get($pairing-md, 'size') >= $mktg-header-weight-threshold and map-get($pairing, 'size') < $mktg-header-weight-threshold) {
           font-weight: $mktg-header-weight-large !important;
         }
       }
@@ -40,14 +40,14 @@
 
     @if (nth($sizes, 2) != nth($sizes, 3)) {
       @include breakpoint(lg) {
-        font-size: map-get($pairing-lg, "size") !important;
-        line-height: map-get($pairing-lg, "lh") !important;
+        font-size: map-get($pairing-lg, 'size') !important;
+        line-height: map-get($pairing-lg, 'lh') !important;
 
-        @if (map-get($pairing-lg, "size") >= $mktg-header-spacing-threshold and map-get($pairing-md, "size") < $mktg-header-spacing-threshold) {
+        @if (map-get($pairing-lg, 'size') >= $mktg-header-spacing-threshold and map-get($pairing-md, 'size') < $mktg-header-spacing-threshold) {
           letter-spacing: $mktg-header-spacing-large !important;
         }
 
-        @if (map-get($pairing-lg, "size") >= $mktg-header-weight-threshold and map-get($pairing-md, "size") < $mktg-header-weight-threshold) {
+        @if (map-get($pairing-lg, 'size') >= $mktg-header-weight-threshold and map-get($pairing-md, 'size') < $mktg-header-weight-threshold) {
           font-weight: $mktg-header-weight-large !important;
         }
       }
@@ -73,21 +73,21 @@
     $pairing-md: map-get($mktg-body-pairings, nth($sizes, 2));
     $pairing-lg: map-get($mktg-body-pairings, nth($sizes, 3));
 
-    font-size: map-get($pairing, "size") !important;
-    line-height: map-get($pairing, "lh") !important;
-    @if (map-get($pairing, "size") >= $mktg-body-spacing-threshold) { letter-spacing: $mktg-body-spacing-large !important; }
-    @if (map-get($pairing, "size") >= $mktg-body-weight-threshold) { font-weight: $font-weight-semibold; }
+    font-size: map-get($pairing, 'size') !important;
+    line-height: map-get($pairing, 'lh') !important;
+    @if (map-get($pairing, 'size') >= $mktg-body-spacing-threshold) { letter-spacing: $mktg-body-spacing-large !important; }
+    @if (map-get($pairing, 'size') >= $mktg-body-weight-threshold) { font-weight: $font-weight-semibold; }
 
     @if (nth($sizes, 1) != nth($sizes, 2)) {
       @include breakpoint(md) {
-        font-size: map-get($pairing-md, "size") !important;
-        line-height: map-get($pairing-md, "lh") !important;
+        font-size: map-get($pairing-md, 'size') !important;
+        line-height: map-get($pairing-md, 'lh') !important;
 
-        @if (map-get($pairing-md, "size") >= $mktg-body-spacing-threshold and map-get($pairing, "size") < $mktg-body-spacing-threshold) {
+        @if (map-get($pairing-md, 'size') >= $mktg-body-spacing-threshold and map-get($pairing, 'size') < $mktg-body-spacing-threshold) {
           letter-spacing: $mktg-body-spacing-large !important;
         }
 
-        @if (map-get($pairing-md, "size") >= $mktg-body-weight-threshold and map-get($pairing, "size") < $mktg-body-weight-threshold) {
+        @if (map-get($pairing-md, 'size') >= $mktg-body-weight-threshold and map-get($pairing, 'size') < $mktg-body-weight-threshold) {
           font-weight: $font-weight-medium;
         }
       }
@@ -95,14 +95,14 @@
 
     @if (nth($sizes, 2) != nth($sizes, 3)) {
       @include breakpoint(lg) {
-        font-size: map-get($pairing-lg, "size") !important;
-        line-height: map-get($pairing-lg, "lh") !important;
+        font-size: map-get($pairing-lg, 'size') !important;
+        line-height: map-get($pairing-lg, 'lh') !important;
 
-        @if (map-get($pairing-lg, "size") >= $mktg-body-spacing-threshold and map-get($pairing-md, "size") < $mktg-body-spacing-threshold) {
+        @if (map-get($pairing-lg, 'size') >= $mktg-body-spacing-threshold and map-get($pairing-md, 'size') < $mktg-body-spacing-threshold) {
           letter-spacing: $mktg-body-spacing-large !important;
         }
 
-        @if (map-get($pairing-lg, "size") >= $mktg-body-weight-threshold and map-get($pairing-md, "size") < $mktg-body-weight-threshold) {
+        @if (map-get($pairing-lg, 'size') >= $mktg-body-weight-threshold and map-get($pairing-md, 'size') < $mktg-body-weight-threshold) {
           font-weight: $font-weight-medium;
         }
       }

package/marketing/utilities/index.scss

@@ -1,6 +1,6 @@
-@import "../support/index.scss";
+@import '../support/index.scss';
 // utilities
-@import "./animations.scss";
-@import "./borders.scss";
-@import "./filters.scss";
-@import "./layout.scss";
+@import './animations.scss';
+@import './borders.scss';
+@import './filters.scss';
+@import './layout.scss';

package/marketing/utilities/layout.scss

@@ -6,7 +6,7 @@
 @each $breakpoint, $variant in $marketing-position-variants {
   @include breakpoint($breakpoint) {
     @each $scale, $size in $spacer-map-extended {
-      @if ($size != 0 or $variant != "") {
+      @if ($size != 0 or $variant != '') {
         .top#{$variant}-#{$scale}     { top: $size !important; }
         .right#{$variant}-#{$scale}   { right: $size !important; }
         .bottom#{$variant}-#{$scale}  { bottom: $size !important; }

package/navigation/underline-nav.scss

@@ -1,18 +1,21 @@
+$nav-height: $spacer-3 * 3; // 48px
+
 .UnderlineNav {
   display: flex;
-  overflow-x: auto;
-  overflow-y: hidden;
   // stylelint-disable-next-line primer/box-shadow
   box-shadow: inset 0 -1px 0 var(--color-border-muted);
   justify-content: space-between;
+  min-height: $nav-height;
 }
 
 .UnderlineNav-body {
   display: flex;
+  align-items: center;
+  gap: $spacer-2;
 }
 
 .UnderlineNav-item {
-  padding: $spacer-2 $spacer-3;
+  padding: 0 $spacer-2;
   font-size: $body-font-size;
   // stylelint-disable-next-line primer/typography
   line-height: 30px;
@@ -21,32 +24,63 @@
   white-space: nowrap;
   background-color: transparent;
   border: 0;
-  // stylelint-disable-next-line primer/borders
-  border-bottom: 2px $border-style transparent;
+  border-radius: $border-radius;
+  position: relative;
 
-  &:hover,
-  &:focus {
+  // increase touch target area
+  &::before {
+    @include minTouchTarget($min-height: $nav-height);
+  }
+
+  &:hover {
     color: var(--color-fg-default);
     text-decoration: none;
-    border-bottom-color: var(--color-neutral-muted);
-    outline: 1px dotted transparent; // Support Firefox custom colors
-    outline-offset: -1px;
-    transition: border-bottom-color 0.12s ease-out;
+    background: var(--color-neutral-muted);
+    transition: background 0.12s ease-out;
   }
 
   &.selected,
-  &[role=tab][aria-selected=true],
-  &[aria-current]:not([aria-current=false]) {
+  &[role='tab'][aria-selected='true'],
+  &[aria-current]:not([aria-current='false']) {
     font-weight: $font-weight-bold;
     color: var(--color-fg-default);
-    border-bottom-color: var(--color-primer-border-active);
-    outline: 1px dotted transparent; // Support Firefox custom colors
-    outline-offset: -1px;
+
+    // current/selected underline
+    &:after {
+      position: absolute;
+      width: 100%;
+      height: 2px;
+      content: '';
+      background: var(--color-primer-border-active);
+      border-radius: $border-radius;
+      right: 50%;
+      // 48px total height / 2 (24px) + 1px
+      bottom: calc(50% - 25px);
+      transform: translate(50%, -50%);
+    }
 
     .UnderlineNav-octicon {
       color: var(--color-fg-muted);
     }
   }
+
+  // remove when global focus state is merged
+  &.focus,
+  &:focus {
+    @include focusOutline;
+    outline-offset: 0;
+  }
+
+  .Counter {
+    margin-left: $spacer-2;
+    color: var(--color-fg-default);
+    background-color: var(--color-neutral-muted);
+
+    &--primary {
+      color: var(--color-fg-on-emphasis);
+      background-color: var(--color-neutral-emphasis);
+    }
+  }
 }
 
 .UnderlineNav--right {
@@ -63,24 +97,18 @@
 
 .UnderlineNav--full {
   display: block;
+
+  // required for underline to align with additional wrapper element
+  .UnderlineNav-body {
+    min-height: $nav-height;
+  }
 }
 
 .UnderlineNav-octicon {
-  margin-right: $spacer-1;
+  margin-right: $spacer-2;
   color: var(--color-fg-subtle);
 }
 
-.UnderlineNav .Counter {
-  margin-left: $spacer-1;
-  color: var(--color-fg-default);
-  background-color: var(--color-neutral-muted);
-
-  &--primary {
-    color: var(--color-fg-on-emphasis);
-    background-color: var(--color-neutral-emphasis);
-  }
-}
-
 .UnderlineNav-container {
   display: flex;
   justify-content: space-between;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@primer/css",
-  "version": "0.0.0-2021102331850",
+  "version": "0.0.0-2021102434749",
   "description": "The CSS implementation of GitHub's Primer Design System",
   "homepage": "https://primer.style/css",
   "author": "GitHub, Inc.",
@@ -24,7 +24,7 @@
   ],
   "scripts": {
     "dist": "script/dist.js",
-    "stylelint": "stylelint --quiet src/**/*.scss",
+    "stylelint": "stylelint --quiet 'src/**/*.scss'",
     "eslint": "eslint deprecations.js __tests__ script",
     "prepublishOnly": "script/prepublish",
     "start": "yarn dev",
@@ -35,19 +35,19 @@
     "storybook": "cd docs && yarn && yarn storybook"
   },
   "dependencies": {
-    "@primer/primitives": "^6.1.0"
+    "@primer/primitives": "^7.1.0"
   },
   "devDependencies": {
     "@changesets/changelog-github": "0.4.1",
-    "@changesets/cli": "2.17.0",
+    "@changesets/cli": "2.18.0",
     "@github/prettier-config": "0.0.4",
     "@koddsson/postcss-sass": "5.0.0",
-    "@primer/stylelint-config": "12.1.0",
+    "@primer/stylelint-config": "12.1.1",
     "autoprefixer": "10.4.0",
     "cssstats": "4.0.2",
-    "eslint": "8.1.0",
-    "eslint-plugin-github": "4.3.3",
-    "eslint-plugin-jest": "25.2.2",
+    "eslint": "8.3.0",
+    "eslint-plugin-github": "4.3.5",
+    "eslint-plugin-jest": "25.2.4",
     "eslint-plugin-prettier": "4.0.0",
     "filesize": "8.0.6",
     "front-matter": "4.0.2",
@@ -63,9 +63,9 @@
     "postcss-simple-vars": "6.0.3",
     "prettier": "2.4.1",
     "semver": "7.3.5",
-    "stylelint": "14.0.1",
+    "stylelint": "14.1.0",
     "stylelint-scss": "4.0.0",
-    "table": "6.7.2"
+    "table": "6.7.3"
   },
   "jest": {
     "testEnvironment": "node",

package/popover/popover.scss

@@ -8,6 +8,9 @@
   width: 232px;
   margin-right: auto;
   margin-left: auto;
+  background-color: var(--color-canvas-overlay);
+  border: $border-width $border-style var(--color-border-default);
+  border-radius: $border-radius;
 
   // Carets
   &::before,
@@ -34,10 +37,13 @@
     border: 7px $border-style transparent;
     border-bottom-color: var(--color-canvas-overlay);
   }
+}
 
-  // TODO: Refactor so that .Popover is not dependant on .Box
-  &.Box {
-    background-color: var(--color-canvas-overlay);
+// No caret
+.Popover-message--no-caret {
+  &::before,
+  &::after {
+    display: none;
   }
 }
 
@@ -182,8 +188,40 @@
 }
 
 .Popover-message--large {
-
   @include breakpoint(sm) {
     min-width: 320px;
   }
 }
+
+// Responsive Popover
+// For < md it will show full-width anchored to the bottom
+
+@media (max-width: ($width-md - 1px)) {
+  .Popover {
+    position: fixed;
+    top: auto;
+    right: 0;
+    bottom: 0;
+    left: 0;
+  }
+
+  .Popover-message {
+    top: auto;
+    right: auto;
+    bottom: auto;
+    left: auto;
+    width: auto;
+    margin: $spacer-2;
+  }
+
+  // Increase tap area for touch input
+  .Popover-message > .btn-octicon {
+    padding: $spacer-2 + $spacer-1 !important;
+  }
+
+  // Remove caret
+  .Popover-message::after,
+  .Popover-message::before {
+    display: none;
+  }
+}

package/support/mixins/layout.scss

@@ -1,7 +1,7 @@
 // Responsive media queries
 
 @mixin breakpoint($breakpoint) {
-  @if $breakpoint == "" {
+  @if $breakpoint == '' {
     @content;
   }
 
@@ -19,8 +19,8 @@
 
     // If the key doesn't exist in the map
     @else {
-      @warn "Unfortunately, no value could be retrieved from `#{$breakpoint}`. "
-        + "Please make sure it is defined in `$breakpoints` map.";
+      @warn 'Unfortunately, no value could be retrieved from `#{$breakpoint}`. '
+        + 'Please make sure it is defined in `$breakpoints` map.';
     }
   }
 }
@@ -47,12 +47,12 @@
 @mixin clearfix {
   &::before {
     display: table;
-    content: "";
+    content: '';
   }
 
   &::after {
     display: table;
     clear: both;
-    content: "";
+    content: '';
   }
 }