search

lilliizumi / homework_portfolio_repo

0 stars 0 forks source link

Feedback on assignment #1 #1

Open barryross opened 7 years ago

barryross commented 7 years ago

FEWD Week #1 Project: HTML/CSS Basics

Description

For the first assignment, you will begin creating the beginnings of your first website. Take what you've learned from class and build a portfolio site that showcases all of the projects you'll be completing over the next few weeks. Build two pages - an 'About Page' and a 'Portfolio' - that link to one another. Your 'About Page' should describe your mission and value proposition and your 'Portfolio' should include descriptions of the work you [will] do for Relaxr, Startup Matchmaker, and CitiPix.

Student: Lilli Izumi

Project URL: https://github.com/lilliizumi/homework_portfolio_repo__

Technical Requirements Does Not Meet Expectations (0) Meets Expectations (1) Exceeds Expectations (2)
Link both pages together using a tag x
Use a unique title and a single unique h1 tag (so, each page should have one of each of these tags, and while on a particular page, the content of the title and the h1 can be the same, the content of these tags should be different on the "about" page then they are on the "portfolio" page x
Show images using img tags including unique alt attributes x
Use an external CSS stylesheet to style your pages x
Select at least 5 elements on the page and apply at least 5 different css properties (font-size, color, font-family, background, text-decoration) to these elements (so, your style.css will have at least 5 css rules, that each contain at least 5 CSS declarations) x
- Use at least 6 of these HTML tags on each page: (h1, h2, h3, h4, h5, h6, p, ul, ol, a, img, nav, span) x
Follow naming conventions, maintain consistency across your .html and .css files x
Indent nested elements to increase your code's readability x
TOTAL: 16/16
barryross commented 7 years ago

Hey Lilli, Great work on this first assignment! I'm happy to see you personalized the assignment and included info about yourself! I like the little story on your homepage!

I encourage you to see how you could consolidate your three separate .css files into a single one that both pages could link to and use.

And, it would be totally fine to treat your index.html file as if it were about-me.html. To clarify, I notice index.html and about-me.html are very similar, and typically index.html is the homepage of a website...so I would encourage you to just have the two files, index.html and portfolio.html, because in the case of this site, about-me.html is essentially acting as the homepage. Please let me know if you have questions about what I'm suggesting here, and see the inline comments for additional feedback.

Thanks!