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

feat: add similar design by type #933

Conversation

FaheemOnHub
Copy link
Contributor

This PR fixes depends on: https://github.com/layer5io/meshery-cloud/pull/3217

Changes:

  1. Adds a carousel slider to both "Designs by Organization" and "Designs by Type" for better navigation and presentation.
Screenshot 2025-02-23 at 12 03 05 AM

Signed commits

  • Yes, I signed my commits.

Signed-off-by: TheFaheem <faheemmushtaq89@gmail.com>
Copy link
Member

@sudhanshutech sudhanshutech left a comment

Choose a reason for hiding this comment

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

@FaheemOnHub few things.

  1. I guess you don't need to show arrow icons when there are not enough cards to scroll.
  2. The arrow icons shouldn't be overflowing outside the container

@amitamrutiya
Copy link
Contributor

@FaheemOnHub was all review implemented. If yes the merge conflict and lint failing.

FaheemOnHub and others added 5 commits March 4, 2025 12:17
Signed-off-by: TheFaheem <faheemmushtaq89@gmail.com>
Signed-off-by: TheFaheem <35933338+FaheemOnHub@users.noreply.github.com>
Signed-off-by: TheFaheem <faheemmushtaq89@gmail.com>
Signed-off-by: TheFaheem <faheemmushtaq89@gmail.com>
Signed-off-by: TheFaheem <faheemmushtaq89@gmail.com>
@FaheemOnHub
Copy link
Contributor Author

@FaheemOnHub was all review implemented. If yes the merge conflict and lint failing.

its fixed

@amitamrutiya amitamrutiya merged commit eb85e4c into layer5io:master Mar 4, 2025
4 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