search

meshery / meshery.io

Website for Meshery
https://meshery.io
Apache License 2.0
459 stars 540 forks source link

Fix: modal issues #1705

Closed abhishekmmgn closed 1 month ago

abhishekmmgn commented 2 months ago

Description

This PR fixes #

Notes for Reviewers List of changes/fixes:

Added spacing between filters and the catalog z index and top margin of the help modal. Read more shows only if there is more content General spacing General UI issues Added mini card (related pattern card) on viewport width ~600 - 1150 changed ...read more to Read more Added button to meshery.io/extensions card Converted title to link and changed external link icon to internal Changed the service mesh-looking image to a service-mesh-pattern image Add related patterns on small-width The gap in the catalog

Signed commits

netlify[bot] commented 2 months ago

Deploy Preview for mesheryio-preview ready!

Name Link
Latest commit bf65f6d1cb3f77ff6b169e25108f3cde84b94cb2
Latest deploy log https://app.netlify.com/sites/mesheryio-preview/deploys/66473724864e1a0008a7aef3
Deploy Preview https://deploy-preview-1705--mesheryio-preview.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.

abhishekmmgn commented 2 months ago

@ritiksaxena124 Kindly review the PR.

yash37158 commented 2 months ago

@abhishekmmgn Can you please point to the issue you were working on ?

yash37158 commented 2 months ago

image

The top spacing for the "Extension" title is not responsive. It was working correctly previously.

ritiksaxena124 commented 2 months ago

Hey @abhishekmmgn let's discuss this on today's website meet if you are available. I have added it as an agenda item

abhishekmmgn commented 2 months ago

We can merge the PR now if it can. There are a lot of commits on top of it now, so I get merge conflicts when pushing the changes. In next commit, I'll remove the buttons in the extensions catalog And will fix the h2 heading in extensions and also in the modals.

stale[bot] commented 1 month ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

leecalcote commented 1 month ago

@iArchitSharma review, please.