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

Hds 2460 general documentation updates #1378

Closed
wants to merge 88 commits into from

Conversation

laurakarhu
Copy link
Contributor

@laurakarhu laurakarhu commented Oct 10, 2024

Description

There's quite many things in documentation that require updating when moving to next major version 4.0.0

Related Issue

Closes HDS-2460

Motivation and Context

  • update migration guide
  • update component statuses

How Has This Been Tested?

demosite

Demos:

Links to demos are in the comments

Screenshots (if appropriate):

Add to changelog

these updates tell more about previous changes already described in changelog

mrTuomoK and others added 30 commits October 9, 2024 12:48
…pe to add CommonHTMLAttributes as data-* to props.
The arrow icon takes now space. Therefore it needed a background color and container and aligner elements.
…date

The old command has rimraf which removes all images.
Include showArrow and information -props to the component props
…from Table

Heading id is created when needed
Table id was for checkbox and it is also created when needed.

Updated docs, tests and snapshot.
Better usage of core css.

Cannot use compose in core, so not-so-inconvenient "helper-text-gap" was added.
…exists

Short HelperText stays on same line as ErrorText
Flexbox change requires label + tooltip combination to be wrapped.

Made external components or otherwise focus would be lost from tooltip button on re-render, if anonymous functions were used.
NikoHelle and others added 26 commits October 9, 2024 12:51
…ve and rename react tests to react-* and core added to core-*, add core build to workflow
Added only takeAllStorySnapshots, other changes are from prettier.
Missing "yarn" before tsc causes it to be run globally
Notification errors only show with ts-check-stories. Caused by incompatible child types. Unknown why ts-check does not show it, or VSCode.
Dialog icon was not shown before
… playwright tests, add body spacing to helper for body-overflowing pages
Removed aria-label descripiton when not necessary or useful.
Update breadcrumbs to stable, dropdown to deprecated, header to beta. Add content to migration guide.
Copy link

Test Results

 1 files  18 suites   3m 27s ⏱️
 9 tests  9 ✅ 0 💤 0 ❌
18 runs  18 ✅ 0 💤 0 ❌

Results for commit c66143d.

Copy link

Preview found from hds-demo docs/preview_1378

Demos

Docs
Core Storybook
React Storybook

@laurakarhu
Copy link
Contributor Author

This pr is again not working properly, starting a new one an closing this

@laurakarhu laurakarhu closed this Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants