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

London class 9 -Mickey Haile - HTM/CSS -week 1 - #243

Open
mickeyhaile2 wants to merge 5 commits into
CodeYourFuture:masterfrom
mickeyhaile2:master
Open

London class 9 -Mickey Haile - HTM/CSS -week 1#243
mickeyhaile2 wants to merge 5 commits into
CodeYourFuture:masterfrom
mickeyhaile2:master

Conversation

@mickeyhaile2

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:
  • 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?

@JDysiewicz
JDysiewicz self-requested a review October 26, 2022 19:27
@JDysiewicz

Copy link
Copy Markdown
Member

try and make sure to fill out the template body; knowing what you find hard/easy or any sticking points is really useful for us :P

@JDysiewicz JDysiewicz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looking good so far, add a couple more article tags and use the pseudo selector first child to style the first article.

Comment thread index.html
Comment on lines +56 to +62
<ul class="arty2">Fun Facts About Rabbits</ul>


<Li>To express happiness, bunnies will sometimes jump around and flick their heads and feet. ...</li>
<li>Like deer, a female rabbit is called a “doe” and a male rabbit is called a “buck.”</li>
<li> bunny's big ears aren't just for listening! ...</li>
<li> those amazing ears can also be rotated almost a full circle to 270 degrees.</li>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

your ul should wrap your list items to make the structure easier to parse - e.g.

<ul class="arty2">
   <li>Some text</li>
   <li>some other text</li>
</ul>

Comment thread index.html
Comment on lines +48 to +50
<iframe width="695" height="391" src="https://youtu.be/XF4TimSeGiU"
title="Thing you need to know about rabbits" frameborder="0"
allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice use of an iframe to make the site a bit more dynamic

Comment thread css/style.css


.hello{
font-size: 100px;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

100px is very large for a font size 😅

Comment thread css/style.css
Comment on lines +32 to +40
.hello1{
font-size: 100px;
text-align: center;
padding-left: 20px;
padding-right: 20px;
font-size: 18px;


}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

instead of repeating the same stylings for multiple different classes, you can apply the styling to a single class, and apply multiple classes to an element (e.g. <div class="mystyling hello1">)

Comment thread index.html
<br>
<div class="arty">
<h2 class=" links">Summary</h2>
<article>When you look at the rabbits they inspire us to look after our enviroment therefore we must protect this precious species <br>as they

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice use of an article here, although you need to add a couple more article tags, and use the pseudo selector first child to style the first one

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants