Skip to content

Miguel Garcia#13

Open
garcia13 wants to merge 6 commits intojoinpursuit:mainfrom
garcia13:main
Open

Miguel Garcia#13
garcia13 wants to merge 6 commits intojoinpursuit:mainfrom
garcia13:main

Conversation

@garcia13
Copy link

No description provided.

<title>Personal Website</title>
</head>
<body>
<div class="menu">
Copy link
Contributor

Choose a reason for hiding this comment

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

Use semantic HTML like nav, main, section where ever possible

</div>
</section>

<footer><br /><br /><br /><br /><br /><br />Thanks For Visiting!</footer>
Copy link
Contributor

Choose a reason for hiding this comment

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

Try to avoid using br tags. Instead use css to control all appearance.

@@ -0,0 +1,218 @@
body {
font-family: Ubuntu, Roboto, sans-serif;
display: flexbox;
Copy link
Contributor

Choose a reason for hiding this comment

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

display: flex not flexbox

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