London | May - 26 | Liridona Shehu | Sprint 2 | Wireframe by shehu-dona · Pull Request #1371 · CodeYourFuture/Module-Onboarding

shehu-dona · 2026-05-30T11:36:58Z

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

Wireframe project completed. It has 2 articles with 3 different images and links.

netlify · 2026-05-30T11:37:02Z

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	`b3e66b7`
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a1acbdc958b8800088645c0
😎 Deploy Preview	https://deploy-preview-1371--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: 100 (no change 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

To better match the wireframe, can you
- Keep the page title and text beneath it centered on the page
- Make the title of articles 2 and 3 fully aligned horizontally
- Prevent the footer and article content overlap

cjyuan · 2026-05-30T21:31:30Z

      <h1>Wireframe</h1>
      <p>
        This is the default, provided code and no changes have been made yet.
      </p>


Can you also replace these placeholder text?

cjyuan · 2026-05-30T21:32:08Z

      <article>
-        <img src="placeholder.svg" alt="" />
-        <h2>Title</h2>
+        <img src="https://docs.github.com/assets/cb-69181/images/help/repository/headings-toc.png" alt="read.md img"/>


Providing meaningful alt text is important because it ensures images are accessible to people using screen readers and also helps when images
fail to load, giving users essential context about the content or function of the image.

Can you think of a more descriptive alt value to every image?

cjyuan · 2026-05-30T21:39:59Z

+        Think of it like creating a copy of your project where you can safely:<br>
+
+        Add new features<br>
+        Fix bugs<br>
+        Test ideas<br>
+
+        while the original code remains unchanged.


Using <br> to force line break is a poor practice.

Can you find a more suitable HTML elements, with or without using CSS, to list the three phrases on line 43-45?

dona-shehu added 3 commits May 29, 2026 20:30

Add Article for Read me,Wireframe,Git Branch.

155397d

Add img and link for each article

dbf7cc4

Fixed position of footer

b3e66b7