Skip to content

fixed css styling issues#565

Closed
Mratyunjay-ch-45 wants to merge 1 commit intokeploy:mainfrom
Mratyunjay-ch-45:Mratyunjay
Closed

fixed css styling issues#565
Mratyunjay-ch-45 wants to merge 1 commit intokeploy:mainfrom
Mratyunjay-ch-45:Mratyunjay

Conversation

@Mratyunjay-ch-45
Copy link

What has changed?

This PR resolves UI inconsistencies and enhances the visual design of the documentation homepage.

  • 🛠 Fixed styling issues in the Installation Guide section:

    • Added a hover scale-up and shadow effect to the platform logo cards (Windows, Mac, Linux) for better interactivity.
  • 🎯 Resolved height and styling inconsistencies in the Products section:

    • Ensured all product cards have uniform height and styling.
    • Added a hover effect with scale and shadow to improve user experience.
  • 🔗 Enhanced appearance of external links (YouTube, GitHub, Slack, etc.):

    • Added hover effects for a more engaging look.

Type of change

  • Bug fix (non-breaking change which fixes UI/UX issues)
  • New feature
  • Breaking change
  • Documentation update

How Has This Been Tested?

  • Ran npm run build and npm run serve to ensure the homepage renders correctly after changes.
  • Visually verified styling improvements and hover interactions in the browser.

(Include screenshots or a short GIF here if possible.)

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you and congratulations 🎉 for opening your very first pull request in keploy

@pratik-mahalle
Copy link
Contributor

Hey @Mratyunjay-ch-45, Thank you for raising the pull request. After carefullu reviewing the chnages, unfortunately we can`t approve this. This being a part of an assignment in api fellowship would be counted since you were suppsoed to create a pr but wont be merged since this is not that relevant of a change. I’m sure you understand, and I appreciate your cooperation on this. We will be closing this PR for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants