@primer/css 0.0.0-202111820121 → 0.0.0-2021118211841

package/marketing/support/mixins.scss

@@ -0,0 +1,66 @@
+@mixin btn-solid-mktg($color, $bg, $bg2, $bg3, $bg4) {
+  color: $color;
+  background-color: $bg2;
+  background-image: linear-gradient(-180deg, $bg 0%, $bg2 100%);
+
+  &::before {
+    background-image: linear-gradient(-180deg, $bg3 0%, $bg4 100%);
+  }
+
+  &:hover,
+  &.hover,
+  &:active,
+  &.selected,
+  &[aria-selected=true],
+  [open] > & {
+    background-color: $bg2;
+    background-image: linear-gradient(-180deg, $bg 0%, $bg2 100%);
+  }
+
+  &:focus,
+  &.focus {
+    outline: 0;
+    box-shadow: 0 0 0 0.2em rgba($bg2, 0.4);
+  }
+
+  &:disabled,
+  &.disabled,
+  &[aria-disabled=true] {
+    pointer-events: none;
+    cursor: default;
+    opacity: 0.5;
+  }
+}
+
+@mixin btn-outline-mktg($color, $color2, $border, $borderHover, $borderFocus, $borderFocusInset) {
+  color: $color;
+  background: none;
+  box-shadow: 0 0 0 1px $border inset;
+
+  &::before {
+    display: none;
+  }
+
+  &:hover,
+  &.hover,
+  &:active,
+  &.selected,
+  &[aria-selected=true],
+  [open] > & {
+    color: $color2;
+    background: none;
+    box-shadow: 0 0 0 2px $borderHover inset;
+  }
+
+  &:focus,
+  &.focus {
+    outline: 0;
+    box-shadow: 0 0 0 2px $borderFocus inset, 0 0 0 4px $borderFocusInset;
+  }
+
+  &:disabled,
+  &.disabled,
+  &[aria-disabled=true] {
+    opacity: 0.5;
+  }
+}

package/marketing/support/variables.scss

@@ -1,5 +1,6 @@
+// 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;
@@ -121,7 +122,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;
         }
       }
@@ -113,3 +113,29 @@
 .text-medium {
   font-weight: $font-weight-medium !important;
 }
+
+// Pullquote
+
+@mixin pullquote {
+  padding-top: 0;
+  padding-bottom: 0;
+  padding-left: $spacer;
+  margin-bottom: $spacer-4;
+  font-family: $mono-font;
+  font-size: $h4-size;
+  line-height: 1.4;
+  color: var(--color-color-text-secondary);
+  border-left: 3px solid var(--color-border-default);
+
+  @include breakpoint(md) {
+    padding-left: $spacer * 1.5;
+    margin-bottom: $spacer-5;
+    margin-left: (-$spacer * 1.5) - 3px;
+    font-size: 18px;
+    line-height: $lh-default;
+  }
+}
+
+.pullquote {
+  @include pullquote;
+}

package/marketing/utilities/animations.scss

@@ -7,45 +7,3 @@
     transform: scale3d(1.025, 1.025, 1.025);
   }
 }
-
-// Animated arrow symbol, used in marketing links, buttons, etc.
-.btn-mktg,
-.link-mktg,
-.arrow-target-mktg {
-  .octicon {
-    width: 1em;
-    height: 1em;
-  }
-
-  .arrow-symbol-mktg {
-    transition: transform 0.2s;
-    transform: translateX(0);
-
-    // stylelint-disable-next-line selector-max-type
-    path:last-child {
-      stroke-dasharray: 10;
-      stroke-dashoffset: 10;
-      transition: stroke-dashoffset 0.2s;
-    }
-  }
-
-  @media screen and (prefers-reduced-motion: no-preference) {
-    &:hover,
-    &:focus {
-      .arrow-symbol-mktg {
-        transform: translateX(4px);
-
-        // stylelint-disable-next-line selector-max-type, selector-max-specificity, max-nesting-depth
-        path:last-child {
-          stroke-dashoffset: 20;
-        }
-      }
-    }
-
-    &:active {
-      .arrow-symbol-mktg {
-        transform: translateX(6px);
-      }
-    }
-  }
-}

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,99 +1,50 @@
-$nav-height: $spacer-3 * 3 !default; // 48px
-
 .UnderlineNav {
   display: flex;
-  min-height: $nav-height;
   overflow-x: auto;
   overflow-y: hidden;
   // stylelint-disable-next-line primer/box-shadow
   box-shadow: inset 0 -1px 0 var(--color-border-muted);
-  -webkit-overflow-scrolling: auto;
   justify-content: space-between;
 }
 
 .UnderlineNav-body {
   display: flex;
-  align-items: center;
-  gap: $spacer-2;
-  list-style: none;
 }
 
 .UnderlineNav-item {
-  position: relative;
-  display: flex;
-  padding: 0 $spacer-2;
+  padding: $spacer-2 $spacer-3;
   font-size: $body-font-size;
   // stylelint-disable-next-line primer/typography
   line-height: 30px;
   color: var(--color-fg-default);
   text-align: center;
   white-space: nowrap;
-  cursor: pointer;
   background-color: transparent;
   border: 0;
-  border-radius: $border-radius;
-  align-items: center;
-
-  // renders a visibly hidden "copy" of the label in bold, reserving box space for when label becomes bold on selected
-  [data-content]::before {
-    display: block;
-    height: 0;
-    font-weight: $font-weight-bold;
-    visibility: hidden;
-    content: attr(data-content);
-  }
-
-  // increase touch target area
-  &::before {
-    @include minTouchTarget($min-height: $nav-height);
-  }
+  // stylelint-disable-next-line primer/borders
+  border-bottom: 2px $border-style transparent;
 
-  // hover state was "sticking" on mobile after click
-  @media (pointer: fine) {
-    &:hover {
-      color: var(--color-fg-default);
-      text-decoration: none;
-      background: var(--color-action-list-item-default-hover-bg);
-      transition: background 0.12s ease-out;
-    }
+  &:hover,
+  &:focus {
+    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;
   }
 
   &.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;
-      right: 50%;
-      // 48px total height / 2 (24px) + 1px
-      bottom: calc(50% - 25px);
-      width: 100%;
-      height: 2px;
-      content: '';
-      background: var(--color-primer-border-active);
-      border-radius: $border-radius;
-      transform: translate(50%, -50%);
-    }
-  }
-
-  // remove when global focus state is merged
-  &.focus,
-  &:focus {
-    @include focusOutline;
-    outline-offset: -2px;
-  }
-
-  .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-octicon {
+      color: var(--color-fg-muted);
     }
   }
 }
@@ -112,18 +63,22 @@ $nav-height: $spacer-3 * 3 !default; // 48px
 
 .UnderlineNav--full {
   display: block;
-
-  // required for underline to align with additional wrapper element
-  .UnderlineNav-body {
-    min-height: $nav-height;
-  }
 }
 
 .UnderlineNav-octicon {
-  display: inline !important;
-  margin-right: $spacer-2;
-  color: var(--color-fg-muted);
-  fill: var(--color-fg-muted);
+  margin-right: $spacer-1;
+  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 {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@primer/css",
-  "version": "0.0.0-202111820121",
+  "version": "0.0.0-2021118211841",
   "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,37 +35,37 @@
     "storybook": "cd docs && yarn && yarn storybook"
   },
   "dependencies": {
-    "@primer/primitives": "^7.2.0"
+    "@primer/primitives": "^6.1.0"
   },
   "devDependencies": {
-    "@changesets/changelog-github": "0.4.2",
-    "@changesets/cli": "2.18.1",
+    "@changesets/changelog-github": "0.4.1",
+    "@changesets/cli": "2.17.0",
     "@github/prettier-config": "0.0.4",
     "@koddsson/postcss-sass": "5.0.0",
-    "@primer/stylelint-config": "12.1.1",
+    "@primer/stylelint-config": "12.1.0",
     "autoprefixer": "10.4.0",
     "cssstats": "4.0.2",
-    "eslint": "8.4.0",
-    "eslint-plugin-github": "4.3.5",
-    "eslint-plugin-jest": "25.3.0",
+    "eslint": "8.1.0",
+    "eslint-plugin-github": "4.3.3",
+    "eslint-plugin-jest": "25.2.2",
     "eslint-plugin-prettier": "4.0.0",
     "filesize": "8.0.6",
     "front-matter": "4.0.2",
     "fs-extra": "10.0.0",
     "globby": "12.0.2",
-    "jest": "27.4.3",
+    "jest": "27.3.1",
     "js-yaml": "4.1.0",
-    "postcss": "8.4.4",
+    "postcss": "8.3.11",
     "postcss-calc": "8.0.0",
     "postcss-import": "14.0.2",
     "postcss-load-config": "3.1.0",
     "postcss-scss": "4.0.2",
     "postcss-simple-vars": "6.0.3",
-    "prettier": "2.5.1",
+    "prettier": "2.4.1",
     "semver": "7.3.5",
-    "stylelint": "14.1.0",
+    "stylelint": "14.0.1",
     "stylelint-scss": "4.0.0",
-    "table": "6.7.5"
+    "table": "6.7.2"
   },
   "jest": {
     "testEnvironment": "node",

package/popover/popover.scss

@@ -8,9 +8,6 @@
   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,
@@ -37,13 +34,10 @@
     border: 7px $border-style transparent;
     border-bottom-color: var(--color-canvas-overlay);
   }
-}
 
-// No caret
-.Popover-message--no-caret {
-  &::before,
-  &::after {
-    display: none;
+  // TODO: Refactor so that .Popover is not dependant on .Box
+  &.Box {
+    background-color: var(--color-canvas-overlay);
   }
 }
 
@@ -188,40 +182,8 @@
 }
 
 .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 !important;
-    right: 0 !important;
-    bottom: 0 !important;
-    left: 0 !important;
-  }
-
-  .Popover-message {
-    top: auto;
-    right: auto;
-    bottom: auto;
-    left: auto;
-    width: auto !important;
-    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: "";
   }
 }

package/support/mixins/misc.scss

@@ -10,7 +10,7 @@
     width: 8px;
     height: 16px;
     pointer-events: none;
-    content: ' ';
+    content: " ";
     clip-path: polygon(0 50%, 100% 0, 100% 100%);
   }
 
@@ -24,27 +24,3 @@
     background-color: $border;
   }
 }
-
-// global focus styles
-
-@mixin focusOutline {
-  z-index: 1;
-  outline: 2px solid var(--color-accent-fg);
-  outline-offset: 2px;
-}
-
-// if min-width is undefined, return only min-height
-@mixin minTouchTarget($min-height: 32px, $min-width: '') {
-  position: absolute;
-  top: 50%;
-  left: 50%;
-  width: 100%;
-  height: 100%;
-  min-height: $min-height;
-  content: '';
-  transform: translateX(-50%) translateY(-50%);
-
-  @if $min-width != '' {
-    min-width: $min-width;
-  }
-}

package/support/variables/layout.scss

@@ -150,11 +150,11 @@ $breakpoints: (
 // breakpoints and create both responsive and non-responsive classes in one
 // loop:
 $responsive-variants: (
-  '': '',
-  sm: '-sm',
-  md: '-md',
-  lg: '-lg',
-  xl: '-xl',
+  "": "",
+  sm: "-sm",
+  md: "-md",
+  lg: "-lg",
+  xl: "-xl",
 ) !default;
 
 // responive utility position values
@@ -178,9 +178,8 @@ $sidebar-narrow-width: (
 ) !default;
 
 $sidebar-wide-width: (
-  md: 296px,
   lg: 320px,
-  xl: 344px
+  xl: 336px
 ) !default;
 
 $gutter: (
@@ -199,4 +198,4 @@ $gutter-spacious: (
   md: $spacer-4,
   lg: $spacer-5,
   xl: $spacer-6
-) !default;
+) !default;