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

fix(styles): update form switch HCM styles #2667

Merged
merged 2 commits into from
Feb 19, 2024

Conversation

alizedebray
Copy link
Contributor

No description provided.

@alizedebray alizedebray requested a review from a team as a code owner February 15, 2024 15:44
Copy link

changeset-bot bot commented Feb 15, 2024

🦋 Changeset detected

Latest commit: e7e4dc8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 10 packages
Name Type
@swisspost/design-system-styles Patch
@swisspost/design-system-components-angular-workspace Patch
@swisspost/design-system-components Patch
@swisspost/design-system-demo Patch
@swisspost/design-system-documentation Patch
@swisspost/internet-header Patch
@swisspost/design-system-intranet-header-workspace Patch
@swisspost/design-system-intranet-header Patch
@swisspost/design-system-components-react Patch
@swisspost/design-system-components-angular Patch

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

@swisspost-bot
Copy link
Contributor

swisspost-bot commented Feb 15, 2024

Preview environment ready: https://preview-2667--swisspost-web-frontend.netlify.app
Preview environment ready: https://preview-2667--swisspost-design-system-next.netlify.app

border: form-check.$form-check-input-border-width solid CanvasText;

&:checked {
background-color: CanvasText;
Copy link
Contributor

@imagoiq imagoiq Feb 15, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can use AccentColor and as a fallback CanvasText. I'm not sure of what I'm saying, but it looks like it's the color to show that an element is active. It's supported by Firefox (though I don't see the difference) and under a flag on blink engine browsers.

Copy link

sonarcloud bot commented Feb 19, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@alizedebray alizedebray merged commit 24e374d into main Feb 19, 2024
9 checks passed
@alizedebray alizedebray deleted the 2645-hcm-darklight-switch-is-not-usable branch February 19, 2024 08:49
gfellerph pushed a commit that referenced this pull request Feb 20, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @swisspost/design-system-icons@1.3.0

### Minor Changes

- Added icon number 2575. (by
[@swisspost-bot](https://github.com/swisspost-bot) with
[#2677](#2677))

## @swisspost/design-system-components@2.0.2

### Patch Changes

- Fixed double lines between accordion items on chrome/safari. (by
[@imagoiq](https://github.com/imagoiq) with
[#2615](#2615))

- Fixed two issues with tooltips and Angular integration. Tooltips are
now being displayed on buttons that are added to the DOM asynchronously
and tooltip elements are no longer duplicated on route change. (by
[@gfellerph](https://github.com/gfellerph) with
[#2621](#2621))

-   Updated dependencies:
    -   @swisspost/design-system-styles@6.6.2

## @swisspost/design-system-components-angular@1.0.1

### Patch Changes

-   Updated dependencies:
    -   @swisspost/design-system-components@2.0.2

## @swisspost/internet-header@1.13.6

### Patch Changes

- Added secure flag for cookies. (by
[@imagoiq](https://github.com/imagoiq) with
[#2629](#2629))

- Fixed display of internet-header icons on high-contrast mode with
light theme. (by [@imagoiq](https://github.com/imagoiq) with
[#2669](#2669))

- Fixed missing accessible labels on post-meta navigation links. (by
[@gfellerph](https://github.com/gfellerph) with
[#2700](#2700))

-   Updated dependencies:
    -   @swisspost/design-system-styles@6.6.2

## @swisspost/design-system-intranet-header@5.0.9

### Patch Changes

-   Updated dependencies:
    -   @swisspost/design-system-styles@6.6.2

## @swisspost/design-system-styles@6.6.2

### Patch Changes

- Updated high contrast styles for the form switch component. (by
[@alizedebray](https://github.com/alizedebray) with
[#2667](#2667))

- Fixed display of icons in high-contrast mode in webkit. (by
[@imagoiq](https://github.com/imagoiq) with
[#2638](#2638))

- Deprecated sizes `loader-sm` and `loader-xs` for the spinner. They are
replace respectively with `loader-40` and `loader-16`. (by
[@imagoiq](https://github.com/imagoiq) with
[#2612](#2612))

- Removed transparent background for notification in high-contrast mode.
(by [@imagoiq](https://github.com/imagoiq) with
[#2653](#2653))

- Fixed visibility of select arrow in icon in high-contrast mode with
light theme. (by [@imagoiq](https://github.com/imagoiq) with
[#2696](#2696))

- Sets the font-size of h5 on small mobile devices to 16px as defined in
Figma. (by [@gfellerph](https://github.com/gfellerph) with
[#2610](#2610))

## @swisspost/design-system-documentation@3.1.0

### Minor Changes

- Added contrast ratios agains black & white to color documentation. (by
[@b1aserlu](https://github.com/b1aserlu) with
[#2376](#2376))

- Added override classes to the heading documentation and removed the
utitlites/headings, which was previouvly used to document the override
classes. (by [@b1aserlu](https://github.com/b1aserlu) with
[#2640](#2640))

- Added a documentation page for the post spinner component. (by
[@imagoiq](https://github.com/imagoiq) with
[#2612](#2612))

- Migrated documentation for card-button to storybook. (by
[@b1aserlu](https://github.com/b1aserlu) with
[#2619](#2619))

### Patch Changes

- Fixed missing archive links in migrated page. (by
[@imagoiq](https://github.com/imagoiq) with
[#2663](#2663))

- Removed empty pages. (by [@imagoiq](https://github.com/imagoiq) with
[#2666](#2666))

- Documented `post-accordion-item` properties on Accordion page. (by
[@imagoiq](https://github.com/imagoiq) with
[#2630](#2630))

- Removed input type color variant as it is not compatible with the
current input styles. (by [@imagoiq](https://github.com/imagoiq) with
[#2628](#2628))

- Documented collapseChange event from post-accordion-item on
post-accordion. (by [@imagoiq](https://github.com/imagoiq) with
[#2620](#2620))

- Improved rendering of the home page for high-contrast mode. (by
[@imagoiq](https://github.com/imagoiq) with
[#2653](#2653))

-   Updated dependencies:
    -   @swisspost/design-system-icons@1.3.0
    -   @swisspost/design-system-styles@6.6.2
    -   @swisspost/internet-header@1.13.6
    -   @swisspost/design-system-components@2.0.2
    -   @swisspost/design-system-intranet-header@5.0.9
    -   @swisspost/design-system-components-angular@1.0.1
    -   @swisspost/design-system-components-react@1.0.28

## @swisspost/design-system-components-angular-workspace@1.0.2

### Patch Changes

-   Updated dependencies:
    -   @swisspost/design-system-styles@6.6.2
    -   @swisspost/design-system-components@2.0.2

## @swisspost/design-system-components-react@1.0.28

### Patch Changes

-   Updated dependencies:
    -   @swisspost/design-system-components@2.0.2

## @swisspost/design-system-demo@7.2.2

### Patch Changes

-   Updated dependencies:
    -   @swisspost/design-system-styles@6.6.2
    -   @swisspost/design-system-intranet-header@5.0.9

## @swisspost/design-system-intranet-header-workspace@3.0.9

### Patch Changes

-   Updated dependencies:
    -   @swisspost/design-system-styles@6.6.2

## @swisspost/design-system-nextjs-integration@0.1.1

### Patch Changes

-   Updated dependencies:
    -   @swisspost/design-system-styles@6.6.2
    -   @swisspost/internet-header@1.13.6
    -   @swisspost/design-system-components-react@1.0.28

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.

None yet

3 participants