Peer Review- Yirong Shen

[yshen12138]

Hi, Dave, I like the idea your are making a website about yourself, but I do think you have a lot to work on still!

Technical issue: 1. Your selfie image is crushed because you don't have it in your image file.
2.Your "my work in web tech" link is not working because you don't have the document under your project folder.
3. You haven't created links on your html page that navigate to your other pages.
Content Issue: 1. you should work on the css more for example makes the links look identical to your texts, and adjust font-size to
differentiate subtitles and content,
2. The image breaks through your text which does not look good, may use table to organize it.

[daveyc1234]

Hey yshen, this is not my website. for some reason i still have my old page up and cannot update it. i am talking to professor now

…

On Tue, Nov 8, 2022 at 11:17 AM yshen12138 ***@***.***> wrote: Hi, Dave, I like the idea your are making a website about yourself, but I do think you have a lot to work on still! Technical issue: 1. Your selfie image is crushed because you don't have it in your image file. 2.Your "my work in web tech" link is not working because you don't have the document under your project folder. 3. You haven't created links on your html page that navigate to your other pages. Content Issue: 1. you should work on the css more for example makes the links look identical to your texts, and adjust font-size to differentiate subtitles and content, 2. The image breaks through your text which does not look good, may use table to organize it. — Reply to this email directly, view it on GitHub <#1>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A223UU3O7UDWDA5CJRCFWW3WHJ4IRANCNFSM6AAAAAAR2P6RJY> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[daveyc1234]

It is now working

…

On Tue, Nov 8, 2022 at 11:46 AM Crouch, David ***@***.***> wrote: Hey yshen, this is not my website. for some reason i still have my old page up and cannot update it. i am talking to professor now On Tue, Nov 8, 2022 at 11:17 AM yshen12138 ***@***.***> wrote: > Hi, Dave, I like the idea your are making a website about yourself, but I > do think you have a lot to work on still! > > Technical issue: 1. Your selfie image is crushed because you don't have > it in your image file. > 2.Your "my work in web tech" link is not working because you don't have > the document under your project folder. > 3. You haven't created links on your html page that navigate to your > other pages. > Content Issue: 1. you should work on the css more for example makes the > links look identical to your texts, and adjust font-size to > differentiate subtitles and content, > 2. The image breaks through your text which does not look good, may use > table to organize it. > > — > Reply to this email directly, view it on GitHub > <#1>, or unsubscribe > <https://github.com/notifications/unsubscribe-auth/A223UU3O7UDWDA5CJRCFWW3WHJ4IRANCNFSM6AAAAAAR2P6RJY> > . > You are receiving this because you are subscribed to this thread.Message > ID: ***@***.***> >

[daveyc1234]

If this helps, I know I need help on my table and maybe improving my navigation bar into a dropdown menu. Do you think you could help me there?