Skip to content

Mitch - #40

Open
MFiissel wants to merge 2 commits into
masterfrom
mitch
Open

Mitch#40
MFiissel wants to merge 2 commits into
masterfrom
mitch

Conversation

@MFiissel

@MFiissel MFiissel commented Oct 1, 2020

Copy link
Copy Markdown
Collaborator

I believe i understand now

@sprihozhiy sprihozhiy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Try to read again about the fundamentals of HTML on MDN and rewrite your code, please.

https://developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML

@@ -0,0 +1,34 @@
<html>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing above the <html> tag



<head>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

<head> should has at least <title> tag


<body>
<h1>Welcome to the <b>sports store</b></h1>
<h2><hr>Hear you will find all your sports need <p></h2>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What was the purpose of putting <hr> and <p> tags inside of the <h2> tag?

<body>
<h1>Welcome to the <b>sports store</b></h1>
<h2><hr>Hear you will find all your sports need <p></h2>
<ul>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Inside of this tag can be only a list item tag (<li>)

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