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

Manchester NW_4-Erin_Dyson-HTML/CSS-4 - #62

Closed
erin-switchstitch wants to merge 5 commits into
CodeYourFuture:masterfrom
erin-switchstitch:master
Closed

Manchester NW_4-Erin_Dyson-HTML/CSS-4#62
erin-switchstitch wants to merge 5 commits into
CodeYourFuture:masterfrom
erin-switchstitch:master

Conversation

@erin-switchstitch

@erin-switchstitch erin-switchstitch commented May 21, 2021

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: Erin Dyson
  • Your City: Manchester
  • Your Slack Name: erin-switchstitch

Homework Details

  • Module: HTML / CSS
  • Week: Lesson 1

Notes

  • What did you find easy?
    Aligning text with inline elements

  • What did you find hard?

Working out how to get 3 elements (2 images, 1 text) to display across the width of the screen
Also working out why some changes won't overide a selector with lower specificity

  • What do you still not understand?

Making the website properly resposive

  • Any other notes?

I have set out most of the HTML and CSS. Still need to make HTML semantic, and need to finetune the CSS so that all the elements are spaced and styled correctly.
Started to make the page semantic by adding article elements in <main>. I have also added a 3rd article. Need to add links.
Added links to all the articles. Made first:child for the first article to make it slightly bigger. Still need to fix the header (it doesn't stretch evenly).
Changed some styling elements, and worked on some responsivity. Need to do more work to get this page properly responsive. Hoping to avoid loads of @media requests
Comment thread index.html
Comment on lines +29 to +47
<header><div id="wrapper">

<div><image class="h1-image-1"
src="https://1gew6o3qn6vx9kp3s42ge0y1-wpengine.netdna-ssl.com/wp-content/uploads/prod/sites/5/2020/12/cyver.jpg"
alt="Locked paddlocks with a blue background"></image></div>
<div>
<h1> Cyber Security in the 21st Century</h1>
</div>
<div>
<image class="h1-image-2"
src="https://1gew6o3qn6vx9kp3s42ge0y1-wpengine.netdna-ssl.com/wp-content/uploads/prod/sites/5/2020/12/cyver.jpg"
alt="Locked paddlocks with a blue background"></image>
</div>





</div></header>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There are a couple of improvements you can make in terms of code formatting here - for example putting child elements on a new line, removing some of the extra whitespace etc.

Also - the best tag to use for images is (it is self closing). will work in most browsers but is not an official part of the HTML tag set! which can be confusing

Comment thread index.html

<div><image class="h1-image-1"
src="https://1gew6o3qn6vx9kp3s42ge0y1-wpengine.netdna-ssl.com/wp-content/uploads/prod/sites/5/2020/12/cyver.jpg"
alt="Locked paddlocks with a blue background"></image></div>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

good alt text :)

@peedeerich

Copy link
Copy Markdown

Well done on this project! This is a good effort and the site works well in terms of responsiveness - I know that you have identified from the subsequent course work that you could adopt a "mobile first" approach to this using some of the tools we have now covered which would help to tidy up some of the CSS etc

a couple of more specific observations:

  • your HTML code would benefit from some formatting (checking indentation is correct, putting child elements on new lines etc.) You can get 90% of the way there in VS code by auto formatting
    On Windows Shift + Alt + F
    On Mac Shift + Option + F
    On Ubuntu Ctrl + Shift + I
  • there are a few elements which seem to exist only to create space in the layout (e.g.
    ) - this isn't particularly semantic. I suspect you know a few more CSS tools that would help with this now! but simply moving margins etc to the element you are positioning would help

But overall this is very good!

@github-actions

Copy link
Copy Markdown

Your coursework submission has been closed because nobody has interacted with it in six weeks. You are welcome to re-open it to get more feedback.

@github-actions github-actions Bot added the Stale label Jul 30, 2021
@github-actions github-actions Bot closed this Jul 30, 2021
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