Skip to content
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

Cheetahs - Ariam Y. #111

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Cheetahs - Ariam Y. #111

wants to merge 2 commits into from

Conversation

arusphere
Copy link

Personal Portfolio Site

Congratulations! You're submitting your assignment!

Comprehension Questions

Question Answer
Did you have to resolve any issues when running the HTML Validator? If so, what were they?
Why is it important to consider and use semantic HTML?
How did you decide to structure your CSS?
What was the most challenging piece of this assignment?
Describe one area that you gained more clarity on when completing this assignment
Optional
Did you deploy to GitHub Pages? If so, what is the URL to your website?

Copy link

@goeunpark goeunpark left a comment

Choose a reason for hiding this comment

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

Great job, Ariam! ✨

This website sparked a lot of joy! Your semantic HTML is clear and you squeezed in three website styles in one project. Great work going above and beyond, I think you and CSS will have a beautiful future ahead! This project is a Green. 🟢

Keep it up! 💯

Comment on lines +4 to +6
display: grid;
grid-template-columns: 20% auto 20%;
grid-template-rows: .5fr 1fr .25fr;

Choose a reason for hiding this comment

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

Fantastic use of grid!

Comment on lines +87 to +105
.tags li:nth-of-type(1) {
background-color: royalblue;
}

.tags li:nth-of-type(2) {
background-color: orange;
}

.tags li:nth-of-type(3) {
background-color: mediumvioletred;
}

.tags li:nth-of-type(4) {
background-color: teal;
}

.tags li:nth-of-type(5) {
background-color: purple;
}

Choose a reason for hiding this comment

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

Awesome! ✨

Comment on lines +16 to +18
display: flex;
flex-direction: column-reverse;

Choose a reason for hiding this comment

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

Great use of flexbox to align this header image! 💯

text-align: center;
font-weight: 300;
color:lavender;
background-color:whitesmoke

Choose a reason for hiding this comment

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

Don't forget your ending semicolons! 😉

Comment on lines +14 to +15
<a href="file:///Users/ariamyonas/Developer/projects/personal-portfolio-site/pages/index.html">Home</a>
<a href="file:///Users/ariamyonas/Developer/projects/personal-portfolio-site/pages/portfolio.html">Portfolio</a>

Choose a reason for hiding this comment

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

To make these hyperlinks accessible to other folks who pull up your code, we can access them by their relative path, like this:

<a href="index.html">Home</a>
<a href="portfolio.html">Portfolio</a> 

Comment on lines +46 to +54
<footer>
<nav id ='footer-nav'>
<ul class = 'footer-tags'>
<li><a href = 'https://github.com/arusphere'>GitHub</a></li>
<lil><a href = 'https://www.linkedin.com/in/ariam-y-5a8ab4134/'>LinkedIn</a></lil>
<lil><a href = '[email protected]'>Email</a></lil>
</ul>
</nav>
</footer>

Choose a reason for hiding this comment

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

Beautiful footer! 🌹

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