London | 26-ITP-May | Remi M. | Sprint 2 | Wireframe

[remimarcelle]

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
This pull request attempts to complete the requirements for the Wireframe task. The page is built using using semantic HTML tags and includes a header, main section, and footer. Inside the main section are three articles, each featuring a clear heading, a casual and informative summary, and a functional external link.

The footer has been updated with contextual information about the project, including a link to the relevant GitHub issue.

Lighthouse scores at the time of submission were 92 for accessibility

Related issue: remimarcelle/My-Coursework-Planner#7

Questions
There was something I really struggled with and that is making the images fit into the article boxes, and also putting a divider line between the image and article title. I tried so many different ways. So, if you have advice about that that would be great. Also, what do I do with all of these forks? Do I just delete them? Because it's making my Github messy.

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	87093d3
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a0c78ecd5739c0008f5b98a
😎 Deploy Preview	https://deploy-preview-1306--cyf-onboarding-module.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	2 paths audited Performance: 100 (no change from production) Accessibility: 100 (no change from production) Best Practices: 96 (🔴 down 4 from production) SEO: 86 (no change 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.

[cjyuan]

Code looks good. Can you address the following issues to fully meet all the requirements?

• For each article, include an actual summary and assign a URL of a relevant resource to the link.

• Improve the Lighthouse Accessibility score from 92 to 100.

• Address the warning given by https://validator.w3.org/.

[remimarcelle]

done

[cjyuan]

Changes look good. Well done!

[cjyuan]

There was something I really struggled with and that is making the images fit into the article boxes, and also putting a divider line between the image and article title. I tried so many different ways. So, if you have advice about that that would be great.

If it were me, I would consult AI, asking it interactively until I get the layout I expect and until I understand every piece of code AI shows me.

CSS Grid Layout is a very powerful way to layout elements. You may wan to explore and experiment with this layout mechanism.

Also, what do I do with all of these forks? Do I just delete them? Because it's making my Github messy.

If you are certain you do no need anything in a repo for good, you can delete the repo.
If you want to hide your repo from public, you can make them private.