Skip to content

Redesigning the contact page #129

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.

Sign up for GitHub

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

Jump to bottom
Closed
dikshashukla455 opened this issue Apr 9, 2022 · 6 comments · May be fixed by #158
Closed

Redesigning the contact page #129

dikshashukla455 opened this issue Apr 9, 2022 · 6 comments · May be fixed by #158
Assignees
@dikshashukla455

Comments

@dikshashukla455
Copy link
Contributor

Thank you for taking the time to file a Bug report.

Describe the bug
There are two bugs in the contact page:

  1. the color of the contact card looks bulky.
  2. the contact form should be in the center.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the website (https://journal-policy-tracker.netlify.app/)
  2. Click on the Contact link in the navbar
  3. Scroll down to the contact form
  4. See an error

Actual behavior

Screenshots
Screenshot (1030)

Expected behavior
The color of the contact card should be off-white, have a slightly greyish box-shadow, and have a hover transition. The contact form should be in the middle and add a heading 'send us a message above the contact form to enhance the user's experience.
Screenshots
ezgif com-gif-maker (8)

Screenshot (1043)

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Smartphone (please complete the following information):

Device: OnePlus Nord CE
OS: Android 11
Browser: Chrome
Version [e.g. 22]
@dikshashukla455
Copy link
Contributor Author

Greetings @padmajabhol, Can I work on this issue? If it is valid.

@padmajabhol
Copy link
Member

Not a major ui change but could be nice to have. Do we need this @yochannah @ikayz ?

@ikayz
Copy link
Contributor

ikayz commented Apr 12, 2022

@padmajabhol would be nice to have that, yes. Though the call to action bit not sure I'd have to loop in @yochannah

@yochannah
Copy link
Contributor

yochannah commented Apr 15, 2022 via email

@dikshashukla455 dikshashukla455 mentioned this issue Apr 22, 2022
Closed
4 tasks
@dikshashukla455
Copy link
Contributor Author

Hello everyone, I have added the PR . Kindly review it. Thank you.

@dikshashukla455 dikshashukla455 mentioned this issue May 9, 2022
Open
4 tasks
@dikshashukla455
Copy link
Contributor Author

hi @padmajabhol, I have resent the pull request which has no merge conflicts. Kindly review it. Thank you

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

@dikshashukla455 dikshashukla455

Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@yochannah @ikayz @dikshashukla455 @padmajabhol