Updates and notes

[cS2622]

• changed About in site navigation to Home
• change row-cols-2 to just row, row-cols-2 means that any cols within it are two equal width but you only put the picture in a col, not any of the text - so this is not necessary

[netlify]

✅ Deploy Preview for careybarry ready!

Name	Link
🔨 Latest commit	355d1b9
🔍 Latest deploy log	https://app.netlify.com/projects/careybarry/deploys/6a1526f90a928600087586c5
😎 Deploy Preview	https://deploy-preview-2--careybarry.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
🤖 Make changes	Run an agent on this branch

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[cS2622]

I am going to put all of my notes for your site in comments on this pull request. I hope that makes them easy to understand! You did a great job :)

[cS2622]

Index.html

• Very clear commenting
  • I appreciated how the SEO metadata was concise and informative

[cS2622]

Index.html

• I like how you organized your header with the separating lines and had the social media and contacts at the very top of the page.

[cS2622]

Index.html

• I think that to make the site header a bit easier to distinguish from the section links (About, Education, Photos) you should move your name header and pronouns beneath the site nav bar and rename About to Home in the site nav bar. I was initially confused why About was repeated in both nav bars before I realized that one linked to the home page and one linked to the About section specifically.

[cS2622]

Index.html

• I was a little confused on your three sections having different structures. I would recommend if you want to use the bootstrap grid system to make sure that you consistently use it. For example, the about me section was not its own container and just a row in the larger page container. This is also a good place to play around with how different information is presented visually.

[cS2622]

Index.html

• I think you did a really good job organizing the images you included, only thing I would maybe change is making the alt descriptions a bit more literal. Ex. Changing “A sign on campus with surrounding flowers” to “A wide sign that says ‘Northeastern University’ surrounded by pink and green flowers.”
• I am also really interested in photography so it was cool to see some of your photos!

[cS2622]

teaching.html

• I moved script for js bootstrap to right before /body and add defer tag - move dthis so that the site loads more efficiently and loads JS after the HTML
• I think you did a good job with the inclusion of the two videos and all of their attributes making sure that they can be watched smoothly on the site or just linked to YouTube.
• In the future could add more detailed descriptions of what you do for the various course topics you list
• Similar to Professor Gomez’s site you could add the specific classes you’ve taught during your times as a professor

[cS2622]

publications.html

• I think you did a good job guiding the Claude agent to ensure all of the information was accurate and matched your existing color palette
• I think your use of Claude to organize all of your research publications drawing from your cites through Google Scholar was a very efficient way to use it, and I especially like the filter feature that used a bit of JS.

[cS2622]

style.css

• I think you did a good job at having a consistent style (color and font wise) across the entire site (excluding the publications tab which was separate). I think if you wanted to further improve the site in the future you could experiment some with more visually interesting colors/shapes for the header.

[cS2622]

Design Document

• Your design document has well designed wireframes and user personas but is missing user stories. For future design documents I recommend also including user stories, the structure that I normally use is, “As a [persona], I [want to], [so that].” An example for you (just based off of the info on your home page) could be as simple as, “As a new student in the NU Physician Assistant Program I want to learn more about my professors’ experience so that I can ask them about research I am also interested in.” I also have found that attaching acceptance criteria to user stories can help organize the actual technical features that might be necessary for achieving the user story (ex. “I can switch to a tab that outlines research publications”).

[cS2622]

ReadMe

• I think you did a really good job with your readme file, I really like the gifs you included (I didn’t manage to figure out how to do that for my project but I will definitely be trying in the next one!) and everything was well organized.
• While I did like how you selected custom colors for your website design I am excited to see what creative features you could add with some JavaScript in the future!