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

Align Contact Components and Update it in Home Page #40

Closed
Saumya40-codes opened this issue Dec 25, 2024 · 2 comments · Fixed by #45
Closed

Align Contact Components and Update it in Home Page #40

Saumya40-codes opened this issue Dec 25, 2024 · 2 comments · Fixed by #45
Assignees
Labels
good-first-issue priority: high Needs to be solved, in some case should be solved first for other issue to move forward size: medium

Comments

@Saumya40-codes
Copy link
Member

Saumya40-codes commented Dec 25, 2024

Carry Forwards: #11

The work for the contact us section is done with following PRs

Now, we can create a Contact Component where both of this are aligned based on the figma design: https://www.figma.com/design/kHauIvxVeOb8diy8jO6jJd/Website?node-id=0-1&t=VaLTx4vGpSXga6dU-1

Tasks

  • Align both ContactUs (see here) and Social media icons component (see here) based on the figma design in to one Contact component
  • Do any necessary changes required in the two component itself for the alignment as required in figma (this might include, changing width, height, icon sizes, etc)
  • Add this component(Contact) into src/pages/HomePage.jsx
@Saumya40-codes Saumya40-codes added priority: high Needs to be solved, in some case should be solved first for other issue to move forward size: medium good-first-issue labels Dec 25, 2024
@Aryansukhadia
Copy link
Contributor

@Saumya40-codes can you assign this issue to me?

@Saumya40-codes
Copy link
Member Author

@Saumya40-codes can you assign this issue to me?

👍 You can start with it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-first-issue priority: high Needs to be solved, in some case should be solved first for other issue to move forward size: medium
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants