Skip to content

Comments

Kayla contact form #35

Open
KaylaYuChen wants to merge 9 commits intomainfrom
kayla-contact-form
Open

Kayla contact form #35
KaylaYuChen wants to merge 9 commits intomainfrom
kayla-contact-form

Conversation

@KaylaYuChen
Copy link
Collaborator

@KaylaYuChen KaylaYuChen commented Feb 19, 2026

Made the Contact Form

Screenshot 2026-02-18 at 11 51 29 PM

Checklist:

after creating your PR please mark all of these:

-[ ] Global colors used and I also used this: rgba(87,87, 87, 0.80) that was missing from globals.scss

  • Text/images/paths are not hard coded (passed in by props)

Notes & Questions for Kyhara and Austin:

  1. I used the 100% for width...online/chat said if use rem it will break logic and inputs in a row may overflow
  2. -I used the field font listed in Figma but I'm unsure why they don't look identical

@KaylaYuChen KaylaYuChen linked an issue Feb 19, 2026 that may be closed by this pull request
Copy link
Collaborator

@austin1im austin1im left a comment

Choose a reason for hiding this comment

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

your logic for the formData and handling the change and submit is great! pls fix styling and also i think u need to merge with main to get the space grotesk font. there's also the _components from main and it would be nice if you could move your component into there

@@ -0,0 +1,72 @@
.container{
width: 100%;
max-width:1000px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove this max-width

color: var(--dark-gray);
}
.form input,
.form textarea{
Copy link
Collaborator

Choose a reason for hiding this comment

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

there should only ever be one .form, all ur html element stylings that you're applying should be form should be nested and done like below
.form {
----styling
----textarea {
--------styling
----}
}

Copy link
Collaborator

Choose a reason for hiding this comment

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

also same thing with setting placeholder stylings, nest in whatever u need it to be and then do like &::placeholder { styling }

.submit{
align-self: flex-end;
padding: 0.7rem 2.2rem;
border-radius: 0.6rem;
Copy link
Collaborator

Choose a reason for hiding this comment

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

ur border-radius doesn't match figma

}

.field label {
font-family: "Space Grotesk", sans-serif;
Copy link
Collaborator

Choose a reason for hiding this comment

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

why is there also sans-serif

}

.form textarea{
min-height: 140px;
Copy link
Collaborator

Choose a reason for hiding this comment

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

should be rem

@austin1im austin1im assigned austin1im and KaylaYuChen and unassigned austin1im Feb 19, 2026
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.

Contact Form

2 participants