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 2396 core scss v4 for checkbox #1370

Merged
merged 5 commits into from
Oct 22, 2024

Conversation

timwessman
Copy link
Contributor

@timwessman timwessman commented Oct 2, 2024

Description

  • Convert the CSS of the Checkbox component to SCSS.
  • Documentation added

NOTE:

  • There is a small difference on visual test of SelectionGroup. I can't find out why. The failing SelectrionGroup -story looks pixel perfect in Storybook, but on visual-test, margin between helper-text and checkboxes are couple of pixels smaller than it should.

Related Issue

Closes HDS-2396

How Has This Been Tested?

  • visual tests
  • e2e tests

Demos:

Links to demos are in the comments

Screenshots (if appropriate):

Add to changelog

  • Added needed line to changelog

Copy link

github-actions bot commented Oct 2, 2024

Preview found from hds-demo docs/preview_1370

Demos

Docs
Core Storybook
React Storybook

Copy link

github-actions bot commented Oct 2, 2024

Test Results

 1 files  26 suites   4m 19s ⏱️
13 tests 13 ✅ 0 💤 0 ❌
26 runs  26 ✅ 0 💤 0 ❌

Results for commit 598fe7f.

♻️ This comment has been updated with latest results.

@timwessman timwessman force-pushed the hds-2396-core-scss-v4-for-checkbox branch from ac4d013 to 55bde63 Compare October 3, 2024 06:38
@mrTuomoK
Copy link
Contributor

mrTuomoK commented Oct 3, 2024

Hmm... wonder why the visual tests fail for SelectionGroup 🤔

@timwessman timwessman force-pushed the hds-2396-core-scss-v4-for-checkbox branch from dc89665 to 598fe7f Compare October 22, 2024 07:07
@timwessman timwessman merged commit 5ca2637 into release-4.0.0 Oct 22, 2024
4 checks passed
@timwessman timwessman deleted the hds-2396-core-scss-v4-for-checkbox branch October 22, 2024 12:59
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.

2 participants