Skip to content

Conversation

@pujitakalinadhabhotla
Copy link

I added buttons to the volunteer sign-up. I added Next, Back and Submit buttons to match the figma.
Screenshot 2025-02-05 at 3 09 06 AM
image

Copy link

@robpatterson13 robpatterson13 left a comment

Choose a reason for hiding this comment

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

Looks good to me! Just one small thing:

Comment on lines 29 to 39
.card.back .card-text {
display: flex;
align-items: center;
justify-content: center;
text-align: center;
padding: 12px;
font-size: 16px;
line-height: 1.5;
height: 100%;
}

Choose a reason for hiding this comment

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

💭 It looks like this might be a change for GI-60 (I could be wrong though, let me know if I am!)

<Input variant="filled" height="40px" width="100%" />
</VStack>

const termsAndConditionsCheckboxesMap: CheckboxField[] = [

Choose a reason for hiding this comment

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

Could you explain why this deletion was made?

Copy link

@hams7504 hams7504 left a comment

Choose a reason for hiding this comment

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

Besides those 2 comments you may have to address, everything looks great!

@mahekagg mahekagg merged commit 1f7639b into main Feb 22, 2025
3 checks 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.

6 participants