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

MOON-338: Add component CardSelector #460

Merged
merged 13 commits into from
Dec 6, 2024
Merged

MOON-338: Add component CardSelector #460

merged 13 commits into from
Dec 6, 2024

Conversation

Eevolee
Copy link
Contributor

@Eevolee Eevolee commented Nov 27, 2024

JIRA

https://jira.jahia.org/browse/MOON-338

Description

Add components CardSelector & EmptyCardSelector
Component CardSelector has dragging icon enabled with isDraggable prop but is not currently handling dragging

@Eevolee Eevolee added the minor Increment the minor version when merged label Nov 27, 2024
gflores-jahia
gflores-jahia previously approved these changes Nov 27, 2024
Copy link
Contributor

@gflores-jahia gflores-jahia left a comment

Choose a reason for hiding this comment

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

For reference this is the jcontent-specific implementation from the old design system https://github.com/Jahia/jcontent/blob/master/src/javascript/ContentEditor/DesignSystem/ReferenceCard/ReferenceCard.jsx

It looks like this has support for most of the information that's currently included in it, with a small comment regarding EmptyCardSelector.

Copy link
Collaborator

@ffffffelix ffffffelix left a comment

Choose a reason for hiding this comment

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

I think we forgot to add the import of the components in src/components/index.ts

@ffffffelix ffffffelix merged commit b5a8976 into develop Dec 6, 2024
3 checks passed
@ffffffelix ffffffelix deleted the MOON-338 branch December 6, 2024 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants