Skip to content

NW5-Manchester-Doris Siu-HTML&CSS-Week1 - #476

Open
Doris-Siu wants to merge 5 commits into
CodeYourFuture:masterfrom
Doris-Siu:master
Open

NW5-Manchester-Doris Siu-HTML&CSS-Week1#476
Doris-Siu wants to merge 5 commits into
CodeYourFuture:masterfrom
Doris-Siu:master

Conversation

@Doris-Siu

Copy link
Copy Markdown

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: Doris Siu
  • Your City: Manchester
  • Your Slack Name: Doris Siu

Homework Details

  • Module: 2
  • Week: 1

Notes

  • What did you find easy? HTML

  • What did you find hard? CSS

  • What do you still not understand? /

  • Any other notes? /

Comment thread css/style.css
}


.rect_4 {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Doris nice job :)
just use a meaningful class name in your html so when you use them as a selector in css your code is more readable.

Comment thread index.html
<div class="main">


<div class="hero">

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

you can also use semantic html instead of div, such as

which you use them in some parts of your code 👍

Comment thread index.html
<h2 class="heading-underline">Everyone needs a little Karma.</h2>

</div>
<br>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

for spacing between some parts you can use Margin or Padding instead of br :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants