apalm112 / portfolio Goto Github PK

Change the callback to the .fetchAll method in portfolio.js with the appropriate method name instead of just "callback".

Finish the .countRepos method so that it appends the repository info to the DOM with the .map, .reduce methods & Handlebars template.

• On 'portfolio.js' need to work out .reduce(); -

Project.countRepos = function(callback) { return Project.all.map(function(title) { console.log(title); return title; });

The h1 tag in the header is not centered on the page. Move to the center for a better look for it.

Resolve how items appear over background image. Image is currently static.

create a state.css file

The about info section that's appended upon click event on the about tab, is offset to the left. Move to the center for a better look.

Move hover, active, link, and focus from base.css to state.css

Refactor the Project constructor function in portfolio.js so that it doesn't name each specific property, title, updatedOn, course.

You can remove - <script src="data/projects.json"></script> - from index.html. It's already running in ajax call.

The main-nav is not aligned up with the h1 tag and the code.png. Re-align those three items for a better look to the page.

In the Project.prototype replace the .forEach method with an appropriate Functional Programming method, such as filter, map or reduce.

Linkedin link needs to go to your profile. When I opened it it went to mine.

Remove the body as an attribute to the Project object. The entire body of it does not need to be a data attribute.

The hamburger menu icon is not being appended to the DOM in mobile view. Possible fix be reinstalling the icomoons files.