Skip to content

ITP-2026-1 , ahmad deeb , week 1 , Feature/wireframe #73

Closed
deebahmad197-sketch wants to merge 3 commits intoHackYourFutureBelgium:mainfrom
deebahmad197-sketch:feature/wireframe
Closed

ITP-2026-1 , ahmad deeb , week 1 , Feature/wireframe #73
deebahmad197-sketch wants to merge 3 commits intoHackYourFutureBelgium:mainfrom
deebahmad197-sketch:feature/wireframe

Conversation

@deebahmad197-sketch
Copy link

@deebahmad197-sketch deebahmad197-sketch commented Feb 20, 2026

Learners, PR Template

Self checklist

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

Changelist

Briefly explain your PR.

Questions

Ask any questions you have for your reviewer.

@deebahmad197-sketch
Copy link
Author

the last commit is my only one , the first two was a mistake

@fgsumer
Copy link

fgsumer commented Feb 24, 2026

Thanks for your submission 🙂

Firstly, two things regarding structure:

  • You created a new folder (index.html.2), which was not required for this assignment. Please follow the existing structure unless the instructions say otherwise.
  • Your file is named new.txt. It must be an .html file (for example, index.html). A .txt file will not be interpreted as a webpage by the browser.

Secondly,

  • You need to update style.css file too. Even minimal styling is expected.
  • Fix typos: “undertanding”, “visiual”, “elemnts”
  • Run Lighthouse and fix any remaining accessibility warnings to reach 100

Be sure your assignment meets all the acceptance criteria described in README.md file.

@fgsumer fgsumer closed this Mar 22, 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.

2 participants