NW6 | Rabia Avci | HTML-CSS | Week 1

[RbAvci]

Use semantic markup
Fix the broken images
Style buttons
Add spacing
Hover effects

[netlify]

✅ Deploy Preview for cyf-bfr ready!

Name	Link
🔨 Latest commit	9acaa5e
🔍 Latest deploy log	https://app.netlify.com/sites/cyf-bfr/deploys/6519fec89b51220008da11ff
😎 Deploy Preview	https://deploy-preview-472--cyf-bfr.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[Ara225]

Hi,
Some good work here, you're doing very well but also issues I'm afraid. Here's what needs to change:

• The text and buttons over the big images are not in the same place as the images say
• All the text in the cards is orange, not just the headers
• There's far more padding/whitespace between the edge of the page and the content.
  Please try to fix and If this doesn't make sense, let me know

[RbAvci]

Thanks for your review @Ara225!

I studied flex layout and then decided to re-work on this task. Therefore I started from scratch and fixed all the issues. I will appreciate it if you can review it again.

Thanks in advance!

[MaryamAlattal]

Semantic markup was added successfully.

[MaryamAlattal]

-The broken images were successfully added with the absolute path.
-The required colour in the CSS was corrected.

[MaryamAlattal]

The buttons were styled properly with a hover effect.

[MaryamAlattal]

The code was corrected by adding class to style.css, Well done!

[MaryamAlattal]

Good job by modifying the class.

[MaryamAlattal]

The transition and hover effects are working properly.

[MaryamAlattal]

The spacing and fixing positions are working well, but since you used Flexbox try to add a collapsible navbar.

[RbAvci]

The spacing and fixing positions are working well, but since you used Flexbox try to add a collapsible navbar.

• Thanks for your review, Maryam. Do you mean to add .navigation__list { flex-wrap: wrap;} ?

Hi, Some good work here, you're doing very well but also issues I'm afraid. Here's what needs to change:

• The text and buttons over the big images are not in the same place as the images say
• All the text in the cards is orange, not just the headers
• There's far more padding/whitespace between the edge of the page and the content.
  Please try to fix and If this doesn't make sense, let me know

-Thanks for your review, Anna. I've made all the changes to the lists above as you've asked.

[Ara225]

Looking really good! I think everything I metioned has been fixed well :)