-
Notifications
You must be signed in to change notification settings - Fork 16
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
audit updates to Content-principles and links added #1819
base: develop
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: e7e0529 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Quality Gate passedIssues Measures |
|
||
If content already appears on the Export Service or the [Department of Agriculture, Fisheries and Forestry](https://www.agriculture.gov.au/) (DAFF) website, link to the authoritative source. | ||
If content already appears somewhere online link to the authoritative source. For example, the [Export Service](https://exports.agriculture.gov.au/). Or the [Department of Agriculture, Fisheries and Forestry](https://www.agriculture.gov.au/) (DAFF) website. |
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.
Feels weird starting this sentence with "or", shouldn't it just be a continuation? For example, the Export Service of the DAFF website
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.
@Suzanne-aff As discussed
@@ -7,7 +7,7 @@ order: 1 | |||
|
|||
You can use these standards to guide you to create content that is clear and effective. | |||
|
|||
Good content reduces the cognitive load on users and helps them achieve their aims more easily. | |||
Good content reduces the cognitive load on users. It helps them achieve their aims more easily. |
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.
Weirdly, having a new sentence for "It" seems to increase cognitive load for me 😅
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.
@Suzanne-aff As discussed
Provide an overview of your changes here. Include screenshots, photos or links if necessary.
View preview
Checklist
Preflight
accordion: Updated padding
ordocs: Updated header links
yarn changeset
. Learn more about change management.Testing
yarn test
to ensure tests are passing. If required, runyarn test -u
to update any generated snapshots.Documentation
Creating new component
docs/overview.mdx
anddocs/code.mdx
at a minimum)yarn
in the root of the repo to do this)docs/components/designSystemComponents.tsx
).storybook/stories/KitchenSink.tsx
)docs/playroom/snippets.ts
)docs/components/pictograms/index.tsx
)