Skip to content
This repository has been archived by the owner on Jan 14, 2024. It is now read-only.

London Class 10 - Iryna Lynyk - HTML/CSS I - Coursework - Week 1 #341

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

IrynaLypnyk
Copy link

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

@IrynaLypnyk IrynaLypnyk added the review requested I would like a mentor to review my PR label Apr 5, 2023
@IrynaLypnyk IrynaLypnyk changed the title London Class 10 - Iryna Lynyk - HTMl/CSS - Week 1 London Class 10 - Iryna Lynyk - HTML/CSS I - Coursework - Week 1 Apr 5, 2023
Copy link

@migmow migmow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great piece of work! really well done 💯 I only suggest using more descriptive class names. For example which image you mean by img class.
Also, you have some common styles applied to different elements throughout your code. Consider consolidating these common styles into reusable classes that you can apply to multiple elements.

.article img{
max-width: 60%;
}
}
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one in terms of responsiveness!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
review requested I would like a mentor to review my PR reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants