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

london8-Ezgi Gunes-HTML/CSS-Week3 #251

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Ezgi3406
Copy link

@Ezgi3406 Ezgi3406 commented Dec 3, 2021

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?


/* TABLET: */
@media screen and (min-width: 550px) {
.homepage-main {

Choose a reason for hiding this comment

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

I couldn't find .homepage-main class in HTML. Am I missing it ?


}

.footer h1 {

Choose a reason for hiding this comment

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

It makes me confused as footer and h1 are element names in HTML, may be it would be better rename for this class.

"feature feature feature . aside aside"
"product product product . aside aside";
}

Choose a reason for hiding this comment

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

These grid area names don't seem to exist in CSS .

@LucyMac LucyMac added the LDN-8 London class #8 (start Nov 2021) label Dec 4, 2021
@jstadnik jstadnik added the waiting for changes A teacher has requested that you make some changes to your work label Dec 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
LDN-8 London class #8 (start Nov 2021) waiting for changes A teacher has requested that you make some changes to your work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants