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

fix: landing page hero section button alignment fixed #228

Merged
merged 2 commits into from
Oct 29, 2023
Merged

fix: landing page hero section button alignment fixed #228

merged 2 commits into from
Oct 29, 2023

Conversation

tanpreetjolly
Copy link
Contributor

@tanpreetjolly tanpreetjolly commented Oct 22, 2023

What does this PR do?

It fixes the alignment of landing page button which was overlapping with the text.

Fixes #254
I have used a flex container for the parent container to fix the alignment and layout of the hero section so that the hero button is no longer overlapping
Before :

Screenshot from 2023-10-22 13-25-07

After :

Screenshot from 2023-10-22 13-32-19

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How should this be tested?

  • Go to the Landing Page
  • Check the Hero Section
  • Check the alignment "Get Started for Free" button in the Hero Section

@github-actions
Copy link

github-actions bot commented Oct 22, 2023

Thank you for following the naming conventions! 🙏

@tanpreetjolly
Copy link
Contributor Author

#254

@@ -161,7 +161,7 @@ export const HeroSection = () => {
<div className="h-60 bg-gradient-to-br from-violet-500 to-violet-600 blur-[200px] dark:from-sky-900 dark:to-sky-950"></div>
<div className="h-60 bg-gradient-to-r from-violet-500 to-violet-600 blur-[200px] dark:from-sky-800 dark:to-sky-900"></div>
</div>
<main className="text-center">
<main className="text-center flex flex-col">
Copy link
Contributor

Choose a reason for hiding this comment

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

If you had added justify-center items-center here , it would have achieved the same result without the need to make changes to the button. Nevertheless, I believe the output remains consistent in both scenarios👍.

Copy link
Collaborator

@hemantwasthere hemantwasthere Oct 27, 2023

Choose a reason for hiding this comment

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

@tanpreetjolly Can you please do the changes that @sohan9819 recommended instead of current change you have done.

@sohan9819
Copy link
Contributor

LGTM 👍

@tanpreetjolly
Copy link
Contributor Author

Yeah I think you are right, that would achieve the same result without touching the button.

@@ -161,7 +161,7 @@ export const HeroSection = () => {
<div className="h-60 bg-gradient-to-br from-violet-500 to-violet-600 blur-[200px] dark:from-sky-900 dark:to-sky-950"></div>
<div className="h-60 bg-gradient-to-r from-violet-500 to-violet-600 blur-[200px] dark:from-sky-800 dark:to-sky-900"></div>
</div>
<main className="text-center">
<main className="text-center flex flex-col">
Copy link
Collaborator

@hemantwasthere hemantwasthere Oct 27, 2023

Choose a reason for hiding this comment

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

@tanpreetjolly Can you please do the changes that @sohan9819 recommended instead of current change you have done.

@tanpreetjolly
Copy link
Contributor Author

All right I will do that

@tanpreetjolly
Copy link
Contributor Author

@hemantwasthere I have done the changes you requested please check

Copy link
Collaborator

@hemantwasthere hemantwasthere left a comment

Choose a reason for hiding this comment

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

LGTM!

@hemantwasthere hemantwasthere merged commit 106d6ed into piyushgarg-dev:main Oct 29, 2023
2 checks passed
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.

[Bug]: Text overlapping with Button on Landing Page in Mozilla Firefox
3 participants