Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tag cell in data-grid extended #2343

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

tshimber
Copy link
Contributor

Fixes #2297.
Props size and type for tag cell in data-grid changed from hardcoded to dynamic. Description in storybook extended

Copy link

netlify bot commented Aug 27, 2024

Deploy Preview for marvelous-moxie-a6e2fe ready!

Name Link
🔨 Latest commit 58edcf8
🔍 Latest deploy log https://app.netlify.com/sites/marvelous-moxie-a6e2fe/deploys/670cce37a55fee0008fca882
😎 Deploy Preview https://deploy-preview-2343--marvelous-moxie-a6e2fe.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tshimber tshimber changed the title feat: tag cell in data-grid extended tag cell in data-grid extended Aug 27, 2024
@amir-ba amir-ba added the feature New component features and enhancements label Aug 28, 2024
@tshimber tshimber force-pushed the feature-tag-cell-extended branch from f10ad35 to 9f9e0cd Compare August 28, 2024 10:51
@@ -669,7 +669,7 @@ Expected format: string

## Tags Cell

Expected format: comma delimited `string`, eg `'one, two, three'` or array of objects with content and color keys, e.g. `{content: 'Apple', color: 'red'}`
Expected format: comma delimited `string`, eg `'one, two, three'` or array of objects with required content key and optional color, type, size keys, e.g. `{content: 'Apple', color: 'red'}`. Acceptable values for color, type, size are same as for Tag component.
Copy link
Collaborator

Choose a reason for hiding this comment

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

please add the link to Tag component for the user to jump to

Copy link
Contributor Author

Choose a reason for hiding this comment

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

link added

@amir-ba
Copy link
Collaborator

amir-ba commented Oct 16, 2024

LGTM!

@amir-ba amir-ba merged commit ee21b87 into telekom:main Oct 16, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New component features and enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Additional Tags Cell Controls
2 participants