@gravity-ui/page-constructor 2.8.2 → 2.8.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/CHANGELOG.md CHANGED
@@ -1,5 +1,19 @@
1
1
  # Changelog
2
2
 
3
+ ## [2.8.4](https://github.com/gravity-ui/page-constructor/compare/v2.8.3...v2.8.4) (2023-04-17)
4
+
5
+
6
+ ### Bug Fixes
7
+
8
+ * **Content:** remove links margin ([#306](https://github.com/gravity-ui/page-constructor/issues/306)) ([62972b4](https://github.com/gravity-ui/page-constructor/commit/62972b445e28016fe250c0361c91b84324456fe6))
9
+
10
+ ## [2.8.3](https://github.com/gravity-ui/page-constructor/compare/v2.8.2...v2.8.3) (2023-04-14)
11
+
12
+
13
+ ### Bug Fixes
14
+
15
+ * **BasicCard:** schema validation ([#304](https://github.com/gravity-ui/page-constructor/issues/304)) ([a8627ca](https://github.com/gravity-ui/page-constructor/commit/a8627ca131547281674e5f24df85b830875539f7))
16
+
3
17
  ## [2.8.2](https://github.com/gravity-ui/page-constructor/compare/v2.8.1...v2.8.2) (2023-04-12)
4
18
 
5
19
 
@@ -11,6 +11,7 @@ export interface ButtonTabsProps {
11
11
  activeTab?: string | null;
12
12
  onSelectTab?: (tabId: string | null) => void;
13
13
  tabSize?: ButtonSize;
14
+ qa?: string;
14
15
  }
15
16
  declare const ButtonTabs: React.FC<ButtonTabsProps>;
16
17
  export default ButtonTabs;
@@ -5,7 +5,7 @@ const react_1 = tslib_1.__importStar(require("react"));
5
5
  const utils_1 = require("../../utils");
6
6
  const index_1 = require("../index");
7
7
  const b = (0, utils_1.block)('button-tabs');
8
- const ButtonTabs = ({ className, items, activeTab, onSelectTab, tabSize = 'l', }) => {
8
+ const ButtonTabs = ({ className, items, activeTab, onSelectTab, tabSize = 'l', qa, }) => {
9
9
  const activeTabId = (0, react_1.useMemo)(() => {
10
10
  if (activeTab) {
11
11
  return activeTab;
@@ -17,6 +17,6 @@ const ButtonTabs = ({ className, items, activeTab, onSelectTab, tabSize = 'l', }
17
17
  onSelectTab(tabId);
18
18
  }
19
19
  }, [onSelectTab]);
20
- return (react_1.default.createElement("div", { className: b(null, className) }, items.map(({ id, title }) => (react_1.default.createElement(index_1.Button, { text: title, className: b('item', { active: id === activeTabId }), key: title, size: tabSize, onClick: () => handleClick(id) })))));
20
+ return (react_1.default.createElement("div", { className: b(null, className), "data-qa": qa }, items.map(({ id, title }) => (react_1.default.createElement(index_1.Button, { text: title, className: b('item', { active: id === activeTabId }), key: title, size: tabSize, onClick: () => handleClick(id) })))));
21
21
  };
22
22
  exports.default = ButtonTabs;
@@ -0,0 +1,58 @@
1
+ "use strict";
2
+ Object.defineProperty(exports, "__esModule", { value: true });
3
+ const tslib_1 = require("tslib");
4
+ const react_1 = tslib_1.__importDefault(require("react"));
5
+ const react_2 = require("@testing-library/react");
6
+ const user_event_1 = tslib_1.__importDefault(require("@testing-library/user-event"));
7
+ const ButtonTabs_1 = tslib_1.__importDefault(require("../ButtonTabs"));
8
+ const qaId = 'button-tabs-component';
9
+ const items = [
10
+ {
11
+ id: '0',
12
+ title: 'tab-1',
13
+ },
14
+ {
15
+ id: '1',
16
+ title: 'tab-2',
17
+ },
18
+ {
19
+ id: '2',
20
+ title: 'tab-3',
21
+ },
22
+ ];
23
+ describe('ButtonTabs', () => {
24
+ test('render ButtonTabs by default', async () => {
25
+ (0, react_2.render)(react_1.default.createElement(ButtonTabs_1.default, { items: items, qa: qaId }));
26
+ const buttonTabs = react_2.screen.getByTestId(qaId);
27
+ expect(buttonTabs).toBeInTheDocument();
28
+ expect(buttonTabs).toBeVisible();
29
+ expect(buttonTabs).not.toBeDisabled();
30
+ });
31
+ test('has active tab', async () => {
32
+ const activeTabId = 1;
33
+ (0, react_2.render)(react_1.default.createElement(ButtonTabs_1.default, { items: items, qa: qaId, activeTab: String(activeTabId) }));
34
+ const buttons = react_2.screen.getAllByRole('button');
35
+ buttons.forEach((button, index) => {
36
+ if (index === activeTabId) {
37
+ expect(button).toHaveClass('pc-button-tabs__item_active');
38
+ }
39
+ expect(button).toHaveClass('pc-button-block_theme_normal');
40
+ });
41
+ });
42
+ test('add className', () => {
43
+ const className = 'my-class';
44
+ (0, react_2.render)(react_1.default.createElement(ButtonTabs_1.default, { items: items, qa: qaId, className: className }));
45
+ const buttonTabs = react_2.screen.getByTestId(qaId);
46
+ expect(buttonTabs).toHaveClass(className);
47
+ });
48
+ test('call onSelectTab', async () => {
49
+ const user = user_event_1.default.setup();
50
+ const handleOnClick = jest.fn();
51
+ (0, react_2.render)(react_1.default.createElement(ButtonTabs_1.default, { items: items, qa: qaId, onSelectTab: handleOnClick }));
52
+ const buttons = react_2.screen.getAllByRole('button');
53
+ buttons.forEach(async (button, i) => {
54
+ await user.click(button);
55
+ expect(handleOnClick).toHaveBeenCalledTimes(i + 1);
56
+ });
57
+ });
58
+ });
@@ -509,7 +509,7 @@ exports.MapProps = {
509
509
  exports.CardBase = {
510
510
  border: {
511
511
  type: 'string',
512
- enum: ['border', 'shadow', 'none'],
512
+ enum: ['line', 'shadow', 'none'],
513
513
  },
514
514
  };
515
515
  exports.BlockHeaderProps = {
@@ -1,7 +1,7 @@
1
1
  export declare const BasicCard: {
2
2
  'basic-card': {
3
3
  additionalProperties: boolean;
4
- required: string[];
4
+ required: never[];
5
5
  properties: {
6
6
  url: {
7
7
  type: string;
@@ -10,7 +10,7 @@ const BasicCardContentProps = lodash_1.default.omit(schema_2.ContentBase, ['size
10
10
  exports.BasicCard = {
11
11
  'basic-card': {
12
12
  additionalProperties: false,
13
- required: ['url'],
13
+ required: [],
14
14
  properties: Object.assign(Object.assign(Object.assign(Object.assign({}, common_1.BaseProps), common_1.CardBase), BasicCardContentProps), { url: {
15
15
  type: 'string',
16
16
  }, icon: schema_1.ImageProps, target: {
@@ -42,10 +42,6 @@ unpredictable css rules order in build */
42
42
  }
43
43
  .pc-content__link {
44
44
  display: block;
45
- margin-right: 32px;
46
- }
47
- .pc-content__link:last-child {
48
- margin-right: 0;
49
45
  }
50
46
  .pc-content_centered.pc-content_centered {
51
47
  margin: 0 auto;
@@ -12,6 +12,7 @@ export interface ButtonTabsProps {
12
12
  activeTab?: string | null;
13
13
  onSelectTab?: (tabId: string | null) => void;
14
14
  tabSize?: ButtonSize;
15
+ qa?: string;
15
16
  }
16
17
  declare const ButtonTabs: React.FC<ButtonTabsProps>;
17
18
  export default ButtonTabs;
@@ -3,7 +3,7 @@ import { block } from '../../utils';
3
3
  import { Button } from '../index';
4
4
  import './ButtonTabs.css';
5
5
  const b = block('button-tabs');
6
- const ButtonTabs = ({ className, items, activeTab, onSelectTab, tabSize = 'l', }) => {
6
+ const ButtonTabs = ({ className, items, activeTab, onSelectTab, tabSize = 'l', qa, }) => {
7
7
  const activeTabId = useMemo(() => {
8
8
  if (activeTab) {
9
9
  return activeTab;
@@ -15,6 +15,6 @@ const ButtonTabs = ({ className, items, activeTab, onSelectTab, tabSize = 'l', }
15
15
  onSelectTab(tabId);
16
16
  }
17
17
  }, [onSelectTab]);
18
- return (React.createElement("div", { className: b(null, className) }, items.map(({ id, title }) => (React.createElement(Button, { text: title, className: b('item', { active: id === activeTabId }), key: title, size: tabSize, onClick: () => handleClick(id) })))));
18
+ return (React.createElement("div", { className: b(null, className), "data-qa": qa }, items.map(({ id, title }) => (React.createElement(Button, { text: title, className: b('item', { active: id === activeTabId }), key: title, size: tabSize, onClick: () => handleClick(id) })))));
19
19
  };
20
20
  export default ButtonTabs;
@@ -0,0 +1,55 @@
1
+ import React from 'react';
2
+ import { render, screen } from '@testing-library/react';
3
+ import userEvent from '@testing-library/user-event';
4
+ import ButtonTabs from '../ButtonTabs';
5
+ const qaId = 'button-tabs-component';
6
+ const items = [
7
+ {
8
+ id: '0',
9
+ title: 'tab-1',
10
+ },
11
+ {
12
+ id: '1',
13
+ title: 'tab-2',
14
+ },
15
+ {
16
+ id: '2',
17
+ title: 'tab-3',
18
+ },
19
+ ];
20
+ describe('ButtonTabs', () => {
21
+ test('render ButtonTabs by default', async () => {
22
+ render(React.createElement(ButtonTabs, { items: items, qa: qaId }));
23
+ const buttonTabs = screen.getByTestId(qaId);
24
+ expect(buttonTabs).toBeInTheDocument();
25
+ expect(buttonTabs).toBeVisible();
26
+ expect(buttonTabs).not.toBeDisabled();
27
+ });
28
+ test('has active tab', async () => {
29
+ const activeTabId = 1;
30
+ render(React.createElement(ButtonTabs, { items: items, qa: qaId, activeTab: String(activeTabId) }));
31
+ const buttons = screen.getAllByRole('button');
32
+ buttons.forEach((button, index) => {
33
+ if (index === activeTabId) {
34
+ expect(button).toHaveClass('pc-button-tabs__item_active');
35
+ }
36
+ expect(button).toHaveClass('pc-button-block_theme_normal');
37
+ });
38
+ });
39
+ test('add className', () => {
40
+ const className = 'my-class';
41
+ render(React.createElement(ButtonTabs, { items: items, qa: qaId, className: className }));
42
+ const buttonTabs = screen.getByTestId(qaId);
43
+ expect(buttonTabs).toHaveClass(className);
44
+ });
45
+ test('call onSelectTab', async () => {
46
+ const user = userEvent.setup();
47
+ const handleOnClick = jest.fn();
48
+ render(React.createElement(ButtonTabs, { items: items, qa: qaId, onSelectTab: handleOnClick }));
49
+ const buttons = screen.getAllByRole('button');
50
+ buttons.forEach(async (button, i) => {
51
+ await user.click(button);
52
+ expect(handleOnClick).toHaveBeenCalledTimes(i + 1);
53
+ });
54
+ });
55
+ });
@@ -505,7 +505,7 @@ export const MapProps = {
505
505
  export const CardBase = {
506
506
  border: {
507
507
  type: 'string',
508
- enum: ['border', 'shadow', 'none'],
508
+ enum: ['line', 'shadow', 'none'],
509
509
  },
510
510
  };
511
511
  export const BlockHeaderProps = {
@@ -1,7 +1,7 @@
1
1
  export declare const BasicCard: {
2
2
  'basic-card': {
3
3
  additionalProperties: boolean;
4
- required: string[];
4
+ required: never[];
5
5
  properties: {
6
6
  url: {
7
7
  type: string;
@@ -6,7 +6,7 @@ const BasicCardContentProps = _.omit(ContentBase, ['size', 'theme']);
6
6
  export const BasicCard = {
7
7
  'basic-card': {
8
8
  additionalProperties: false,
9
- required: ['url'],
9
+ required: [],
10
10
  properties: Object.assign(Object.assign(Object.assign(Object.assign({}, BaseProps), CardBase), BasicCardContentProps), { url: {
11
11
  type: 'string',
12
12
  }, icon: ImageProps, target: {
@@ -42,10 +42,6 @@ unpredictable css rules order in build */
42
42
  }
43
43
  .pc-content__link {
44
44
  display: block;
45
- margin-right: 32px;
46
- }
47
- .pc-content__link:last-child {
48
- margin-right: 0;
49
45
  }
50
46
  .pc-content_centered.pc-content_centered {
51
47
  margin: 0 auto;
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@gravity-ui/page-constructor",
3
- "version": "2.8.2",
3
+ "version": "2.8.4",
4
4
  "description": "Gravity UI Page Constructor",
5
5
  "license": "MIT",
6
6
  "repository": {