From b86b9db22b3161c98de15d391364b57ea64464a3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Matilda=20Landstr=C3=B6m?= Date: Mon, 31 Mar 2025 07:29:18 +0000 Subject: [PATCH] Merged in fix/icon-color (pull request #1672) fix: change to correct icon color * fix: change to correct icon color Approved-by: Erik Tiekstra Approved-by: Fredrik Thorsson --- .../ExperienceList/experienceList.module.css | 2 +- .../DestinationPage/ExperienceList/index.tsx | 2 +- .../Header/HeaderLink/headerLink.module.css | 9 +++++---- .../components/Header/HeaderLink/index.tsx | 8 +++++--- .../components/LanguageSwitcher/index.tsx | 12 ++++++++---- 5 files changed, 20 insertions(+), 13 deletions(-) diff --git a/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/experienceList.module.css b/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/experienceList.module.css index 9c95cc698..502b4a4bb 100644 --- a/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/experienceList.module.css +++ b/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/experienceList.module.css @@ -1,6 +1,6 @@ .experienceList { list-style: none; display: flex; - gap: var(--Spacing-x1); + gap: var(--Space-x1); flex-wrap: wrap; } diff --git a/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/index.tsx b/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/index.tsx index eaadf20fb..e52e6e3dc 100644 --- a/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/index.tsx +++ b/apps/scandic-web/components/ContentType/DestinationPage/ExperienceList/index.tsx @@ -21,7 +21,7 @@ export default function ExperienceList({ experiences }: ExperienceListProps) { {experienceList.map(({ Icon, name }) => (
  • - + {name}
  • diff --git a/apps/scandic-web/components/Header/HeaderLink/headerLink.module.css b/apps/scandic-web/components/Header/HeaderLink/headerLink.module.css index bf642527f..894a1c443 100644 --- a/apps/scandic-web/components/Header/HeaderLink/headerLink.module.css +++ b/apps/scandic-web/components/Header/HeaderLink/headerLink.module.css @@ -1,17 +1,18 @@ .headerLink { display: flex; align-items: center; - gap: var(--Spacing-x1); + gap: var(--Space-x1); + color: var(--Text-Interactive-Secondary); } .headerLink:hover { - color: var(--Base-Text-High-contrast); + color: var(--Text-Interactive-Default); } .headerLink .icon * { - fill: var(--Base-Text-Medium-contrast); + fill: var(--Icon-Interactive-Default); } .headerLink:hover .icon * { - fill: var(--Base-Text-High-contrast); + fill: var(--Icon-Interactive-Secondary); } diff --git a/apps/scandic-web/components/Header/HeaderLink/index.tsx b/apps/scandic-web/components/Header/HeaderLink/index.tsx index eb1367dd7..44d71b6b6 100644 --- a/apps/scandic-web/components/Header/HeaderLink/index.tsx +++ b/apps/scandic-web/components/Header/HeaderLink/index.tsx @@ -2,8 +2,9 @@ import Link from "next/link" +import { Typography } from "@scandic-hotels/design-system/Typography" + import { IconByIconName } from "@/components/Icons/IconByIconName" -import Caption from "@/components/TempDesignSystem/Text/Caption" import styles from "./headerLink.module.css" @@ -17,17 +18,18 @@ export default function HeaderLink({ onClick = () => undefined, }: HeaderLinkProps) { return ( - + {iconName ? ( ) : null} {children} - + ) } diff --git a/apps/scandic-web/components/LanguageSwitcher/index.tsx b/apps/scandic-web/components/LanguageSwitcher/index.tsx index 13eac0dd4..4cb051eb7 100644 --- a/apps/scandic-web/components/LanguageSwitcher/index.tsx +++ b/apps/scandic-web/components/LanguageSwitcher/index.tsx @@ -6,6 +6,7 @@ import FocusLock from "react-focus-lock" import { useIntl } from "react-intl" import { MaterialIcon } from "@scandic-hotels/design-system/Icons" +import { Typography } from "@scandic-hotels/design-system/Typography" import { languages } from "@/constants/languages" import { trpc } from "@/lib/trpc/client" @@ -16,7 +17,6 @@ import { useHandleKeyUp } from "@/hooks/useHandleKeyUp" import useLang from "@/hooks/useLang" import SkeletonShimmer from "../SkeletonShimmer" -import Caption from "../TempDesignSystem/Text/Caption" import LanguageSwitcherContainer from "./LanguageSwitcherContainer" import LanguageSwitcherContent from "./LanguageSwitcherContent" import { languageSwitcherVariants } from "./variants" @@ -106,14 +106,18 @@ export default function LanguageSwitcher({ type }: LanguageSwitcherProps) { aria-label={isLanguageSwitcherOpen ? closeMsg : openMsg} onClick={handleClick} > - - + + {languages[currentLanguage]} - +