-
-
Notifications
You must be signed in to change notification settings - Fork 622
Birmingham-WM5-HTML/CSS-Rawan-Almutairi-Karma Store| Week1/2 #656
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
…structions-clarifications Adds tasks and designs, and splits these into folders <!-- The title for your pull request should be made in this format Birmingham- WM5-Rawan-Almutairi-HTML/CSS-1/2 CITY CLASS_NO - FIRST_NAME LAST_NAME - MODULE - WEEK_NO For example, London Class 7 - Chris Owen - HTML/CSS - Week 1 Please complete the details below this message --> **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?
✅ Deploy Preview for cyf-module-project-html-css ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done @rawan0-0,
I have got some feedback for you.
Try and fill PR template to reflect your name and other questions there to help guide you properly.
Your Home page image is not showing I suggest you check the src to see if you fill in the right image source.
@@ -6,14 +6,78 @@ | |||
<title>Karma</title> | |||
<link href='https://fonts.googleapis.com/css?family=Roboto:400,500,300' rel='stylesheet' type='text/css'> | |||
<link rel="stylesheet" href="css/normalize.css"> | |||
<link rel="stylesheet" href="css/style.css"> | |||
<link rel="stylesheet" href="css\style.css"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this line of code your CSS is not well-linked you need to change " \ " to "/" to look like this (href="css/style.css").
Adds tasks and designs, and splits these into folders
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 repositoryYour Details
Homework Details
Notes
What did you find easy?
What did you find hard?
What do you still not understand?
Any other notes?