@orangesk/orange-design-system 2.0.0-beta.2 → 2.0.0-beta.21

package/src/components/Expander/Expander.static.ts

@@ -13,35 +13,34 @@ export default class Expander {
   }
 
   toggle() {
-    console.log('Toggle event triggered');
-    const summary = this.element.querySelector('summary');
-    
+    const summary = this.element.querySelector("summary");
+
     if (!summary) {
       return;
     }
-    
-    const summaryText = summary.getAttribute('data-summary');
-    const summaryOpenedText = summary.getAttribute('data-summary-opened');
+
+    const summaryText = summary.getAttribute("data-summary");
+    const summaryOpenedText = summary.getAttribute("data-summary-opened");
 
     summary.setAttribute(
-      'aria-expanded',
-      summary.getAttribute('aria-expanded') === 'true' ? 'false' : 'true'
+      "aria-expanded",
+      summary.getAttribute("aria-expanded") === "true" ? "false" : "true",
     );
 
     if (summaryOpenedText) {
       const text =
-        summary.getAttribute('aria-expanded') === 'true'
+        summary.getAttribute("aria-expanded") === "true"
           ? summaryOpenedText
           : summaryText;
 
       if (text) {
-        summary.setAttribute('aria-label', text);
+        summary.setAttribute("aria-label", text);
       }
     }
   }
 
   init() {
-    this.element.addEventListener('toggle', this.toggle);
+    this.element.addEventListener("toggle", this.toggle);
   }
 
   update() {
@@ -50,7 +49,7 @@ export default class Expander {
   }
 
   destroy() {
-    this.element.removeEventListener('toggle', this.toggle);
+    this.element.removeEventListener("toggle", this.toggle);
   }
 
   static getInstance(el: HTMLElementWithExpander | null): Expander | null {

package/src/components/Expander/Expander.tsx

@@ -11,9 +11,6 @@ import ExpanderStatic from "./Expander.static";
 const CLASS_ROOT = "expander";
 
 interface ExpanderProps {
-  /** Inverse buttons for dark backgrounds */
-  /** Dark theme */
-  colorScheme?: "light" | "dark";
   /** Custom summary text renderer. Passes props as function parameter. */
   renderSummary?: (props: ExpanderProps) => React.ReactNode;
   /** Custom summary text renderer when details are opened. Passes props as function parameter. */
@@ -26,8 +23,8 @@ interface ExpanderProps {
   className?: string;
   /** Child elements */
   children?: React.ReactNode;
-  /** Additional HTML attributes */
-  [key: string]: any;
+  /** Expander takes full width of its container */
+  isFullWidth?: boolean;
 }
 
 export const Expander: React.FC<ExpanderProps> = (props) => {
@@ -38,7 +35,7 @@ export const Expander: React.FC<ExpanderProps> = (props) => {
     summaryOpened,
     renderSummary,
     renderSummaryOpened,
-    colorScheme,
+    isFullWidth,
     ...other
   } = props;
   const [expanderRef] = useStatic(ExpanderStatic);
@@ -46,8 +43,7 @@ export const Expander: React.FC<ExpanderProps> = (props) => {
   const classes = cx(
     CLASS_ROOT,
     {
-      "is-dark": colorScheme === "dark",
-      "is-light": colorScheme === "light",
+      [`${CLASS_ROOT}--fullwidth`]: isFullWidth,
     },
     className,
   );

package/src/components/Expander/styles/style.scss

@@ -1,15 +1,17 @@
 @use "sass:math";
+@use "../../../styles/tools/convert";
 @use "../../../styles/tokens/base";
 @use "../../../styles/tokens/color";
 @use "../../../styles/tokens/space";
 
 @layer components {
   .expander {
-    margin-bottom: space.get();
+    margin-bottom: space.get("large");
+    font-size: convert.to-rem(18px);
 
     summary {
       font-weight: 700;
-      line-height: 1.25;
+      line-height: normal;
       display: flex;
       align-items: center;
 
@@ -41,13 +43,20 @@
       display: none;
     }
 
+    &--fullwidth {
+      .expander__summary {
+        width: 100%;
+        justify-content: space-between;
+      }
+    }
+
     &[open] {
       svg[data-summary-icon] {
         transform: rotate(180deg) !important;
       }
 
       summary {
-        margin-bottom: space.get();
+        margin-bottom: space.get("large");
       }
 
       .expander__summary:has(+ .expander__summary--opened) {

package/src/components/Expander/tests/Expander.conformance.test.js

@@ -1,9 +1,9 @@
-import React from 'react';
-import { render } from '@testing-library/react';
-import { axe } from 'jest-axe';
+import React from "react";
+import { render } from "@testing-library/react";
+import { axe } from "jest-axe";
 
-import { Expander } from '../Expander';
-import { Icon } from '../../Icon';
+import { Expander } from "../Expander";
+import { Icon } from "../../Icon";
 
 const example = (
   <>
@@ -13,10 +13,6 @@ const example = (
       Content
     </Expander>
 
-    <Expander summary="Show details" colorScheme="dark">
-      Content
-    </Expander>
-
     <Expander
       summary="Custom summary"
       renderSummary={({ summary }) => (
@@ -44,16 +40,16 @@ const example = (
   </>
 );
 
-it('is valid html', () => {
+it("is valid html", () => {
   const { container } = render(example);
   expect(container).toHTMLValidate({
     rules: {
-      'element-permitted-content': 'off',
+      "element-permitted-content": "off",
     },
   });
 });
 
-it('is accessible', async () => {
+it("is accessible", async () => {
   const { container } = render(example);
   expect(await axe(container)).toHaveNoViolations();
 });

package/src/components/Expander/tests/Expander.unit.test.js

@@ -1,42 +1,32 @@
-import React from 'react';
-import { render, screen, fireEvent } from '@testing-library/react';
+import React from "react";
+import { render, screen, fireEvent } from "@testing-library/react";
 
-import { Expander } from '../Expander';
-import { Icon } from '../../Icon';
+import { Expander } from "../Expander";
+import { Icon } from "../../Icon";
 
-describe('Expander', () => {
-  it('renders basic expander with summary text', () => {
+describe("Expander", () => {
+  it("renders basic expander with summary text", () => {
     render(<Expander summary="Show details">Content</Expander>);
-    expect(screen.getByText('Show details')).toBeInTheDocument();
-    expect(screen.getByText('Content')).toBeInTheDocument();
+    expect(screen.getByText("Show details")).toBeInTheDocument();
+    expect(screen.getByText("Content")).toBeInTheDocument();
   });
 
-  it('changes summary text when opened', () => {
+  it("changes summary text when opened", () => {
     render(
       <Expander summary="Show details" summaryOpened="Hide details">
         Content
-      </Expander>
+      </Expander>,
     );
 
-    expect(screen.getByText('Show details')).toBeInTheDocument();
+    expect(screen.getByText("Show details")).toBeInTheDocument();
 
     // Click to open the expander
-    fireEvent.click(screen.getByText('Show details'));
+    fireEvent.click(screen.getByText("Show details"));
 
-    expect(screen.getByText('Hide details')).toBeInTheDocument();
+    expect(screen.getByText("Hide details")).toBeInTheDocument();
   });
 
-  it('supports inverse mode', () => {
-    const { container } = render(
-      <Expander summary="Show details" colorScheme="dark">
-        Content
-      </Expander>
-    );
-
-    expect(container.querySelector('.expander.is-dark')).toBeInTheDocument();
-  });
-
-  it('renders custom summary with renderSummary prop', () => {
+  it("renders custom summary with renderSummary prop", () => {
     render(
       <Expander
         summary="Custom summary"
@@ -48,15 +38,15 @@ describe('Expander', () => {
         )}
       >
         Content
-      </Expander>
+      </Expander>,
     );
 
-    expect(screen.getByTestId('custom-summary')).toHaveTextContent(
-      'Custom summary'
+    expect(screen.getByTestId("custom-summary")).toHaveTextContent(
+      "Custom summary",
     );
   });
 
-  it('renders custom summary when opened with renderSummaryOpened prop', () => {
+  it("renders custom summary when opened with renderSummaryOpened prop", () => {
     render(
       <Expander
         summary="Show details"
@@ -69,46 +59,46 @@ describe('Expander', () => {
         )}
       >
         Content
-      </Expander>
+      </Expander>,
     );
 
     // Click to open the expander
-    fireEvent.click(screen.getByText('Show details'));
+    fireEvent.click(screen.getByText("Show details"));
 
-    expect(screen.getByTestId('custom-summary-opened')).toHaveTextContent(
-      'Hide details'
+    expect(screen.getByTestId("custom-summary-opened")).toHaveTextContent(
+      "Hide details",
     );
   });
 
-  it('toggles content visibility when clicked', () => {
+  it("toggles content visibility when clicked", () => {
     const { container } = render(
       <Expander summary="Show details">
         <p data-testid="content">Hidden content</p>
-      </Expander>
+      </Expander>,
     );
 
-    const details = container.querySelector('details');
-    expect(details).not.toHaveAttribute('open');
+    const details = container.querySelector("details");
+    expect(details).not.toHaveAttribute("open");
 
     // Click to open
-    fireEvent.click(screen.getByText('Show details'));
-    expect(details).toHaveAttribute('open');
+    fireEvent.click(screen.getByText("Show details"));
+    expect(details).toHaveAttribute("open");
 
     // Click to close
-    fireEvent.click(screen.getByText('Show details'));
-    expect(details).not.toHaveAttribute('open');
+    fireEvent.click(screen.getByText("Show details"));
+    expect(details).not.toHaveAttribute("open");
   });
 
-  it('has default class expander', () => {
+  it("has default class expander", () => {
     const { getByTestId } = render(
       <Expander data-testid="test-id" summary="Show details">
         Content
-      </Expander>
+      </Expander>,
     );
-    expect(getByTestId('test-id')).toHaveClass('expander');
+    expect(getByTestId("test-id")).toHaveClass("expander");
   });
 
-  it('has additional class when className is set', () => {
+  it("has additional class when className is set", () => {
     const { getByTestId } = render(
       <Expander
         data-testid="test-id"
@@ -116,9 +106,9 @@ describe('Expander', () => {
         summary="Show details"
       >
         Content
-      </Expander>
+      </Expander>,
     );
-    expect(getByTestId('test-id')).toHaveClass('expander');
-    expect(getByTestId('test-id')).toHaveClass('test-class');
+    expect(getByTestId("test-id")).toHaveClass("expander");
+    expect(getByTestId("test-id")).toHaveClass("test-class");
   });
 });

package/src/components/FeatureAccordion/FeatureAccordion.tsx

@@ -12,8 +12,6 @@ import FeatureAccordionStatic from "./FeatureAccordion.static";
 const CLASS_ROOT = "feature-accordion";
 
 interface AccordionProps {
-  /** Color scheme for whole FeatureAccordion */
-  colorScheme?: "light" | "dark" | "white";
   /** Additional CSS classes */
   className?: string;
 }
@@ -21,18 +19,10 @@ interface AccordionProps {
 const FeatureAccordion = (
   props: AccordionProps & { children: React.ReactNode },
 ) => {
-  const { className, children, colorScheme, ...other } = props;
+  const { className, children, ...other } = props;
   const [featureAccordionRef] = useStatic(FeatureAccordionStatic);
 
-  const classes = cx(
-    CLASS_ROOT,
-    {
-      "is-dark": colorScheme === "dark",
-      "is-light": colorScheme === "light",
-      "is-white": colorScheme === "white",
-    },
-    className,
-  );
+  const classes = cx(CLASS_ROOT, className);
 
   const childrenArray = useMemo(
     () => React.Children.toArray(children) as React.ReactElement<any>[],
@@ -88,7 +78,7 @@ const FeatureAccordion = (
           xs: 12,
           md: 6,
         }}
-        className={cx(`${CLASS_ROOT}__preview`, "hide", "show-md")}
+        className={cx(`${CLASS_ROOT}__preview`, "show-md")}
       >
         {childrenArray.map((child) => {
           const id = child.props.id;

package/src/components/FeatureAccordion/styles/mixins.scss

@@ -4,18 +4,14 @@
 @use "../../../styles/tokens/space";
 @use "../../../styles/tokens/breakpoint";
 
-@mixin accordion {
-  @include breakpoint.get("md") {
-    color: var(--color-text-default);
-    background-color: var(--color-fill-subtle);
-    border-color: var(--color-fill-disabled);
-  }
+@mixin base {
+  color: var(--color-text-default);
+  background-color: var(--color-fill-subtle);
+  border-color: var(--color-fill-disabled);
+}
 
+@mixin accordion {
   .accordion {
-    color: var(--color-text-default);
-    background-color: var(--color-fill-subtle);
-    border-color: var(--color-fill-disabled);
-
     margin: convert.to-rem(87px) space.get("xlarge");
 
     @include breakpoint.get("md", "down") {
@@ -32,10 +28,6 @@
   display: flex;
   padding: 0px !important;
 
-  color: var(--color-text-default);
-  background-color: var(--color-fill-subtle);
-  border-color: var(--color-fill-disabled);
-
   img {
     width: 100%;
     height: auto;

package/src/components/FeatureAccordion/styles/style.scss

@@ -2,6 +2,8 @@
 
 @layer components {
   .feature-accordion {
+    @include mixins.base();
+
     &__accordion {
       @include mixins.accordion();
     }
@@ -13,9 +15,5 @@
     &__item {
       @include mixins.item();
     }
-
-    &.is-white {
-      --color-fill-subtle: #fff;
-    }
   }
 }

package/src/components/FeatureAccordion/tests/FeatureAccordion.conformance.test.js

@@ -28,6 +28,8 @@ describe("FeatureAccordion conformance", () => {
     expect(container).toHTMLValidate({
       rules: {
         "attribute-boolean-style": "off",
+        "prefer-native-element": "off",
+        "attribute-empty-style": "off",
       },
     });
   });

package/src/components/FeatureAccordion/tests/FeatureAccordion.unit.test.js

@@ -93,9 +93,4 @@ describe("FeatureAccordion", () => {
     const { container } = renderAccordion({ "data-testid": "fa" });
     expect(container.firstChild).toHaveAttribute("data-testid", "fa");
   });
-
-  it("applies is-white class when colorScheme='white'", () => {
-    const { container } = renderAccordion({ colorScheme: "white" });
-    expect(container.firstChild).toHaveClass("is-white");
-  });
 });

package/src/components/Footer/Footer.tsx

@@ -1,90 +1,129 @@
 import React from "react";
 import cx from "classnames";
 
-import { Section } from "../Section";
-import { SectionProps } from "../Section/Section";
-import { Image } from "../Image";
-import { Grid, GridCol } from "../Grid";
-import { Link } from "../Link";
-import footerData from "./data";
-import { Bar, BarItem } from "../Bar";
+import footerData, { socials } from "./data";
+import {
+  CLASS_BOTTOM,
+  CLASS_BOTTOM_COPYRIGHT,
+  CLASS_CONTAINER,
+  CLASS_GRID,
+  CLASS_LIST,
+  CLASS_LIST_INLINE,
+  CLASS_LOGO,
+  CLASS_ROOT,
+  CLASS_SOCIALS,
+  CLASS_TITLE,
+} from "./constants";
+import { SocialButton } from "../SocialButton";
+import { PlanObnovyImage } from "./PlanObnovy";
 
 interface FooterProps extends React.HTMLAttributes<HTMLDivElement> {
-  /** Sets Footer to use inverse color scheme */
-  isInverse?: boolean;
-  /** Custom data to override default footer data */
   data?: typeof footerData;
   className?: string;
 }
 
-const CLASS_ROOT = "footer";
-
-const Footer: React.FC<FooterProps & SectionProps> = ({
-  className,
-  isInverse,
-  data = footerData,
-  ...other
-}) => {
-  const classes = cx(
-    CLASS_ROOT,
-    {
-      [`${CLASS_ROOT}--inverse`]: isInverse,
-    },
-    className,
-  );
-
+const Footer: React.FC<FooterProps> = ({ className, data = footerData }) => {
   return (
-    <Section tag="footer" className={classes} {...other}>
-      <Image src="/logo.svg" alt="logo" className="mb-xlarge" />
-      <Grid rowGapSize="medium">
-        {data.columns.map((column, index) => (
-          <GridCol
-            size={{
-              lg: 3,
-              md: 4,
-              sm: 6,
-              xs: 12,
-            }}
-            key={`footer-column-${index}`}
+    <footer className={cx(CLASS_ROOT, className)}>
+      <div className={CLASS_CONTAINER}>
+        <a href="/" aria-label="Odkaz na úvodnú stránku" className={CLASS_LOGO}>
+          <svg
+            width="48"
+            height="48"
+            viewBox="0 0 48 48"
+            fill="none"
+            xmlns="http://www.w3.org/2000/svg"
           >
-            <h5 className="mb-large">{column.title}</h5>
-            <ul className="list-unstyled">
-              {column.links.map((link, linkIndex) => (
-                <li
-                  className="mb-small"
-                  key={`footer-link-${index}-${linkIndex}`}
-                >
-                  <Link className={`${CLASS_ROOT}__link`} href={link.url}>
-                    {link.label}
-                  </Link>
-                </li>
-              ))}
-            </ul>
-          </GridCol>
-        ))}
-      </Grid>
+            <path d="M48 0H0V48H48V0Z" fill="black" />
+            <path
+              fillRule="evenodd"
+              clipRule="evenodd"
+              d="M48 48H0V0H48V48Z"
+              fill="#FF7900"
+            />
+            <path
+              d="M5.9078 44.1298C4.1702 44.1298 2.60156 43.0229 2.60156 40.6047C2.60156 38.1855 4.16924 37.0796 5.9078 37.0796C7.6454 37.0796 9.21116 38.1865 9.21116 40.6047C9.21116 43.0229 7.64636 44.1298 5.9078 44.1298ZM5.9078 38.5657C4.59356 38.5657 4.35356 39.7494 4.35356 40.6047C4.35356 41.4591 4.59356 42.6457 5.9078 42.6457C7.22204 42.6457 7.46204 41.4601 7.46204 40.6047C7.46204 39.7503 7.22204 38.5657 5.9078 38.5657Z"
+              fill="white"
+            />
+            <path
+              fillRule="evenodd"
+              clipRule="evenodd"
+              d="M10.418 37.2559H12.0912V38.0412C12.41 37.6179 13.1952 37.1196 13.9316 37.1196C14.0016 37.1196 14.088 37.1196 14.16 37.1283V38.7795H14.0736C13.3095 38.7795 12.4733 38.8985 12.2026 39.4927V43.9356H10.418V37.2559Z"
+              fill="white"
+            />
+            <path
+              d="M18.831 43.3547C18.1777 43.7913 17.4087 44.0223 16.623 44.0181C15.375 44.0181 14.6367 43.1887 14.6367 42.0751C14.6367 40.5794 16.0114 39.7903 18.8444 39.4706V39.0981C18.8444 38.6133 18.4767 38.3291 17.7961 38.3291C17.1164 38.3291 16.5634 38.5989 16.165 39.0971L14.9794 38.4223C15.6015 37.5506 16.55 37.1147 17.8239 37.1147C19.5673 37.1147 20.5455 37.8703 20.5455 39.0981C20.5455 39.0981 20.5436 43.9269 20.5455 43.9394H18.9865L18.831 43.3547ZM16.3686 41.9474C16.3686 42.3967 16.6498 42.8162 17.1606 42.8162C17.7126 42.8162 18.2511 42.5877 18.7887 42.1125V40.5304C17.1471 40.733 16.3686 41.1573 16.3686 41.9474Z"
+              fill="white"
+            />
+            <path
+              fillRule="evenodd"
+              clipRule="evenodd"
+              d="M21.9336 37.4342L23.3899 37.2336L23.5512 38.0256C24.371 37.4246 25.0229 37.1059 25.8437 37.1059C27.2184 37.1059 27.9288 37.8355 27.9288 39.2793V43.9296H26.1691V39.5846C26.1691 38.7667 25.9579 38.399 25.3214 38.399C24.7973 38.399 24.2731 38.6361 23.6741 39.1392V43.9296H21.9336V37.4342Z"
+              fill="white"
+            />
+            <path
+              d="M32.2317 42.5003C33.5642 42.4907 33.6784 41.1323 33.6784 40.2491C33.6784 39.2008 33.1677 38.355 32.2192 38.355C31.5885 38.355 30.8819 38.8158 30.8819 40.3221C30.8819 41.1438 30.9405 42.5118 32.2317 42.5003ZM35.3671 37.2328V43.5582C35.3671 44.6718 35.2787 46.515 32.0983 46.5314C30.785 46.541 29.5658 46.013 29.3239 44.8677L31.0567 44.5874C31.1296 44.9166 31.3322 45.2421 32.3152 45.2421C33.2263 45.2421 33.664 44.8072 33.664 43.7637V42.9851L33.641 42.9611C33.3616 43.4622 32.9373 43.9355 31.9168 43.9355C30.3578 43.9355 29.1328 42.8546 29.1328 40.5966C29.1328 38.3627 30.3981 37.1118 31.817 37.109C33.1514 37.107 33.6429 37.7147 33.76 38.0334L33.736 38.0315L33.8829 37.2328H35.3671ZM39.5709 44.0584C37.6029 44.0584 36.4279 42.7912 36.4279 40.5928C36.4279 38.3704 37.6163 37.1051 39.5431 37.1051C41.4698 37.1051 42.616 38.3339 42.616 40.5045C42.616 40.6216 42.6016 40.7349 42.6016 40.8501H38.1539C38.1664 42.1173 38.6925 42.7557 39.7274 42.7557C40.3907 42.7557 40.8314 42.4869 41.2442 41.9003L42.5306 42.6136C41.9642 43.5611 40.9427 44.0584 39.5709 44.0584ZM40.8899 39.7096C40.8899 38.8178 40.3773 38.2917 39.5431 38.2917C38.7482 38.2917 38.2519 38.8062 38.1827 39.7106L40.8899 39.7096ZM45.6256 34.7214L44.9776 36.3966L44.3191 34.7214H43.7546V36.9294H44.1203V35.0872H44.1299L44.8547 36.9304H45.088L45.808 35.0872H45.8195V36.9304H46.1824V34.7224L45.6256 34.7214ZM41.56 34.7214V35.0152H42.2503V36.9304H42.6151V35.0152H43.3072V34.7214H41.56Z"
+              fill="white"
+            />
+          </svg>
+        </a>
+
+        <div className={CLASS_GRID}>
+          {data.columns.map((column, index) => (
+            <div key={`footer-column-${index}`}>
+              <p className={CLASS_TITLE}>{column.title}</p>
+              <ul className={CLASS_LIST}>
+                {column.links.map((link, linkIndex) => (
+                  <li
+                    className="mb-small"
+                    key={`footer-link-${index}-${linkIndex}`}
+                  >
+                    <a href={link.url}>{link.label}</a>
+                  </li>
+                ))}
+              </ul>
+            </div>
+          ))}
+        </div>
 
-      <hr />
-      {/* TODO - socky siete */}
-      <p className="small">Sledujte nás</p>
+        <hr />
+
+        <div className={CLASS_SOCIALS}>
+          <ul className={cx(CLASS_LIST, CLASS_LIST_INLINE)}>
+            <li>Sledujte nás</li>
+            {socials.map((social, index) => (
+              <li className="mr-small" key={`footer-social-link-${index}`}>
+                <SocialButton
+                  name={social.name}
+                  href={social.url}
+                  aria-label={social.label}
+                />
+              </li>
+            ))}
+          </ul>
+
+          <a
+            href="https://www.orange.sk/orange-slovensko/plan-obnovy"
+            aria-label="Plán obnovy"
+          >
+            <PlanObnovyImage />
+          </a>
+        </div>
 
-      <hr />
+        <hr />
 
-      <Grid className="align-items-end" rowGapSize="xlarge">
-        <GridCol size={{ xs: 12, md: 9 }}>
-          <Bar className="mb-none" space="small">
+        <div className={CLASS_BOTTOM}>
+          <ul className={cx(CLASS_LIST, CLASS_LIST_INLINE)}>
             {data.bottomLinks?.map((link, index) => (
-              <BarItem key={`footer-bottom-link-${index}`}>
-                <Link href={link.url} isNormal>
-                  {link.label}
-                </Link>
-              </BarItem>
+              <li className="mr-small" key={`footer-legal-link-${index}`}>
+                <a href={link.url}>{link.label}</a>
+              </li>
             ))}
-          </Bar>
-        </GridCol>
-        <GridCol size={{ xs: 12, md: 3 }}>{data.copyright}</GridCol>
-      </Grid>
-    </Section>
+          </ul>
+          <span className={CLASS_BOTTOM_COPYRIGHT}>{data.copyright}</span>
+        </div>
+      </div>
+    </footer>
   );
 };