-
Notifications
You must be signed in to change notification settings - Fork 54
Introduces the @yoast/components package #57
Conversation
* develop: (266 commits) Update yarn lock Fix typo in comment in a German morphology stem file Improve code style Fix ESLint errors Don't debounce in the example Move addVariable button to variable Improve code style Updated parameter types of JSDoc of Headings research functions. Update .gitignore Removed blank line padding. Headings research does not return null or the node, but the empty list or a list with one item. Remove empty test script Add missing comma Added test case for a tree with no headings. Created Headdings researcher. Publish Add 4.22.0 changelog Add prepublishOnly scripts to both packages Centralize dist ignore rule Move CSS files to more specific folder ...
* develop: Fix code style Add German transition words Fix typos in German morphology files
All dependencies need to be added to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yoast/components
cannot have a dependency on yoast-components
.
packages/components/ErrorBoundary.js
Outdated
@@ -6,7 +6,7 @@ import { __ } from "@wordpress/i18n"; | |||
import { speak as a11ySpeak } from "@wordpress/a11y"; | |||
|
|||
/* Internal dependencies */ | |||
import colors from "../../style-guide/colors"; | |||
import colors from "../yoast-components/style-guide/colors"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should have no dependencies on yoast-components
in @yoast/components
. So that means we should probably move colors
to @yoast/components
.
* develop: Direct users to the right place Make sure that the app uses components from the packages, not with a relative path First working app version :D Trying to get the app to work; status: unsuccesful and frustrated... Initial move of the yoast-components-app Add install-ci to the pretest instead of the test Init @yoast/social-metadata-previews package Make sure that yoastseo yarn test installs the preium-configuration as this is required for the test Publish Add handy lerna scripts Add prepublish to ESLint Add publish command to scripts Fix ESLint error # Conflicts: # apps/components/App.js # apps/components/ButtonsWrapper.js # apps/components/ComponentsExample.js # apps/components/ExamplesContainer.js # apps/components/HelpCenterWrapper.js # apps/components/SidebarCollapsibleWrapper.js # apps/components/SvgIconsWrapper.js
packages/yoast-components/index.js
Outdated
@@ -27,7 +26,7 @@ import { default as LinkSuggestions } from "./composites/LinkSuggestions/LinkSug | |||
// Composites/KeywordSuggestions imports. | |||
import { default as KeywordSuggestions } from "./composites/KeywordSuggestions/KeywordSuggestions"; | |||
// Composites/basic imports. | |||
import { default as Loader } from "./composites/basic/Loader"; | |||
import { default as Loader } from "../components/Loader"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be written as "@yoast/components/Loader`?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is something that needs to be changed.
Also, the build is still failing. |
@@ -1,4 +1,4 @@ | |||
@import "../../style-guide/colors"; | |||
@import "../../../components/style-guide/colors"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be written with ~
@@ -1,4 +1,4 @@ | |||
@import "accessibility"; | |||
@import "../composites/OnboardingWizard/onboarding-wizard"; | |||
@import "../composites/basic/loader"; | |||
@import "../../components/loader"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This cuold be written with ~
443f500
to
77c422b
Compare
77c422b
to
a7e7e74
Compare
Conflicts: packages/components/Explanation.js packages/components/composites/Textfield.js packages/configuration-wizard/StepButton.js packages/configuration-wizard/components/Choice.js packages/configuration-wizard/components/Explanation.js packages/configuration-wizard/components/StepComponents.js packages/configuration-wizard/config/yoast-theme.js packages/configuration-wizard/onboarding-wizard.scss packages/yoast-components/composites/OnboardingWizard/StepButton.js packages/yoast-components/composites/OnboardingWizard/components/Choice.js packages/yoast-components/composites/OnboardingWizard/components/Explanation.js packages/yoast-components/composites/OnboardingWizard/components/StepComponents.js packages/yoast-components/composites/OnboardingWizard/config/yoast-theme.js packages/yoast-components/composites/OnboardingWizard/onboarding-wizard.scss packages/yoast-components/css/all.scss packages/yoast-components/forms/composites/Textfield.js
Fixes #30
Makes sure
yarn lint
andyarn test
will have the same output as on thedevelop
branch