search

aer03280 / portfolio

Portfolio of Ashley's work
MIT License
0 stars 1 forks source link

portfolio suggestions #5

Open sueanyv opened 7 years ago

sueanyv commented 7 years ago

I think your portfolio structure looks good I suggest you separate your app.js like it is in our blog app.. this means having your data in one file and the bottom part functions in another js file. Also this is just suggestion but the top links you can use icomoon or font awesome site to have icon pics as links so the nav bar may look something like this https://sueanyv.github.io/DesPicker/

thegrimheep commented 7 years ago

Everything in your portfolio looks good. When I forked it the hamburger menu did not work, if you need some help with that let me know and I will help you through it. It took me a few days to get mine going but I don't have a nav menu working. Nice work so far!