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

Feature: Develop CTA & Footer #46

Merged
merged 5 commits into from
Dec 11, 2024
Merged

Feature: Develop CTA & Footer #46

merged 5 commits into from
Dec 11, 2024

Conversation

samoina
Copy link
Collaborator

@samoina samoina commented Dec 8, 2024

Develop CTA & Footer (#18)

📋 Description:

This pull request addresses Issue #18, focusing on creating the CTA and footer section within the home page. The aim was to replicate the design, ensure a modern, responsive user interface and write e2e tests for the components.


✅ What Was Done:

  • Develop the Call To Action section and footer per the Figma designs
  • Make sure the components are fully responsive
    -[X] Write e2e tests in the tests folder, to test the components' look, feel, and functionality

return (
<div
data-test="FooterCTA"
className="relative flex flex-col items-center gap-4 p-4 md:gap-10 lg:flex lg:flex-row lg:p-[100px] lg:pl-[20%]"
Copy link
Collaborator

Choose a reason for hiding this comment

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

consider removing the realtive style to make sure it's properly positioned when scrolled to the end

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I have corrected this to ensure it scrolls well to the end without the initial empty space

@tigawanna tigawanna merged commit 4beac6b into main Dec 11, 2024
1 check 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.

2 participants