Release Tracking

.github/workflows/next_major.yml

@@ -0,0 +1,14 @@
+name: Next Major Pull Request
+on:
+  push:
+    branches:
+      - 'changeset-release/next_major'
+
+jobs:
+  next_major_pr:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Don't Merge
+        run: |
+          echo "Don't merge the next_major, changeset pr into next_major. Instead when you're ready to release a new major version, change the base of this pr to main, and merge."
+          exit 1

CHANGELOG.md

@@ -1,5 +1,29 @@
 # @primer/css
 
+## 20.0.0
+
+### Major Changes
+
+- [#2049](https://github.com/primer/css/pull/2049) [`f4dba96e`](https://github.com/primer/css/commit/f4dba96e0cb78d3d451226cf60b01187678ced45) Thanks [@langermank](https://github.com/langermank)! - Comment box upload focus border-radius
+
+* [#1744](https://github.com/primer/css/pull/1744) [`942f65a4`](https://github.com/primer/css/commit/942f65a45a18b7042ba1ce2703688b62d874cf18) Thanks [@langermank](https://github.com/langermank)! - Global CSS focus styles
+
+- [#1767](https://github.com/primer/css/pull/1767) [`7e01db97`](https://github.com/primer/css/commit/7e01db97f3f2b9d9383f66385ca16f0bdc06abd1) Thanks [@tobiasahlin](https://github.com/tobiasahlin)! - Marketing: Remove unused pullquote selector
+
+* [#1821](https://github.com/primer/css/pull/1821) [`daa2685c`](https://github.com/primer/css/commit/daa2685c596d894b3bae1896bf97c3319cd9816c) Thanks [@jonrohan](https://github.com/jonrohan)! - UnderlineNav `:focus` styles
+  Refactor selected state and spacing
+  Add selected bold state override from github/github
+
+- [#2047](https://github.com/primer/css/pull/2047) [`553d72cc`](https://github.com/primer/css/commit/553d72cc1baaf43a4c743c50cc8881f3811123e9) Thanks [@langermank](https://github.com/langermank)! - UnderlineNav bug fix
+
+* [#2046](https://github.com/primer/css/pull/2046) [`55e2b069`](https://github.com/primer/css/commit/55e2b069a4cbd225af676bef918bbbe2e6cd73b7) Thanks [@langermank](https://github.com/langermank)! - Global focus style CSS from feature flag (next major)
+
+### Patch Changes
+
+- [#2048](https://github.com/primer/css/pull/2048) [`dc529e31`](https://github.com/primer/css/commit/dc529e3102788d5caa136a9d30c58e56721427a7) Thanks [@simurai](https://github.com/simurai)! - Fix headings with an anchor in a summary
+
+* [#2041](https://github.com/primer/css/pull/2041) [`20550bbf`](https://github.com/primer/css/commit/20550bbfaa236ac197ca58805542eaab0bf38fd4) Thanks [@jonrohan](https://github.com/jonrohan)! - Upgrading to stylelint-config-12.4.0
+
 ## 19.8.2
 
 ### Patch Changes

docs/src/stories/components/Link/Link.stories.jsx

@@ -71,7 +71,7 @@ export const LinkTemplate = ({label, variant, href, noUnderline, focusElement, f
   <>
     <a
       href={href}
-      className={clsx(variant && `${variant}`, noUnderline && 'no-underline', focusAllElements && 'focus')}
+      className={clsx('Link', variant && `${variant}`, noUnderline && 'no-underline', focusAllElements && 'focus')}
     >
       {label}
     </a>

docs/src/stories/components/Marketing/MarketingButton.stories.jsx

@@ -7,7 +7,7 @@ export default {
     layout: 'padded'
   },
 
-  excludeStories: ['ButtonTemplate'],
+  excludeStories: ['MarketingButtonTemplate'],
   argTypes: {
     variant: {
       options: [0, 1, 2, 3], // iterator
@@ -77,7 +77,7 @@ const focusMethod = function getFocus() {
   button.focus()
 }
 
-export const ButtonTemplate = ({label, variant, disabled, size, animated, focusElement, focusAllElements}) => (
+export const MarketingButtonTemplate = ({label, variant, disabled, size, animated, focusElement, focusAllElements}) => (
   <>
     <button
       disabled={disabled}
@@ -111,7 +111,7 @@ export const ButtonTemplate = ({label, variant, disabled, size, animated, focusE
   </>
 )
 
-export const Playground = ButtonTemplate.bind({})
+export const Playground = MarketingButtonTemplate.bind({})
 Playground.args = {
   animated: false,
   focusElement: false,

docs/src/stories/components/Marketing/MarketingLink.stories.jsx

@@ -7,7 +7,7 @@ export default {
     layout: 'padded'
   },
 
-  excludeStories: ['LinkTemplate'],
+  excludeStories: ['MarketingLinkTemplate'],
   argTypes: {
     size: {
       options: [0, 1], // iterator
@@ -67,7 +67,7 @@ const focusMethod = function getFocus() {
   link.focus()
 }
 
-export const LinkTemplate = ({label, emphasis, href, size, focusElement, focusAllElements}) => (
+export const MarketingLinkTemplate = ({label, emphasis, href, size, focusElement, focusAllElements}) => (
   <>
     <a
       href={href}
@@ -98,7 +98,7 @@ export const LinkTemplate = ({label, emphasis, href, size, focusElement, focusAl
   </>
 )
 
-export const Playground = LinkTemplate.bind({})
+export const Playground = MarketingLinkTemplate.bind({})
 Playground.args = {
   label: 'Link label',
   href: '/',

docs/src/stories/components/Navigation/UnderlineNav.stories.jsx

@@ -0,0 +1,73 @@
+import React from 'react'
+import clsx from 'clsx'
+import {UnderlineNavItemTemplate} from './UnderlineNavItem.stories'
+
+export default {
+  title: 'Components/Navigation/UnderlineNav',
+  excludeStories: ['UnderlineNavTemplate'],
+  layout: 'padded',
+  argTypes: {
+    variant: {
+      options: [0, 1, 2], // iterator
+      mapping: ['', 'UnderlineNav--right', 'UnderlineNav--full'], // values
+      control: {
+        type: 'inline-radio',
+        labels: ['default', 'align-right', 'fullwidth']
+      },
+      description: 'nav positioning options',
+      table: {
+        category: 'CSS'
+      }
+    },
+    children: {
+      description: 'creates a slot for children',
+      table: {
+        category: 'HTML'
+      }
+    },
+    actionStart: {
+      description: 'action to left of nav',
+      table: {
+        category: 'HTML'
+      }
+    },
+    actionEnd: {
+      description: 'action to right of nav',
+      table: {
+        category: 'HTML'
+      }
+    }
+  }
+}
+
+export const UnderlineNavTemplate = ({variant, children, actionStart, actionEnd}) => (
+  <>
+    <nav className={clsx('UnderlineNav', variant && `${variant}`)}>
+      {actionStart}
+      {variant === 'UnderlineNav--full' ? (
+        <div class="container-lg UnderlineNav-container">
+          <ul class="UnderlineNav-body" role="tablist">
+            {children}
+          </ul>
+        </div>
+      ) : (
+        <ul class="UnderlineNav-body" role="tablist">
+          {children}
+        </ul>
+      )}
+      {actionEnd}
+    </nav>
+  </>
+)
+
+export const Playground = UnderlineNavTemplate.bind({})
+Playground.args = {
+  variant: 0,
+  children: (
+    <>
+      <UnderlineNavItemTemplate label="Item" semanticItemType="button" selected usesDataContent />
+      <UnderlineNavItemTemplate label="Item" semanticItemType="button" usesDataContent />
+      <UnderlineNavItemTemplate label="Item" semanticItemType="button" usesDataContent />
+    </>
+  )
+}

docs/src/stories/components/Navigation/UnderlineNavAction.stories.jsx

@@ -0,0 +1,56 @@
+import React from 'react'
+import clsx from 'clsx'
+import {ButtonTemplate} from '../Button/Button.stories'
+import {LinkTemplate} from '../Link/Link.stories'
+
+export default {
+  title: 'Components/Navigation/UnderlineNav/Action',
+  excludeStories: ['UnderlineNavActionTemplate'],
+  layout: 'padded',
+  argTypes: {
+    semanticItemType: {
+      options: ['button', 'link'],
+      control: {
+        type: 'inline-radio'
+      },
+      description: 'item can be a button or a link',
+      table: {
+        category: 'HTML'
+      }
+    },
+    label: {
+      name: 'label',
+      type: 'string',
+      description: 'Item label text',
+      table: {
+        category: 'HTML'
+      }
+    },
+    focusElement: {
+      control: {type: 'boolean'},
+      description: 'set manual focus on item',
+      table: {
+        category: 'Interactive'
+      }
+    }
+  }
+}
+
+export const UnderlineNavActionTemplate = ({semanticItemType, label, focusElement}) => {
+  return (
+    <div class="UnderlineNav-actions">
+      {semanticItemType === 'button' ? (
+        <ButtonTemplate label={label} focusAllElements={focusElement} />
+      ) : (
+        <LinkTemplate label={label} focusAllElements={focusElement} />
+      )}
+    </div>
+  )
+}
+
+export const Playground = UnderlineNavActionTemplate.bind({})
+Playground.args = {
+  semanticItemType: 'button',
+  label: 'Action',
+  focusElement: false
+}

docs/src/stories/components/Navigation/UnderlineNavItem.stories.jsx

@@ -0,0 +1,140 @@
+import React from 'react'
+import clsx from 'clsx'
+import useToggle from '../../helpers/useToggle.jsx'
+
+export default {
+  title: 'Components/Navigation/UnderlineNav/Item',
+  excludeStories: ['UnderlineNavItemTemplate'],
+  layout: 'padded',
+  argTypes: {
+    selected: {
+      control: {type: 'boolean'},
+      description: 'active nav item',
+      table: {
+        category: 'CSS'
+      }
+    },
+    usesDataContent: {
+      control: {type: 'boolean'},
+      description: 'creates a hidden label to allow for bold text without layout shift',
+      table: {
+        category: 'CSS'
+      }
+    },
+    semanticItemType: {
+      options: ['button', 'link'],
+      control: {
+        type: 'inline-radio'
+      },
+      description: 'item can be a button or a link',
+      table: {
+        category: 'HTML'
+      }
+    },
+    label: {
+      name: 'label',
+      type: 'string',
+      description: 'Item label text',
+      table: {
+        category: 'HTML'
+      }
+    },
+    focusElement: {
+      control: {type: 'boolean'},
+      description: 'set manual focus on tab item',
+      table: {
+        category: 'Interactive'
+      }
+    },
+    icon: {
+      control: {type: 'boolean'},
+      description: 'show icon',
+      table: {
+        category: 'CSS'
+      }
+    },
+    counter: {
+      control: {type: 'boolean'},
+      description: 'show counter',
+      table: {
+        category: 'CSS'
+      }
+    }
+  }
+}
+
+export const UnderlineNavItemTemplate = ({
+  semanticItemType,
+  label,
+  selected,
+  focusElement,
+  icon,
+  counter,
+  usesDataContent
+}) => {
+  const [isSelected, itemisSelected] = useToggle()
+  return (
+    <li className="d-inline-flex">
+      {semanticItemType === 'button' ? (
+        <button
+          className={clsx('UnderlineNav-item', focusElement && 'focus')}
+          role="tab"
+          aria-selected={selected || isSelected ? 'true' : undefined}
+          onClick={itemisSelected}
+        >
+          {icon && (
+            <svg
+              xmlns="http://www.w3.org/2000/svg"
+              viewBox="0 0 16 16"
+              width="16"
+              height="16"
+              className="UnderlineNav-octicon"
+            >
+              <path
+                fill-rule="evenodd"
+                d="M14.064 0a8.75 8.75 0 00-6.187 2.563l-.459.458c-.314.314-.616.641-.904.979H3.31a1.75 1.75 0 00-1.49.833L.11 7.607a.75.75 0 00.418 1.11l3.102.954c.037.051.079.1.124.145l2.429 2.428c.046.046.094.088.145.125l.954 3.102a.75.75 0 001.11.418l2.774-1.707a1.75 1.75 0 00.833-1.49V9.485c.338-.288.665-.59.979-.904l.458-.459A8.75 8.75 0 0016 1.936V1.75A1.75 1.75 0 0014.25 0h-.186zM10.5 10.625c-.088.06-.177.118-.266.175l-2.35 1.521.548 1.783 1.949-1.2a.25.25 0 00.119-.213v-2.066zM3.678 8.116L5.2 5.766c.058-.09.117-.178.176-.266H3.309a.25.25 0 00-.213.119l-1.2 1.95 1.782.547zm5.26-4.493A7.25 7.25 0 0114.063 1.5h.186a.25.25 0 01.25.25v.186a7.25 7.25 0 01-2.123 5.127l-.459.458a15.21 15.21 0 01-2.499 2.02l-2.317 1.5-2.143-2.143 1.5-2.317a15.25 15.25 0 012.02-2.5l.458-.458h.002zM12 5a1 1 0 11-2 0 1 1 0 012 0zm-8.44 9.56a1.5 1.5 0 10-2.12-2.12c-.734.73-1.047 2.332-1.15 3.003a.23.23 0 00.265.265c.671-.103 2.273-.416 3.005-1.148z"
+              ></path>
+            </svg>
+          )}
+          <span data-content={usesDataContent ? label : undefined}>{label}</span>
+          {counter && <span class="Counter">10</span>}
+        </button>
+      ) : (
+        <a
+          className={clsx('UnderlineNav-item', focusElement && 'focus')}
+          aria-current={selected || isSelected ? 'true' : undefined}
+          onClick={itemisSelected}
+          //   aria-current={selected ? 'page' : undefined}
+        >
+          {icon && (
+            <svg
+              xmlns="http://www.w3.org/2000/svg"
+              viewBox="0 0 16 16"
+              width="16"
+              height="16"
+              className="UnderlineNav-octicon"
+            >
+              <path
+                fill-rule="evenodd"
+                d="M14.064 0a8.75 8.75 0 00-6.187 2.563l-.459.458c-.314.314-.616.641-.904.979H3.31a1.75 1.75 0 00-1.49.833L.11 7.607a.75.75 0 00.418 1.11l3.102.954c.037.051.079.1.124.145l2.429 2.428c.046.046.094.088.145.125l.954 3.102a.75.75 0 001.11.418l2.774-1.707a1.75 1.75 0 00.833-1.49V9.485c.338-.288.665-.59.979-.904l.458-.459A8.75 8.75 0 0016 1.936V1.75A1.75 1.75 0 0014.25 0h-.186zM10.5 10.625c-.088.06-.177.118-.266.175l-2.35 1.521.548 1.783 1.949-1.2a.25.25 0 00.119-.213v-2.066zM3.678 8.116L5.2 5.766c.058-.09.117-.178.176-.266H3.309a.25.25 0 00-.213.119l-1.2 1.95 1.782.547zm5.26-4.493A7.25 7.25 0 0114.063 1.5h.186a.25.25 0 01.25.25v.186a7.25 7.25 0 01-2.123 5.127l-.459.458a15.21 15.21 0 01-2.499 2.02l-2.317 1.5-2.143-2.143 1.5-2.317a15.25 15.25 0 012.02-2.5l.458-.458h.002zM12 5a1 1 0 11-2 0 1 1 0 012 0zm-8.44 9.56a1.5 1.5 0 10-2.12-2.12c-.734.73-1.047 2.332-1.15 3.003a.23.23 0 00.265.265c.671-.103 2.273-.416 3.005-1.148z"
+              ></path>
+            </svg>
+          )}
+          <span data-content={usesDataContent ? label : undefined}>{label}</span>
+          {counter && <span class="Counter">10</span>}
+        </a>
+      )}
+    </li>
+  )
+}
+
+export const Playground = UnderlineNavItemTemplate.bind({})
+Playground.args = {
+  semanticItemType: 'button',
+  label: 'Item',
+  selected: false,
+  focusElement: false,
+  icon: false,
+  counter: false,
+  usesDataContent: true
+}