From 426e3428ae32ee4e91ddcbecbeb122fb45ae04c6 Mon Sep 17 00:00:00 2001 From: ekezieb <59987102+ekezieb@users.noreply.github.com> Date: Wed, 17 Mar 2021 15:41:42 -0700 Subject: [PATCH 1/4] Update README.md --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index 6f13627..6eb10ae 100644 --- a/README.md +++ b/README.md @@ -1,5 +1,7 @@ # PhotoHub +README.md is very impressive. Very detailed and emplains the components of the project. + ## Project Objective Photohub is a platform that users can view, share and comment on their photos. From 22aa37691382be1abcbc2d266312e44868ca5fd8 Mon Sep 17 00:00:00 2001 From: ekezieb <59987102+ekezieb@users.noreply.github.com> Date: Wed, 17 Mar 2021 16:15:07 -0700 Subject: [PATCH 2/4] Project 2 Code Review: index.html Suggested adding container for body page and including the main tag for code after navbar. --- public/index.html | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/public/index.html b/public/index.html index fcfc306..a87bbf9 100644 --- a/public/index.html +++ b/public/index.html @@ -89,8 +89,9 @@ - - +In your body, I would suggest having all your content be in a container as it could help keep your code organized and you'd be able to use some of the bootstraps' special attributes to style your code more easily. +Also, I would use the main tag to separate your navbar from your base content. +