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

Add new image and catalog empty card #762

Merged
merged 5 commits into from
Oct 18, 2024

Conversation

amitamrutiya
Copy link
Contributor

This pull request introduces several new components and updates to the CustomCatalog module, primarily aimed at enhancing the user interface and handling empty states more effectively. The most important changes include the addition of the CatalogCardDesignLogo and EmptyStateCard components, as well as updates to the styling and exports.

New Components:

  • CatalogCardDesignLogo Component: Added a new component to display design logos with optional zoom effect. (src/custom/CustomCatalog/CatalogCardDesignLogo.tsx)
  • EmptyStateCard Component: Introduced a component to handle and display empty states with a message and icon. (src/custom/CustomCatalog/EmptyStateCard.tsx)

Styling Updates:

  • CatalogEmptyStateDiv and EmptyStateDiv: Added new styled components for empty state handling. (src/custom/CustomCatalog/style.tsx)

File and Export Updates:

  • File Renaming: Renamed custom-card.tsx to CustomCard.tsx and updated imports accordingly. (src/custom/CustomCatalog/CustomCard.tsx)
  • Export Updates: Updated exports in index.tsx to include the new components. (src/custom/CustomCatalog/index.tsx, src/custom/index.tsx) [1] [2] [3] [4]

Icon Component:

  • EmptyStyleIcon Component: Added a new icon component for empty state representation. (src/icons/EmptyStyle/EmptyStyleIcon.tsx, src/icons/EmptyStyle/index.tsx) [1] [2]

@leecalcote leecalcote requested review from sudhanshutech, nebula-aac and SAHU-01 and removed request for nebula-aac October 17, 2024 18:43
@leecalcote
Copy link
Member

@amitamrutiya these cards are used in Cloud and in Meshery UI, right? And so, you're verifying compatibility in both places, right?

@amitamrutiya
Copy link
Contributor Author

Yes @leecalcote .

@aabidsofi19 aabidsofi19 merged commit bf8f47b into layer5io:master Oct 18, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants