Skip to content

feat(ContextItem): add ContextItem#35

Merged
mariamelior merged 3 commits intomainfrom
context-item
Nov 18, 2025
Merged

feat(ContextItem): add ContextItem#35
mariamelior merged 3 commits intomainfrom
context-item

Conversation

@mariamelior
Copy link
Copy Markdown
Contributor

No description provided.

@mariamelior mariamelior marked this pull request as ready for review November 18, 2025 08:21
import {ContextItem} from '@/components/atoms/ContextItem';

// Basic usage
<ContextItem> onClick={handleRemoveContext}My Context</ContextItem>;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<ContextItem> onClick={handleRemoveContext}My Context</ContextItem>;
<ContextItem onClick={handleRemoveContext}>My Context</ContextItem>;


| Prop | Type | Required | Default | Description |
| ----------- | ----------------- | -------- | ------- | -------------------- |
| `content` | `React.ReactNode` | ✓ | - | Content of label |
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe add describe for onClick
It's callback use exactly for close/remove contextItem

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, thanks

@mariamelior mariamelior merged commit 7357425 into main Nov 18, 2025
3 checks passed
@gravity-ui gravity-ui Bot mentioned this pull request Nov 19, 2025
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.

2 participants