@orangesk/orange-design-system 2.0.0-beta.11 → 2.0.0-beta.12

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

@@ -1,113 +1,113 @@
-import React from 'react';
-import { render } from '@testing-library/react';
+import React from "react";
+import { render } from "@testing-library/react";
 
-import { Cover } from '../Cover';
+import { Cover } from "../Cover";
 
-const image = '/test-image.jpg';
+const image = "/test-image.jpg";
 const picture = {
-  default: '/test-picture.jpg',
-  md: '/test-picture-md.jpg',
-  xl: '/test-picture-xl.jpg',
+  default: "/test-picture.jpg",
+  md: "/test-picture-md.jpg",
+  xl: "/test-picture-xl.jpg",
 };
 
 // bgSrc prop is required
-describe('rendering Cover', () => {
-  describe('initial state', () => {
-    it('has default class cover', () => {
+describe("rendering Cover", () => {
+  describe("initial state", () => {
+    it("has default class cover", () => {
       const { container } = render(<Cover bgSrc={image} />);
-      expect(container.getElementsByClassName('cover').length).toBe(1);
+      expect(container.getElementsByClassName("cover").length).toBe(1);
     });
-    it('has default class cover__image', () => {
+    it("has default class cover__image", () => {
       const { container } = render(<Cover bgSrc={image} />);
-      expect(container.getElementsByClassName('cover__image').length).toBe(1);
+      expect(container.getElementsByClassName("cover__image").length).toBe(1);
     });
   });
-  describe('passed props', () => {
-    it('passes other props', () => {
+  describe("passed props", () => {
+    it("passes other props", () => {
       const { getByTestId } = render(
-        <Cover data-testid="test" bgSrc={image} />
+        <Cover data-testid="test" bgSrc={image} />,
       );
-      expect(getByTestId('test')).toBeInTheDocument();
+      expect(getByTestId("test")).toBeInTheDocument();
     });
-    it('renders children', () => {
+    it("renders children", () => {
       const { getByText } = render(<Cover bgSrc={image}>test-text</Cover>);
-      expect(getByText('test-text')).toBeInTheDocument();
+      expect(getByText("test-text")).toBeInTheDocument();
     });
-    it('has additional class when className is set', () => {
+    it("has additional class when className is set", () => {
       const { container } = render(
-        <Cover bgSrc={image} className="test-class" />
+        <Cover bgSrc={image} className="test-class" />,
       );
-      expect(container.getElementsByClassName('cover').length).toBe(1);
-      expect(container.getElementsByClassName('test-class').length).toBe(1);
+      expect(container.getElementsByClassName("cover").length).toBe(1);
+      expect(container.getElementsByClassName("test-class").length).toBe(1);
     });
-    it('has class cover__content when children are set', () => {
+    it("has class cover__content when children are set", () => {
       const { container } = render(<Cover bgSrc={image}>test</Cover>);
-      expect(container.getElementsByClassName('cover__content').length).toBe(1);
+      expect(container.getElementsByClassName("cover__content").length).toBe(1);
     });
-    it('has additional class when contentClass and children are set', () => {
+    it("has additional class when contentClass and children are set", () => {
       const { container } = render(
         <Cover bgSrc={image} contentClass="test-content-class">
           test
-        </Cover>
+        </Cover>,
       );
       expect(
-        container.getElementsByClassName('test-content-class').length
+        container.getElementsByClassName("test-content-class").length,
       ).toBe(1);
     });
-    ['small', 'medium', 'large'].map(size => {
+    ["small", "medium", "large", "xlarge"].map((size) => {
       it(`has cover--${size} class when size is set to ${size}`, () => {
         const { container } = render(<Cover bgSrc={image} size={size} />);
         expect(container.getElementsByClassName(`cover--${size}`).length).toBe(
-          1
+          1,
         );
       });
     });
-    ['top', 'bottom'].map(position => {
+    ["top", "bottom"].map((position) => {
       it(`has cover__image--${position} class when bgPosition is set to ${position}`, () => {
         const { container } = render(
-          <Cover bgSrc={image} bgPosition={position} />
+          <Cover bgSrc={image} bgPosition={position} />,
         );
         expect(
-          container.getElementsByClassName(`cover__image--${position}`).length
+          container.getElementsByClassName(`cover__image--${position}`).length,
         ).toBe(1);
       });
     });
-    it('doesnt have cover__image--center class when bgPosition is set to center', () => {
+    it("doesnt have cover__image--center class when bgPosition is set to center", () => {
       const { container } = render(<Cover bgSrc={image} bgPosition="center" />);
       expect(
-        container.getElementsByClassName('cover__image--center').length
+        container.getElementsByClassName("cover__image--center").length,
       ).toBe(0);
     });
-    it('renders img element, not picture, when bgSrc is set to string', () => {
+    it("renders img element, not picture, when bgSrc is set to string", () => {
       const { container } = render(<Cover bgSrc={image} />);
-      expect(container.querySelectorAll('img').length).toBe(1);
-      expect(container.querySelectorAll('picture').length).toBe(0);
+      expect(container.querySelectorAll("img").length).toBe(1);
+      expect(container.querySelectorAll("picture").length).toBe(0);
     });
-    it('img element has src set to bgSrc when bgSrc is string', () => {
+    it("img element has src set to bgSrc when bgSrc is string", () => {
       const { container } = render(<Cover bgSrc={image} />);
-      expect(container.querySelector('img').getAttribute('src')).toBe(
-        '/test-image.jpg'
+      expect(container.querySelector("img").getAttribute("src")).toBe(
+        "/test-image.jpg",
       );
     });
-    it('renders picture element with source and img elements when bgSrc is set to object with multiple properties', () => {
+    it("renders picture element with source and img elements when bgSrc is set to object with multiple properties", () => {
       const { container } = render(<Cover bgSrc={picture} />);
-      expect(container.querySelectorAll('picture').length).toBe(1);
-      expect(container.querySelectorAll('img').length).toBe(1);
-      expect(container.querySelectorAll('source').length > 0).toBeTruthy();
+      expect(container.querySelectorAll("picture").length).toBe(1);
+      expect(container.querySelectorAll("img").length).toBe(1);
+      expect(container.querySelectorAll("source").length > 0).toBeTruthy();
     });
-    it('img element has src set to bgSrc.default when bgSrc is object', () => {
+    it("img element has src set to bgSrc.default when bgSrc is object", () => {
       const { container } = render(<Cover bgSrc={picture} />);
-      expect(container.querySelector('img').getAttribute('src')).toBe(
-        picture.default
+      expect(container.querySelector("img").getAttribute("src")).toBe(
+        picture.default,
       );
     });
     Object.keys(picture)
-      .filter(el => el !== 'default')
-      .map(key => {
-        it('srcSet of source element has correct value from bgSrc object', () => {
+      .filter((el) => el !== "default")
+      .map((key) => {
+        it("srcSet of source element has correct value from bgSrc object", () => {
           const { container } = render(<Cover bgSrc={picture} />);
           const correctSource = container.querySelector(
-            `source[srcset="${picture[key]}"]`
+            `source[srcset="${picture[key]}"]`,
           );
           expect(correctSource).toBeInTheDocument();
         });

package/src/components/Forms/Group/styles/mixins.scss

@@ -34,6 +34,20 @@
     border-top-left-radius: 0;
     border-bottom-left-radius: 0;
   }
+
+  > *:not(:first-child):not(:last-child) {
+    border-radius: 0 !important;
+  }
+
+  > *:first-child:not(:last-child) {
+    border-top-right-radius: 0 !important;
+    border-bottom-right-radius: 0 !important;
+  }
+
+  > *:last-child:not(:first-child) {
+    border-top-left-radius: 0 !important;
+    border-bottom-left-radius: 0 !important;
+  }
 }
 
 @mixin item($config: config.$item) {

package/src/components/Pill/Pill.tsx

@@ -3,7 +3,15 @@ import cx from "classnames";
 
 const CLASS_ROOT = "pill";
 
-export const pillColors = ["white", "gray", "transparent", "orange"] as const;
+export const pillColors = [
+  "surface-primary",
+  "surface-secondary",
+  "surface-tertiary",
+  "surface-subtle",
+  "surface-moderate",
+  "surface-accent",
+  "transparent",
+] as const;
 export type PillColor = (typeof pillColors)[number];
 
 interface PillProps extends HTMLAttributes<HTMLSpanElement> {
@@ -26,11 +34,11 @@ const Pill: React.FC<PillProps> = ({
     className={cx(
       CLASS_ROOT,
       {
-        [`${CLASS_ROOT}--${color}`]: color, // Only apply modifier if color is specified
         "is-light": colorScheme === "light",
         "is-dark": colorScheme === "dark",
         [CLASS_ROOT + "--" + size]: size,
       },
+      color,
       className,
     )}
     {...other}

package/src/components/Pill/styles/config.scss

@@ -4,8 +4,8 @@
 
 $base: (
   font-weight: bold,
-  background-color: map.get(color.$colors-dark, border-subtle),
-  color: color.$white,
+  background-color: var(--color-surface-contrast),
+  color: var(--color-text-inverse),
 );
 
 $sizes: (
@@ -21,22 +21,3 @@ $sizes: (
     xxlarge: convert.to-rem(64px),
   ),
 );
-
-$colors: (
-  "white": (
-    background-color: color.$white,
-    color: color.$black,
-  ),
-  "gray": (
-    background-color: var(--color-surface-subtle),
-    color: var(--color-text-default),
-  ),
-  "orange": (
-    background-color: var(--color-icon-brand),
-    color: var(--color-text-inverse),
-  ),
-  "transparent": (
-    background-color: rgba(#ffffff, 0.5),
-    color: var(--color-text-default),
-  ),
-);

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

@@ -1,3 +1,4 @@
+@use "../../../styles/tokens/color";
 @use "../../../styles/tools/generate";
 @use "./config";
 @use "./mixins";
@@ -8,12 +9,21 @@
     @include mixins.base();
     @include mixins.sizes();
 
-    @each $colorName, $colorProps in config.$colors {
-      &--#{$colorName} {
-        @each $property, $value in $colorProps {
-          #{$property}: $value;
-        }
-      }
+    &.surface-primary,
+    &.surface-subtle,
+    &.surface-moderate,
+    &.surface-accent {
+      color: var(--color-text-default);
+    }
+
+    &.surface-secondary,
+    &.surface-tertiary {
+      color: color.$white;
+    }
+
+    &.transparent {
+      background-color: rgba(#ffffff, 0.5);
+      color: var(--color-text-default);
     }
   }
 }

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

@@ -19,13 +19,19 @@ const example = (
     </p>
     <p>
       <Pill color="transparent">T</Pill>
-      <Pill color="black">D</Pill>
-      <Pill color="white">L</Pill>
+      <Pill color="surface-secondary">S</Pill>
+      <Pill color="surface-tertiary">T</Pill>
+      <Pill color="surface-subtle">Su</Pill>
+      <Pill color="surface-moderate">M</Pill>
+      <Pill color="surface-contrast">C</Pill>
+      <Pill color="surface-accent">A</Pill>
     </p>
     <p>
       <Pill size="small">S</Pill>
       <Pill size="medium">M</Pill>
       <Pill size="large">L</Pill>
+      <Pill size="xlarge">XL</Pill>
+      <Pill size="xxlarge">XXL</Pill>
     </p>
     <p>
       <Pill colorScheme="light">Light</Pill>

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

@@ -35,28 +35,78 @@ describe("rendering Pill", () => {
       expect(getByTestId("test-id")).toHaveClass("pill--large");
     });
 
-    it("has default black styling without modifier", () => {
+    it("supports all available sizes", () => {
+      const sizes = ["small", "medium", "large", "xlarge", "xxlarge"];
+
+      sizes.forEach((size) => {
+        const { getByTestId } = render(
+          <Pill size={size} data-testid={`test-${size}`}>
+            {size}
+          </Pill>,
+        );
+
+        expect(getByTestId(`test-${size}`)).toHaveClass(`pill--${size}`);
+      });
+    });
+
+    it("has default styling without color modifier", () => {
       const { getByTestId } = render(<Pill data-testid="test-id">pill</Pill>);
       expect(getByTestId("test-id")).toHaveClass("pill");
-      expect(getByTestId("test-id")).not.toHaveClass("pill--black");
+      expect(getByTestId("test-id")).not.toHaveClass("surface-secondary");
+    });
+
+    it("has surface-secondary class when color is surface-secondary", () => {
+      const { getByTestId } = render(
+        <Pill color="surface-secondary" data-testid="test-id">
+          pill
+        </Pill>,
+      );
+      expect(getByTestId("test-id")).toHaveClass("surface-secondary");
+    });
+
+    it("has surface-tertiary class when color is surface-tertiary", () => {
+      const { getByTestId } = render(
+        <Pill color="surface-tertiary" data-testid="test-id">
+          pill
+        </Pill>,
+      );
+      expect(getByTestId("test-id")).toHaveClass("surface-tertiary");
+    });
+
+    it("has surface-subtle class when color is surface-subtle", () => {
+      const { getByTestId } = render(
+        <Pill color="surface-subtle" data-testid="test-id">
+          pill
+        </Pill>,
+      );
+      expect(getByTestId("test-id")).toHaveClass("surface-subtle");
+    });
+
+    it("has surface-moderate class when color is surface-moderate", () => {
+      const { getByTestId } = render(
+        <Pill color="surface-moderate" data-testid="test-id">
+          pill
+        </Pill>,
+      );
+      expect(getByTestId("test-id")).toHaveClass("surface-moderate");
     });
 
-    it("has white class when color is white", () => {
+    it("has surface-contrast class when color is surface-contrast", () => {
       const { getByTestId } = render(
-        <Pill color="white" data-testid="test-id">
+        <Pill color="surface-contrast" data-testid="test-id">
           pill
         </Pill>,
       );
-      expect(getByTestId("test-id")).toHaveClass("pill--white");
+      expect(getByTestId("test-id")).toHaveClass("surface-contrast");
     });
 
-    it("has gray class when color is gray", () => {
+    it("has surface-accent class when color is surface-accent", () => {
       const { getByTestId } = render(
-        <Pill color="gray" data-testid="test-id">
+        <Pill color="surface-accent" data-testid="test-id">
           pill
         </Pill>,
       );
-      expect(getByTestId("test-id")).toHaveClass("pill--gray");
+      expect(getByTestId("test-id")).toHaveClass("surface-accent");
     });
 
     it("has transparent class when color is transparent", () => {
@@ -65,11 +115,19 @@ describe("rendering Pill", () => {
           pill
         </Pill>,
       );
-      expect(getByTestId("test-id")).toHaveClass("pill--transparent");
+      expect(getByTestId("test-id")).toHaveClass("transparent");
     });
 
     it("supports all available colors", () => {
-      const colors = ["white", "gray", "transparent"];
+      const colors = [
+        "surface-secondary",
+        "surface-tertiary",
+        "surface-subtle",
+        "surface-moderate",
+        "surface-contrast",
+        "surface-accent",
+        "transparent",
+      ];
 
       colors.forEach((color) => {
         const { getByTestId } = render(
@@ -78,7 +136,7 @@ describe("rendering Pill", () => {
           </Pill>,
         );
 
-        expect(getByTestId(`test-${color}`)).toHaveClass(`pill--${color}`);
+        expect(getByTestId(`test-${color}`)).toHaveClass(color);
       });
     });
 

package/src/components/Table/Row.tsx

@@ -14,8 +14,9 @@ interface RowProps {
 }
 
 const Row = ({ row }: RowProps) => {
-  const { headers, expandableRows } = React.useContext(TableContext);
-  return headers.map(({ key, cellProps }, index, array) => {
+  const { headers, expandableRows, isResponsive } =
+    React.useContext(TableContext);
+  return headers.map(({ key, label, cellProps }, index, array) => {
     const cell = row[key];
     const isObject = typeof cell === "object" && cell !== null;
     const value = isObject && "value" in cell ? cell.value : cell;
@@ -26,9 +27,14 @@ const Row = ({ row }: RowProps) => {
             .reduce((acc, [k, v]) => ({ ...acc, [k]: v }), {})
         : {};
     const rl = row?.expand?.length || 0;
+
     return (
       <React.Fragment key={key}>
-        <Cell {...cellProps} {...props}>
+        <Cell
+          {...cellProps}
+          {...props}
+          {...(isResponsive && { "data-label": label })}
+        >
           {value ? (value as React.ReactNode) : ""}
         </Cell>
         {index === array.length - 1 &&

package/src/components/Table/Table.tsx

@@ -23,6 +23,8 @@ interface TableProps extends React.TableHTMLAttributes<HTMLTableElement> {
   headers: TableHeader[];
   /** Compact table only takes spaced needed to display its content */
   isCompact?: boolean;
+  /** Enable responsive card layout below md breakpoint for better accessibility */
+  isResponsive?: boolean;
   /** If table has horizontal scrollbar */
   isScrollable?: boolean;
   /** Different colors for even and odd rows. */
@@ -43,6 +45,7 @@ const Table: React.FC<TableProps> = ({
   rows,
   footers,
   isCompact,
+  isResponsive,
   isStriped,
   isScrollable,
   colorScheme,
@@ -60,6 +63,7 @@ const Table: React.FC<TableProps> = ({
     {
       [`${CLASS_ROOT}--striped`]: isStriped,
       [`${CLASS_ROOT}--compact`]: isCompact,
+      [`${CLASS_ROOT}--responsive`]: isResponsive,
       "is-dark": colorScheme === "dark",
       "is-light": colorScheme === "light",
     },
@@ -94,6 +98,7 @@ const Table: React.FC<TableProps> = ({
     footers,
     rows: customRows,
     expandableRows,
+    isResponsive,
   };
 
   const table = (
@@ -108,7 +113,12 @@ const Table: React.FC<TableProps> = ({
           />
         }
       >
-        <table ref={tableRef} className={classes} {...other}>
+        <table
+          ref={tableRef}
+          className={classes}
+          {...(isResponsive && { "data-table": "responsive" })}
+          {...other}
+        >
           {caption && (
             <caption {...(isScrollable && { id: captionId })}>
               {caption}

package/src/components/Table/TableContext.ts

@@ -6,6 +6,7 @@ import { TableContextType } from "./types";
 const TableContext = React.createContext<TableContextType>({
   headers: [],
   rows: [],
+  isResponsive: false,
 });
 
 export default TableContext;

package/src/components/Table/docsData.ts

@@ -208,3 +208,28 @@ export const fillRows: TableRow[] = [
     col2: "3",
   },
 ];
+
+export const responsiveHeaders: TableHeader[] = [
+  {
+    key: "account",
+    label: "Účel spracovania",
+  },
+  {
+    key: "legal",
+    label: "Právny základ a doba spracovania",
+  },
+];
+
+export const responsiveRows: TableRow[] = [
+  {
+    account: "Mesačný poplatok (bez Digitálnej odmeny)",
+    legal:
+      "Právny základ - Súhlas (čl. 6 ods. 1 písm. a) GDPR) a zmluva (čl. 6 ods. 1 písm. b) GDPR) a oprávnený záujem (čl. 6 ods. 1 písm. f) GDPR). Všeobecná doba uchovávania – Počas trvania zmluvného vzťahu.",
+  },
+  {
+    account:
+      "Online predaj tovarov a poskytovanie doplnkových a asistenčných služieb",
+    legal:
+      "Právny základ - Zmluva (čl. 6 ods. 1 písm. b) GDPR) a oprávnený záujem (čl. 6 ods. 1 písm. f) GDPR). Všeobecná doba uchovávania – Počas trvania zmluvného vzťahu a uplynutia záručnej doby na predaný tovar.",
+  },
+];

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

@@ -112,3 +112,40 @@
 @mixin compact {
   width: auto;
 }
+
+@mixin responsive-table {
+  thead {
+    display: none;
+  }
+
+  tbody,
+  tbody tr,
+  tbody td {
+    display: block;
+  }
+
+  tbody tr {
+    padding: space.get("small") 0;
+
+    &:last-child {
+      margin-bottom: 0;
+    }
+  }
+
+  tbody td {
+    padding: space.get("medium") 0;
+    border-bottom: none;
+    text-align: left !important;
+
+    &:before {
+      content: attr(data-label);
+      display: block;
+      font-weight: 700;
+      margin-bottom: space.get("xsmall");
+    }
+
+    &:first-child {
+      padding-left: 0;
+    }
+  }
+}

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

@@ -47,6 +47,12 @@
       @include mixins.compact;
     }
 
+    &--responsive {
+      @include breakpoint.get("md", "down") {
+        @include mixins.responsive-table;
+      }
+    }
+
     @each $breakpoint in ("default", "sm", "md", "lg", "xl", "xxl") {
       @include breakpoint.get($breakpoint, "down") {
         &-scrollable#{generate.variant-name($breakpoint)} {

package/src/components/Table/types.ts

@@ -37,4 +37,5 @@ export interface TableContextType {
   rows: TableRow[];
   footers?: TableRow[];
   expandableRows?: ExpandableRows;
+  isResponsive?: boolean;
 }

package/src/components/Tooltip/InfoTooltip.tsx

@@ -26,11 +26,7 @@ const InfoTooltip: React.FC<InfoTooltipProps> = ({
     )}
     {...other}
   >
-    <Icon
-      name="assistance"
-      color="info"
-      className={cx(`${CLASS_ROOT}__icon`, className)}
-    />
+    <Icon name="assistance" className={cx(`${CLASS_ROOT}__icon`, className)} />
     <span className="sr-only">Info</span>
   </Tooltip>
 );

package/src/styles/tokens/color.scss

@@ -112,7 +112,7 @@ $colors-dark: (
 $orange: map.get($colors-light, icon-brand);
 $orange-dark: map.get($colors-light, surface-tertiary);
 $black: map.get($colors-light, icon-default);
-$white: map.get($colors-light, icon-inverse);
+$white: map.get($colors-light, text-inverse);
 $blue: map.get($colors-light, fill-accent1);
 $green: map.get($colors-light, fill-accent2);
 $pink: map.get($colors-light, fill-accent3);