London | 26-ITP-May | Gideon Defar | Sprint 2 | Wireframe

[gideondefar]

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

I reviewed and resubmitted the wireframe exercise.

[netlify]

✅ Deploy Preview for cyf-onboarding-module ready!

Name	Link
🔨 Latest commit	1bfbb95
🔍 Latest deploy log	https://app.netlify.com/projects/cyf-onboarding-module/deploys/6a1ce9677b986f0008456a67
😎 Deploy Preview	https://deploy-preview-1298--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: 91 (🟢 up 5 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]

• Your branch is 16 commits behind the CYF's main. Can you update it? Some of the requirements have changed.

• One of the new requirements is that each article should have a unique image. Can you address this issue?

• Some of the code is not consistently formatted. Please follow this guide to
  enable "Format on Save" in VS Code or use VS Code's "Format Document" feature to keep your code consistently formatted.

[cjyuan]

This is how your current page look like on Netlify:

Its layout does not quite match the wireframe.

• The images are "missing"
• There should be a page title (and a short description beneath it) and both should be centered on the page
• The footer content should also be centered

---

This branch is still not yet up to date. Please use the "Sync fork" button to keep it updated.

---

According to https://validator.w3.org/, there are errors in your code. Can you fix them?

[gideondefar]

I have synced the fork and updated the branch using the "Sync fork" button. The branch is now up to date with the latest changes. Please let me know if there is anything else to action.

[cjyuan]

Code looks good.

This branch included a modified Form-Controls/index.html, which is not related to the Wireframe exercise.
Can you revert the change made to that file or restore the file to its original state?

I will mark this PR as complete first.

---

Here is one possible way to revert changes to a file is via the git restore command.

First, locate a commit before the file was modified. It could be the first commit you made, assuming the branch started in a clean state.

Next, record the first 7 characters of the commit SHA.

Suppose the file is path/to/file, and you want to restore the file to its state before the commit with SHA 1234567.
The command is:

git restore --source=1234567^ path/to/file

After the file is restored, make a commit and push the changes to GitHub.

[cjyuan]

That works well. Good job.