Skip to content

Manchester | 26 ITP May | Szidi Bodo | Sprint 1 | Form Control#1259

Open
bodoszidi wants to merge 7 commits into
CodeYourFuture:mainfrom
bodoszidi:Manchester_26-ITP-MAY_Szidi-Bodo_Sprint1_Form-Control
Open

Manchester | 26 ITP May | Szidi Bodo | Sprint 1 | Form Control#1259
bodoszidi wants to merge 7 commits into
CodeYourFuture:mainfrom
bodoszidi:Manchester_26-ITP-MAY_Szidi-Bodo_Sprint1_Form-Control

Conversation

@bodoszidi
Copy link
Copy Markdown

Self checklist

  • [x ] I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • [x ] I have tested my changes
  • My changes follow the style guide

Changelist

Created the form with validation and basic style - only using HTML and CSS
Lighthouse accessibility check is 100%

Questions

Had difficulties with centring the content - I'm not quite sure why flex on my CSS was not working

bodoszidi added 4 commits May 12, 2026 21:49
-removed text from the form section
-updated footer with my name
added name and email with validation
added T-shirt colour and size to the form with validation
added css style to the content
@netlify
Copy link
Copy Markdown

netlify Bot commented May 12, 2026

Deploy Preview for cyf-onboarding-module ready!

Name Link
🔨 Latest commit 7cf1c6d
🔍 Latest deploy log https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a04d2f4d1a95f000860356e
😎 Deploy Preview https://deploy-preview-1259--cyf-onboarding-module.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
2 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 84 (🔴 down 2 from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

This comment has been minimized.

@bodoszidi bodoszidi changed the title Manchester | 26 itp may | szidi bodo | sprint1 | form control Manchester | 26 itp may | szidi bodo | sprint 1 | form control May 13, 2026
@bodoszidi bodoszidi added 📅 Sprint 1 Assigned during Sprint 1 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Onboarding The name of the module. labels May 13, 2026
@Sambuxc Sambuxc added Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels May 13, 2026
@Sambuxc
Copy link
Copy Markdown

Sambuxc commented May 13, 2026

Great start a few points to mention:

  1. (minor) Your git commit messages all say "Updated index.html"
    This is not good practice because it doesn't give any idea of what was updated. Please bare that in mind for future commit messages.

  2. (critical) You are missing the form submit button
    I am unable to check the required status of your form inputs because there is no submit button. Please add a submit button and check that the required fields work as expected

@Sambuxc Sambuxc added Reviewed Volunteer to add when completing a review with trainee action still to take. and removed Review in progress This review is currently being reviewed. This label will be replaced by "Reviewed" soon. labels May 13, 2026
@Sambuxc
Copy link
Copy Markdown

Sambuxc commented May 13, 2026

Question Response

Q. Had difficulties with centring the content - I'm not quite sure why flex on my CSS was not working

What's important to understand with flexbox is depending on the flex-direction value (row/column), will change how justify-content and align-items work.

It's useful to play around with flex row and adjust the values of the other flex properties vs. flex column - compare how they work.

A useful tool I have used in my early days of learning flexbox - https://flexbox.malven.co/

Pay attention to the keywords "main axis" and "cross axis".

Have a read up and try to achieve what you were intending on this PR to make your form content central.

Just to show you it can be done, I have achieved centering the form content using only 3 css properties:

Screenshot 2026-05-13 at 17 59 54

bodoszidi added 3 commits May 13, 2026 20:24
closed the select tag and added 2 buttons to the form
-updated form field and add required attributes
-fixed styling by adding flex and updating width on CSS
updated the button css style
fixed the colour selector css
@bodoszidi bodoszidi changed the title Manchester | 26 itp may | szidi bodo | sprint 1 | form control Manchester | 26 ITP May | Szidi Bodo | Sprint 1 | Form Control May 13, 2026
@bodoszidi bodoszidi added Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. and removed Reviewed Volunteer to add when completing a review with trainee action still to take. labels May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Module-Onboarding The name of the module. Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 📅 Sprint 1 Assigned during Sprint 1 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants