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

LOndon 10-Anu Thapaliya-HTML-CSS-Coursework-week1#317

Open
anuthapaliy wants to merge 3 commits intoCodeYourFuture:masterfrom
anuthapaliy:master
Open

LOndon 10-Anu Thapaliya-HTML-CSS-Coursework-week1#317
anuthapaliy wants to merge 3 commits intoCodeYourFuture:masterfrom
anuthapaliy:master

Conversation

@anuthapaliy
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?

Copy link
Copy Markdown
Member

@SallyMcGrath SallyMcGrath left a comment

Choose a reason for hiding this comment

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

Thanks for this @anuthapaliy . I really appreciate the time and effort you've put into this. I've left a few comments and asked a few questions. Have a think about them. If you'd like to make changes, you can push those changes to this PR and it will update.

Great work! Thanks! ⭐

Comment thread .vscode/launch.json
@@ -0,0 +1,22 @@
{
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.

Hi @anuthapaliy

I think you've committed a file by mistake. Don't use git add .. Add each file one by one and on purpose, so this doesn't happen for you.

Comment thread css/style.css
*/
body
{
Color: blue;
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.

I don't think this meets the style guide, do you? 🤔

The reason I think this is our style guide asks you to install Prettier and Prettier automatically formats your code.

https://syllabus.codeyourfuture.io/guides/code-style-guide#using-prettier-to-format-code-automatically

Comment thread index.html
<!-- Remember: Use semantic HTML tags like <header>, <main>, <nav>, <footer>, <section> etc -->

<body>
<h1>Social Media </h1>
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.

Is there a sectioning element you could use to group this h1 and p tag? What is the purpose of grouping content with semantic html?

Comment thread index.html
</article>

<article class="instagram">
<h3>Instagram</h3>
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.

Do you think that Instagram and Youtube should be on the same level as Facebook? It seems like there are three blocks the same with different text content on this page. What do you think?

Comment thread index.html
<a href="https://www.britannica.com/topic/YouTube">Read More</a>
</article>
<footer>
Contact-: Anu Thapaliya
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.

Is there a way you could mark up your name as an email?

Comment thread css/style.css
font-family: Impact, Haettenschweiler, 'Arial Narrow Bold', sans-serif;
font-size: medium;
}
.facebook{
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.

Can you think of a way to make a general class that controls the layout for all the articles on this page?

What does .class mean? How is it different from #id ?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants