Skip to content
This repository has been archived by the owner on Jan 4, 2024. It is now read-only.

Next #56

Open
wants to merge 135 commits into
base: main
Choose a base branch
from
Open

Next #56

wants to merge 135 commits into from

Conversation

Jonic
Copy link
Contributor

@Jonic Jonic commented Mar 3, 2021

https://deploy-preview-56--sauce-design-system.netlify.app/

This is a BIG pull request, but it represents a tangible step towards v1.0.0. Almost every last bit of Sauce has been updated to better reflect how we'd like to take the building of our interfaces, with benefits to performance, developer experience, and helping to streamline the design/develop/feedback loop required to help us stay productive while allowing us to react to user and stakeholder feedback.

This version also moves towards having a truly "estate-wide" set of styles, as it sets the groundwork for future employment of brand-specific style tokens. We're also moving towards using CSS custom properties to define our look-and-feel, with the eventual goal of generating these from Design Tokens, and removing the Sass variables. Still some work to be done there, but that's better handled as a cross-team effort once we're ready.

I'm not sure how best to review these changes. I think just a lengthy screenshare conversation is the best way to get this into the main branch.

  • Base styles - emphasis on styling "bare" elements
  • Removes "elements" directory - mostly empty files, some of which we'll likely never use
  • Custom Properties
  • Expanded tokens
  • LOTS of new mixins
  • Moves to using custom properties for overriding styles
  • Creates a brands directory
  • Renamed src to scss - no more confusing "sauce" and "source" when talking about file locations. Now it's "sauce" and "sass", which is still a bit much but WHATEVS.
  • font-size: everything now scaled to root element
  • spacing: ALSO scaled to root element - everything will scale to the root element. EVERYTHING.
  • Primitives: introduces new link and button primitives
  • Replace CSS normalize with modern-css-reset
  • Stubs dev and design docs
    • Adds Typography and Spacing docs
  • Adds Type components with stories
  • Expands Sass tests to (hopefully) cover everything defined as a mixin

@Jonic Jonic self-assigned this Mar 3, 2021
@Jonic Jonic marked this pull request as draft March 3, 2021 19:17
* Remove Storybook

* Update npm tooling

* Add netlify toml config

* Add Fractal theme

* Update docs and components

* Move sauce.scss to fractal theme

* Add dev tooling config

* Remove test Docker config

* bump Node version in nvmrc

* Remove Chromatic tasks from Circle Config

* Add static.mount config

* Remove `clean` task

* Update README

* Fix theme partials

* Add build debug info to README

* Add build:debug script to npm

* Remove file extension config

* Fix favicon and styles references

* Enable static assets in theme

* Fix theme static assets config

* Clarify Node version requirements in README

* Update Circle config to use an Orb

* Update and pin all dependencies

* Update node image version in Circle config

* Configure LTS as node version ot install on Circle

* Add postcss

* Skip node install and just install dependencies

* Update jest config

* Remove test nofifier

* Testing old circle config

* Use lts-browsers for circle config

* Use node orb to run test

* Updated config.yml

* Just use the old config

* Add ci test command

* Add engines

* Format package.json

* Treat package-lock as binary in Git

* Remove gulp-webserver

* Add http-server

* Update README a bit

* Add Type variants

* Update Fractal default context

* Add test type README

* Update Fractal theme

* Remove examples from type notes

* Fix theme styles

* Update dependabot config

* Remove code spell checker

* Don't "bundle" css

* Remove colons from type variant names

* Fix netlify config

* Build gulp first?

* Swap template engine to Nunjucks

* Use nunj extension

* Update syntax highlighting

* Update panels config

* Rename Type

* Lint Markdown

* Add partial overrides for controls and resources
@AnnaKahnRaspPi AnnaKahnRaspPi added this to the Sauce v1 milestone Oct 11, 2021
@Jonic Jonic removed their assignment Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants