@primer/css 19.0.0-rc.dbdedf98 → 19.1.0-rc.c064d682

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@primer/css",
-  "version": "19.0.0-rc.dbdedf98",
+  "version": "19.1.0-rc.c064d682",
   "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,18 +35,18 @@
     "storybook": "cd docs && yarn && yarn storybook"
   },
   "dependencies": {
-    "@primer/primitives": "^7.0.1"
+    "@primer/primitives": "^7.1.0"
   },
   "devDependencies": {
     "@changesets/changelog-github": "0.4.1",
     "@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.2.0",
-    "eslint-plugin-github": "4.3.3",
+    "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",

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: '';
   }
 }

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%);
   }
 

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

package/support/variables/typography.scss

@@ -32,7 +32,7 @@ $lh-condensed: 1.25 !default;
 $lh-default: 1.5 !default;
 
 // Font stacks
-$body-font: -apple-system, BlinkMacSystemFont, "Segoe UI", Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji" !default;
+$body-font: -apple-system, BlinkMacSystemFont, 'Segoe UI', Helvetica, Arial, sans-serif, 'Apple Color Emoji', 'Segoe UI Emoji' !default;
 
 // Monospace font stack
 // Note: SFMono-Regular needs to come before SF Mono to fix an older version of the font in Chrome