search

bryansaylor / homework-portfolio

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: Bryan Saylor

Project URL: https://github.com/bryansaylor/homework-portfolio

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: 15/16
barryross commented 7 years ago

Bryan,

Great work on this first assignment! You are doing some great things here! I left off one mark just b/c that tech requirement was looking for more styling (5+ declarations) on at least 5 elements on the page.

That being said, great work on this, and please inline comments for additional info. 👍