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

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

@@ -0,0 +1,162 @@
+/**
+ * @jest-environment jsdom
+ */
+
+import React from "react";
+import { render } from "@testing-library/react";
+
+import { SocialButton } from "../";
+
+describe("rendering SocialButton", () => {
+  describe("initial state", () => {
+    it("renders as button element by default", () => {
+      const { getByRole } = render(<SocialButton name="x" />);
+      const button = getByRole("button");
+      expect(button).toBeInTheDocument();
+      expect(button.tagName).toBe("BUTTON");
+    });
+
+    it("has default class social-button", () => {
+      const { getByRole } = render(<SocialButton name="x" />);
+      expect(getByRole("button")).toHaveClass("social-button");
+    });
+
+    it("has social platform class", () => {
+      const { getByRole } = render(<SocialButton name="x" />);
+      expect(getByRole("button")).toHaveClass("social-button--x");
+    });
+
+    it("renders Icon component", () => {
+      const { getByRole } = render(<SocialButton name="x" />);
+      const icon = getByRole("button").querySelector("svg");
+      expect(icon).toBeInTheDocument();
+    });
+  });
+
+  describe("passed props - button", () => {
+    it("renders different social platform classes", () => {
+      const platforms = [
+        "x",
+        "facebook-2",
+        "linkedin",
+        "instagram-2",
+        "pinterest",
+        "snapchat",
+        "tiktok",
+        "whatsapp",
+        "youtube",
+        "email",
+      ];
+
+      platforms.forEach((platform) => {
+        const { getByRole, unmount } = render(<SocialButton name={platform} />);
+        expect(getByRole("button")).toHaveClass(`social-button--${platform}`);
+        unmount();
+      });
+    });
+
+    it("adds custom className", () => {
+      const { getByRole } = render(
+        <SocialButton name="x" className="custom-class" />,
+      );
+      expect(getByRole("button")).toHaveClass("custom-class");
+      expect(getByRole("button")).toHaveClass("social-button");
+    });
+
+    it("passes onClick handler", () => {
+      const onClick = jest.fn();
+      const { getByRole } = render(<SocialButton name="x" onClick={onClick} />);
+      getByRole("button").click();
+      expect(onClick).toHaveBeenCalled();
+    });
+
+    it("passes data-testid attribute", () => {
+      const { getByTestId } = render(
+        <SocialButton name="x" data-testid="social-btn" />,
+      );
+      expect(getByTestId("social-btn")).toBeInTheDocument();
+    });
+
+    it("accepts aria-label attribute", () => {
+      const { getByRole } = render(
+        <SocialButton name="x" aria-label="Follow us on X" />,
+      );
+      expect(
+        getByRole("button", { name: "Follow us on X" }),
+      ).toBeInTheDocument();
+    });
+  });
+
+  describe("passed props - anchor", () => {
+    it("renders as anchor element when href is provided", () => {
+      const { getByRole } = render(
+        <SocialButton name="x" href="https://x.com" />,
+      );
+      const link = getByRole("link");
+      expect(link).toBeInTheDocument();
+      expect(link.tagName).toBe("A");
+    });
+
+    it("has href attribute", () => {
+      const { getByRole } = render(
+        <SocialButton name="x" href="https://x.com" />,
+      );
+      expect(getByRole("link")).toHaveAttribute("href", "https://x.com");
+    });
+
+    it("has social platform class as anchor", () => {
+      const { getByRole } = render(
+        <SocialButton name="facebook-2" href="https://facebook.com" />,
+      );
+      expect(getByRole("link")).toHaveClass("social-button--facebook-2");
+    });
+
+    it("passes target attribute", () => {
+      const { getByRole } = render(
+        <SocialButton name="x" href="https://x.com" target="_blank" />,
+      );
+      expect(getByRole("link")).toHaveAttribute("target", "_blank");
+    });
+
+    it("passes rel attribute", () => {
+      const { getByRole } = render(
+        <SocialButton
+          name="x"
+          href="https://x.com"
+          rel="noopener noreferrer"
+        />,
+      );
+      expect(getByRole("link")).toHaveAttribute("rel", "noopener noreferrer");
+    });
+
+    it("adds custom className to anchor", () => {
+      const { getByRole } = render(
+        <SocialButton name="x" href="https://x.com" className="custom-link" />,
+      );
+      expect(getByRole("link")).toHaveClass("custom-link");
+      expect(getByRole("link")).toHaveClass("social-button");
+    });
+
+    it("renders Icon component in anchor", () => {
+      const { getByRole } = render(
+        <SocialButton name="x" href="https://x.com" />,
+      );
+      const icon = getByRole("link").querySelector("svg");
+      expect(icon).toBeInTheDocument();
+    });
+  });
+
+  describe("ref forwarding", () => {
+    it("forwards ref to button element", () => {
+      const ref = React.createRef();
+      render(<SocialButton name="x" ref={ref} />);
+      expect(ref.current).toBeInstanceOf(HTMLButtonElement);
+    });
+
+    it("forwards ref to anchor element", () => {
+      const ref = React.createRef();
+      render(<SocialButton name="x" href="https://x.com" ref={ref} />);
+      expect(ref.current).toBeInstanceOf(HTMLAnchorElement);
+    });
+  });
+});

package/src/components/Stepbar/Stepbar.tsx

@@ -19,8 +19,6 @@ export interface StepbarProps extends React.HTMLAttributes<HTMLElement> {
   showTitlesFrom?: ShowTitlesFrom;
   /** Array of step objects */
   steps: StepbarStep[];
-  /** Visual variant of the stepbar */
-  colorScheme?: "dark" | "light";
 }
 
 const CLASS_ROOT = "stepbar";
@@ -29,7 +27,6 @@ const Stepbar: React.FC<StepbarProps> = ({
   showTitlesFrom,
   steps,
   label = "krok",
-  colorScheme,
   className,
   ...other
 }) => {
@@ -68,8 +65,6 @@ const Stepbar: React.FC<StepbarProps> = ({
       className={cx(
         CLASS_ROOT,
         {
-          "is-dark": colorScheme === "dark",
-          "is-light": colorScheme === "light",
           [`${CLASS_ROOT}--wide-${showTitlesFrom}`]: showTitlesFrom,
         },
         className,

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

@@ -12,7 +12,7 @@
   flex-wrap: nowrap;
   list-style-type: none;
   padding-left: 0;
-  margin: 0 0 space.get() 0;
+  margin: 0 0 space.get("large") 0;
   max-width: none;
   font-weight: bold;
 }

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. */
@@ -30,8 +32,6 @@ interface TableProps extends React.TableHTMLAttributes<HTMLTableElement> {
   /** rows to print out in the table. */
   rows: TableRow[];
   className?: string;
-  /** Sets color scheme */
-  colorScheme?: "dark" | "light";
 }
 
 const CLASS_ROOT = "table";
@@ -43,9 +43,9 @@ const Table: React.FC<TableProps> = ({
   rows,
   footers,
   isCompact,
+  isResponsive,
   isStriped,
   isScrollable,
-  colorScheme,
   expandableRowsCaptions = {
     header: "Rozšíriteľný",
     emptyRow: "Nerozšíriteľný riadok",
@@ -60,8 +60,7 @@ const Table: React.FC<TableProps> = ({
     {
       [`${CLASS_ROOT}--striped`]: isStriped,
       [`${CLASS_ROOT}--compact`]: isCompact,
-      "is-dark": colorScheme === "dark",
-      "is-light": colorScheme === "light",
+      [`${CLASS_ROOT}--responsive`]: isResponsive,
     },
     className,
   );
@@ -94,6 +93,7 @@ const Table: React.FC<TableProps> = ({
     footers,
     rows: customRows,
     expandableRows,
+    isResponsive,
   };
 
   const table = (
@@ -108,7 +108,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

@@ -5,13 +5,12 @@
 
 @mixin base {
   width: 100%;
-  margin-bottom: space.get();
+  margin-bottom: space.get("large");
   color: var(--color-text-default);
-  background-color: var(--color-background-primary);
 }
 
 @mixin cells {
-  padding: space.get("small") space.get("small") space.get("small") 0;
+  padding: space.get("medium");
   vertical-align: middle;
   font-size: inherit;
   line-height: inherit;
@@ -20,7 +19,7 @@
 @mixin header-cells {
   vertical-align: bottom;
   text-align: left;
-  border-bottom: config.$fat-border-size solid var(--color-border-strong);
+  border-bottom: config.$fat-border-size solid var(--color-border-contrast);
   font-weight: 700;
 }
 
@@ -38,7 +37,7 @@
 }
 
 @mixin rows-first-cell {
-  padding-left: space.get("small");
+  padding-left: 0;
 }
 
 @mixin body-sibling {
@@ -113,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/tests/Table.conformance.test.js

@@ -64,7 +64,11 @@ const example = (
 it("is valid html", () => {
   const { container } = render(example);
   expect(container).toHTMLValidate({
-    rules: { "attribute-boolean-style": "off", "valid-id": "off" },
+    rules: {
+      "attribute-boolean-style": "off",
+      "valid-id": "off",
+      "attribute-empty-style": "off",
+    },
   });
 });
 

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

@@ -51,37 +51,6 @@ describe("rendering Table", () => {
       expect(getByTestId("test-id").tagName).toBe("TABLE");
       expect(getByTestId("test-id")).toHaveClass("table");
     });
-    it("applies dark colorScheme class", () => {
-      const { getByTestId } = render(
-        <Table
-          data-testid="test-id"
-          headers={[]}
-          rows={[]}
-          colorScheme="dark"
-        />,
-      );
-      expect(getByTestId("test-id")).toHaveClass("is-dark");
-      expect(getByTestId("test-id")).not.toHaveClass("is-light");
-    });
-    it("applies light colorScheme class", () => {
-      const { getByTestId } = render(
-        <Table
-          data-testid="test-id"
-          headers={[]}
-          rows={[]}
-          colorScheme="light"
-        />,
-      );
-      expect(getByTestId("test-id")).toHaveClass("is-light");
-      expect(getByTestId("test-id")).not.toHaveClass("is-dark");
-    });
-    it("does not apply colorScheme class by default", () => {
-      const { getByTestId } = render(
-        <Table data-testid="test-id" headers={[]} rows={[]} />,
-      );
-      expect(getByTestId("test-id")).not.toHaveClass("is-dark");
-      expect(getByTestId("test-id")).not.toHaveClass("is-light");
-    });
   });
   describe("testing headers", () => {
     it("renders headers", () => {

package/src/components/Table/types.ts

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

package/src/components/Tabs/Tabs.tsx

@@ -17,8 +17,6 @@ interface TabsProps extends React.HTMLAttributes<HTMLUListElement> {
   activeTabIndex?: number;
   /** Additional classes for tabs header. */
   classesTabNav?: string;
-  /** Color theme for the tabs */
-  colorScheme?: "dark" | "light";
   /** All tabs have equal width inside a full-width container */
   hasEqualTabWidth?: boolean;
   /** Fullwidth container */
@@ -40,7 +38,6 @@ const Tabs: React.FC<TabsProps> = ({
   hasEqualTabWidth,
   variant = "standard",
   children,
-  colorScheme,
   ...other
 }) => {
   const CLASS_TABNAV = "tab-list";
@@ -50,8 +47,6 @@ const Tabs: React.FC<TabsProps> = ({
     { [`${CLASS_TABNAV}--equal`]: hasEqualTabWidth },
     { [`${CLASS_TABNAV}--fullwidth`]: isFullWidth },
     { [`${CLASS_TABNAV}--light`]: variant === "light" },
-    { "is-dark": colorScheme === "dark" },
-    { "is-light": colorScheme === "light" },
     classesTabNav,
     className,
   );

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

@@ -8,7 +8,7 @@
   display: flex;
   overflow-x: auto;
   overflow-y: visible;
-  margin-bottom: space.get();
+  margin-bottom: space.get("large");
   align-items: stretch;
 
   @include generate.css-map($spacing);

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

@@ -99,14 +99,6 @@ describe("rendering Tabs", () => {
         }
       }
     });
-    it("has class is-dark when colorScheme is set to dark", () => {
-      const { getByTestId } = render(<TabsExample colorScheme="dark" />);
-      expect(getByTestId("test-id")).toHaveClass("is-dark");
-    });
-    it("has class is-light when colorScheme is set to light", () => {
-      const { getByTestId } = render(<TabsExample colorScheme="light" />);
-      expect(getByTestId("test-id")).toHaveClass("is-light");
-    });
   });
   describe("checking fireEvents", () => {
     it("click in disabled tab button doesnt cause change", () => {

package/src/components/Tag/Tag.tsx

@@ -21,8 +21,6 @@ export type TagSize = "small" | "large";
 interface CommonProps {
   color?: TagColor;
   size?: TagSize;
-  /** @deprecated */
-  isInverse?: boolean;
   isDisabled?: boolean;
   className?: string;
   children?: React.ReactNode;
@@ -55,7 +53,6 @@ const Tag = React.forwardRef<HTMLElement, TagProps>((props, ref) => {
   const {
     color,
     size,
-    isInverse,
     isDisabled,
     className,
     children,
@@ -64,7 +61,6 @@ const Tag = React.forwardRef<HTMLElement, TagProps>((props, ref) => {
   } = props;
 
   const classes = cx(CLASS_ROOT, className, {
-    "tag--inverse": isInverse,
     [`tag--${color}`]: color,
     [`tag--${size}`]: size,
   });

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

@@ -1,5 +1,4 @@
 @use "../../../styles/tokens/space";
-@use "../../../styles/tokens/color";
 @use "../../../styles/tools/convert";
 @use "sass:map";
 
@@ -24,7 +23,7 @@ $sizes: (
     font-weight: 400,
   ),
   large: (
-    padding: convert.to-rem(12.5px) convert.to-rem(15px),
+    padding: convert.to-rem(11px) convert.to-rem(15px),
     font-size: convert.to-rem(16px),
     line-height: convert.to-rem(18px),
     font-weight: 700,
@@ -46,43 +45,40 @@ $colors: (
     color: var(--color-text-default),
   ),
   "orange": (
-    background-color: map.get(color.$colors-light, icon-accent),
-    color: color.$black,
+    background-color: var(--tag-orange),
+    color: var(--tag-text-dark),
   ),
   "black": (
-    background-color: map.get(color.$colors-dark, border-subtle),
-    color: color.$white,
+    background-color: var(--tag-black),
+    color: var(--tag-text-light),
   ),
   "yellow": (
-    // TODO add to config
-    background-color: #fff6b6,
-    color: color.$black,
+    background-color: var(--tag-yellow),
+    color: var(--tag-text-dark),
   ),
   "green": (
-    // TODO add to config
-    background-color: #b8ebd6,
-    color: color.$black,
+    background-color: var(--tag-green),
+    color: var(--tag-text-dark),
   ),
   "blue": (
-    // TODO add to config
-    background-color: #b5e8f7,
-    color: color.$black,
+    background-color: var(--tag-blue),
+    color: var(--tag-text-dark),
   ),
   "success": (
-    background-color: map.get(color.$colors-light, icon-positive),
-    color: color.$white,
+    background-color: var(--tag-success),
+    color: var(--tag-text-light),
   ),
   "danger": (
-    background-color: map.get(color.$colors-light, icon-negative),
-    color: color.$white,
+    background-color: var(--tag-danger),
+    color: var(--tag-text-light),
   ),
   "info": (
-    background-color: map.get(color.$colors-light, icon-information),
-    color: color.$white,
+    background-color: var(--tag-info),
+    color: var(--tag-text-light),
   ),
   "warning": (
-    background-color: map.get(color.$colors-light, icon-warning),
-    color: color.$black,
+    background-color: var(--tag-warning),
+    color: var(--tag-text-dark),
   ),
   "transparent": (
     background-color: transparent,

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

@@ -20,6 +20,39 @@
       }
     }
 
+    // Special styling for default tags inside .background-secondary parent
+    .background-secondary & {
+      // Target tags that don't have any color variant class
+      // Use negative lookahead-like approach: exclude if class contains any color variant
+      &:not([class*="--orange"]):not([class*="--black"]):not(
+          [class*="--yellow"]
+        ):not([class*="--green"]):not([class*="--blue"]):not(
+          [class*="--success"]
+        ):not([class*="--danger"]):not([class*="--info"]):not(
+          [class*="--warning"]
+        ):not([class*="--transparent"]) {
+        background-color: var(--color-surface-moderate);
+
+        &:where(button),
+        &:where(a) {
+          background-color: var(--color-surface-moderate);
+
+          &:not([disabled]):not([aria-disabled="true"]) {
+            &:focus-visible,
+            &:hover {
+              background-color: color.$gray-600;
+            }
+
+            &:active,
+            &.is-active {
+              color: var(--color-text-inverse);
+              background-color: var(--color-background-contrast);
+            }
+          }
+        }
+      }
+    }
+
     // Transparent class is a special variant of color, it's not possible to make transparent variant for each color at the moment.
     &--transparent {
       @include mixins.color-transparent();