Skip to content

PLASMA-7286: update Cell types#2810

Open
oki1oki wants to merge 1 commit into
devfrom
PLASMA-7286
Open

PLASMA-7286: update Cell types#2810
oki1oki wants to merge 1 commit into
devfrom
PLASMA-7286

Conversation

@oki1oki
Copy link
Copy Markdown
Collaborator

@oki1oki oki1oki commented May 27, 2026

Core

Cell

  • CellProps теперь принимает стандартные атрибуты HTMLDivElement

What/why changed

  • CellProps теперь принимает стандартные атрибуты HTMLDivElement

Summary by CodeRabbit

  • New Features
    • Cell component now accepts and properly forwards standard HTML div attributes, enabling greater flexibility when using the component.

Review Change Stack

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @salutejs/plasma-asdk@0.378.0-canary.2810.26522573163.0
npm install @salutejs/plasma-b2c@1.620.0-canary.2810.26522573163.0
npm install @salutejs/plasma-core@1.228.0-canary.2810.26522573163.0
npm install @salutejs/plasma-giga@0.347.0-canary.2810.26522573163.0
npm install @salutejs/plasma-homeds@0.347.0-canary.2810.26522573163.0
npm install @salutejs/plasma-hope@1.374.0-canary.2810.26522573163.0
npm install @salutejs/plasma-icons@1.239.0-canary.2810.26522573163.0
npm install @salutejs/plasma-new-hope@0.364.0-canary.2810.26522573163.0
npm install @salutejs/plasma-tokens-b2b@1.56.0-canary.2810.26522573163.0
npm install @salutejs/plasma-tokens-b2c@0.67.0-canary.2810.26522573163.0
npm install @salutejs/plasma-tokens-web@1.71.0-canary.2810.26522573163.0
npm install @salutejs/plasma-tokens@1.140.0-canary.2810.26522573163.0
npm install @salutejs/plasma-typo@0.44.0-canary.2810.26522573163.0
npm install @salutejs/plasma-ui@1.350.0-canary.2810.26522573163.0
npm install @salutejs/plasma-web@1.622.0-canary.2810.26522573163.0
npm install @salutejs/sdds-bizcom@0.352.0-canary.2810.26522573163.0
npm install @salutejs/sdds-cs@0.356.0-canary.2810.26522573163.0
npm install @salutejs/sdds-dfa@0.350.0-canary.2810.26522573163.0
npm install @salutejs/sdds-finai@0.343.0-canary.2810.26522573163.0
npm install @salutejs/sdds-insol@0.347.0-canary.2810.26522573163.0
npm install @salutejs/sdds-netology@0.351.0-canary.2810.26522573163.0
npm install @salutejs/sdds-os@0.22.0-canary.2810.26522573163.0
npm install @salutejs/sdds-platform-ai@0.351.0-canary.2810.26522573163.0
npm install @salutejs/sdds-sbcom@0.352.0-canary.2810.26522573163.0
npm install @salutejs/sdds-scan@0.350.0-canary.2810.26522573163.0
npm install @salutejs/sdds-serv@0.351.0-canary.2810.26522573163.0
npm install @salutejs/core-themes@0.31.0-canary.2810.26522573163.0
npm install @salutejs/plasma-themes@0.52.0-canary.2810.26522573163.0
npm install @salutejs/sdds-themes@0.67.0-canary.2810.26522573163.0
npm install @salutejs/sdds-api-tests@0.9.0-canary.2810.26522573163.0
npm install @salutejs/plasma-cy-utils@0.158.0-canary.2810.26522573163.0
npm install @salutejs/plasma-sb-utils@0.228.0-canary.2810.26522573163.0
npm install @salutejs/plasma-tokens-utils@0.52.0-canary.2810.26522573163.0
# or 
yarn add @salutejs/plasma-asdk@0.378.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-b2c@1.620.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-core@1.228.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-giga@0.347.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-homeds@0.347.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-hope@1.374.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-icons@1.239.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-new-hope@0.364.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-tokens-b2b@1.56.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-tokens-b2c@0.67.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-tokens-web@1.71.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-tokens@1.140.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-typo@0.44.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-ui@1.350.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-web@1.622.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-bizcom@0.352.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-cs@0.356.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-dfa@0.350.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-finai@0.343.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-insol@0.347.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-netology@0.351.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-os@0.22.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-platform-ai@0.351.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-sbcom@0.352.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-scan@0.350.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-serv@0.351.0-canary.2810.26522573163.0
yarn add @salutejs/core-themes@0.31.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-themes@0.52.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-themes@0.67.0-canary.2810.26522573163.0
yarn add @salutejs/sdds-api-tests@0.9.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-cy-utils@0.158.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-sb-utils@0.228.0-canary.2810.26522573163.0
yarn add @salutejs/plasma-tokens-utils@0.52.0-canary.2810.26522573163.0

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 27, 2026

📝 Walkthrough

Walkthrough

The Cell component is updated to accept and forward standard HTML div attributes. The CellProps type now extends HTMLAttributes<HTMLDivElement>, and the cellRoot implementation captures remaining props via rest destructuring and spreads them onto the underlying Root element.

Changes

Cell HTML Attribute Forwarding

Layer / File(s) Summary
Props type contract and imports
packages/plasma-new-hope/src/components/Cell/Cell.types.ts
CellProps now intersects HTMLAttributes<HTMLDivElement> with custom and textbox prop types. HTMLAttributes import is added from React. Comment text adjusted.
Props forwarding in cellRoot implementation
packages/plasma-new-hope/src/components/Cell/Cell.tsx
cellRoot captures remaining props via ...rest destructuring and spreads them onto the Root element, enabling standard HTML attributes to flow through.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~7 minutes

Poem

🐰 A Cell hops forward with grace,
Now carrying HTML's embrace,
Div attributes flow through without strain,
As props find their rightful lane,
The component's reach expands today! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and accurately summarizes the main change: updating Cell types to accept standard HTMLDivElement attributes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch PLASMA-7286

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown
Contributor

Theme Builder app deployed!

https://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-2810/

@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview deployed!

website: https://plasma.sberdevices.ru/pr/pr-2810/
asdk storybook: https://plasma.sberdevices.ru/pr/pr-2810/asdk-storybook/
b2c storybook: https://plasma.sberdevices.ru/pr/pr-2810/b2c-storybook/
giga storybook: https://plasma.sberdevices.ru/pr/pr-2810/giga-storybook/
homeds storybook: https://plasma.sberdevices.ru/pr/pr-2810/homeds-storybook/
new-hope storybook: https://plasma.sberdevices.ru/pr/pr-2810/new-hope-storybook/
web storybook: https://plasma.sberdevices.ru/pr/pr-2810/web-storybook/
sdds-bizcom storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-bizcom-storybook/
sdds-cs storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-cs-storybook/
sdds-dfa storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-dfa-storybook/
sdds-finai storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-finai-storybook/
sdds-insol storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-insol-storybook/
sdds-netology storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-netology-storybook/
sdds-os storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-os-storybook/
sdds-platform-ai storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-platform-ai-storybook/
sdds-sbcom storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-sbcom-storybook/
sdds-scan storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-scan-storybook/
sdds-serv storybook: https://plasma.sberdevices.ru/pr/pr-2810/sdds-serv-storybook/
ui storybook: https://plasma.sberdevices.ru/pr/pr-2810/ui-storybook/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant