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

1565: Frontend Contact Us Form #2130

Merged
merged 18 commits into from
Mar 7, 2024
Merged

Conversation

Sun-Mountain
Copy link
Collaborator

@Sun-Mountain Sun-Mountain commented Mar 5, 2024

Changes Made:

  • Added new route /contact for Contact Us page
  • Created a form component to capture user inputs
  • Implemented form validation for required fields
  • Added a POST request to submit form data
  • Added success and error messages
  • Made sure styling was responsive

Screenshots

Screenshot 2024-03-05 at 2 48 51 PM Screenshot 2024-03-05 at 3 28 03 PM Screenshot 2024-03-05 at 2 49 34 PM Screenshot 2024-03-05 at 2 49 24 PM

Copy link
Collaborator

@mflymfly mflymfly left a comment

Choose a reason for hiding this comment

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

  • Box border radius: 8px
  • .contact-us-container: margin == .container plus (should line up with the content above). Let me know if you want me to redline any screens.
  • .contact-info: 24px between "Contact Information" and address info
  • New: address line top margin from 16px to 4px
  • email form: include icon and placeholder text (same as on homepage email subscribe form)
  • email error validation: no error shows when an email is submitted like this "mflynn@fearless"
  • New: move "A red asterisk" text line up to below the "Please reach out" text. Now updated in the designs(Sorry, my mistake there!)
  • Text input: please include "Your message" placeholder text
  • Text input box: 100 px height
  • Update the character count to 1000 characters for now. We need to confirm that but your point about the URL is great and I want to be wary of that for now.
  • Add in second gray info box regarding reporting training issues (now in designs and annotated)
  • new: unstyled "Clear form" button now has underline (like the clear all button on the Training Explorer search)

Tablet (1024px)

  • top margin of content: 48px

Mobile size

  • .contact-us-content margin: 32px 16px

  • buttons can be horizontally aligned side by side like in design here

Copy link
Collaborator

@ChelseaKR ChelseaKR left a comment

Choose a reason for hiding this comment

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

LGTM, pending design changes requested by @mflymfly. Thanks so much @Sun-Mountain!! great job w/ this

@ChelseaKR ChelseaKR merged commit 1190f73 into contact-us-form Mar 7, 2024
2 checks passed
@ChelseaKR ChelseaKR deleted the nz/1565-contact-form branch March 7, 2024 16:11
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.

None yet

3 participants