Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add postcss-preset-primer #4751

Merged
merged 4 commits into from
Jul 30, 2024
Merged

Conversation

joshblack
Copy link
Member

@joshblack joshblack commented Jul 18, 2024

As more and more places are using/compiling CSS with Primer, it would be helpful to have a package for this that acts as a preset for the various plugins/strategies that we recommend.

This PR adds in a proof-of-concept package for this, postcss-preset-primer, that can be shared and reused for compiling CSS for Primer.

Changelog

New

  • Add postcss-preset-primer package

Changed

Removed

Rollout strategy

  • None; if selected, include a brief description as to why

This package is currently private

Copy link

changeset-bot bot commented Jul 18, 2024

⚠️ No Changeset found

Latest commit: 82ec6a7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added the staff Author is a staff member label Jul 18, 2024
Copy link
Contributor

github-actions bot commented Jul 18, 2024

size-limit report 📦

Path Size
packages/react/dist/browser.esm.js 94.87 KB (0%)
packages/react/dist/browser.umd.js 95.11 KB (0%)

@github-actions github-actions bot temporarily deployed to storybook-preview-4751 July 18, 2024 19:05 Inactive
@joshblack joshblack added the skip changeset This change does not need a changelog label Jul 25, 2024
@joshblack joshblack added this pull request to the merge queue Jul 30, 2024
Merged via the queue into main with commit 8977361 Jul 30, 2024
30 checks passed
@joshblack joshblack deleted the feat/add-postcss-preset-primer branch July 30, 2024 20:18
TylerJDev pushed a commit that referenced this pull request Jul 31, 2024
* feat: add postcss-preset-primer

* docs: update usage snippet

* chore: fix eslint rules and remove postcss-mixins

---------

Co-authored-by: Josh Black <[email protected]>
@adam-rocska
Copy link

Any hint maybe on when could this be available for install?

Just wanted to give it a spin according to the getting started ¯\(ツ)/¯ but got this:

rocskaadam@Rocska-MBP src.adam-rocska.github.io % pnpm install postcss-preset-primer --save-dev
 ERR_PNPM_FETCH_404  GET https://registry.npmjs.org/postcss-preset-primer: Not Found - 404

This error happened while installing a direct dependency of /Users/rocskaadam/src/adam-rocska/src/src.adam-rocska.github.io

postcss-preset-primer is not in the npm registry, or you have no permission to fetch it.

No authorization header was set for the request.
Progress: resolved 16, reused 16, downloaded 0, added 0

github-merge-queue bot pushed a commit that referenced this pull request Sep 9, 2024
* Focus close button on second step

* Remove `autofocus` prop

* Add dependencies to focus trap

* Add changeset

* lint fix

* Remove `body` from dependency

* add state to Dialog example

* chore(changeset): enter prerelease mode for v37 (#4789)

* chore(changeset): enter pre-release mode for v37

* ci: remove snapshots when in pre mode

* chore: add version info to all packages

* Revert "chore: add version info to all packages"

This reverts commit 4665bb3.

* chore: update canary to remove pre.json when running

---------

Co-authored-by: Josh Black <[email protected]>

* Autocomplete: Only open menu on click (#4771)

* Only open menu on click instead of just focus

* Update tests

* Add changeset

* Fix typo

* Set `openOnFocus` to `true`

* Add test, move `onFocus` function

* Update docs

* Adjust changeset

* Remove `useCallback`

* Add deprecated notice

* Prep for high contrast theme border-color changes (#4774)

* borders

* fallback

* test(vrt): update snapshots

* test color changes

* alright

* bump

* test(vrt): update snapshots

* more fixes

* test(vrt): update snapshots

* copy from main

* test(vrt): update snapshots

* Create fluffy-ravens-thank.md

* snippy snaps

* update seg control border

* test(vrt): update snapshots

* remove fallbacks

* copy from main

* test(vrt): update snapshots

---------

Co-authored-by: langermank <[email protected]>

* chore: add package version numbers (#4796)

Co-authored-by: Josh Black <[email protected]>

* feat: add postcss-preset-primer (#4751)

* feat: add postcss-preset-primer

* docs: update usage snippet

* chore: fix eslint rules and remove postcss-mixins

---------

Co-authored-by: Josh Black <[email protected]>

* Utilize `aria-describedby` on all `ActionList` descriptions (#4666)

* Add `aria-describedby` to inline description; add `aria-labelledby` to `TrailingVisual`

* Update snapshots

* Add changeset

* Update snapshot

* Update packages/react/src/ActionList/Item.tsx

Co-authored-by: Siddharth Kshetrapal <[email protected]>

* changes from PR feedback

* Update .changeset/lovely-days-march.md

* Update snapshots

* Update lovely-days-march.md

---------

Co-authored-by: Siddharth Kshetrapal <[email protected]>

* Wrap `header` and `footer` with `<Box>`

* Move `<Box>`

* Add back focus to story

* Update behaviors package

* Move back `useFocusTrap`

---------

Co-authored-by: Siddharth Kshetrapal <[email protected]>
Co-authored-by: Josh Black <[email protected]>
Co-authored-by: Josh Black <[email protected]>
Co-authored-by: Katie Langerman <[email protected]>
Co-authored-by: langermank <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip changeset This change does not need a changelog staff Author is a staff member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants