• Your header could use some work. Your name is too small in the logo and I can hardly read it at all on mobile. I don't see the point in having a hamburger menu on mobile, either, since you only have the two links.
  
• Lots of the layout and content is kind of trite and won't stand out in a crowded applicant field. I don't know how many times I've seen "Hi, I'm X" or those three columns with ugly SVG graphics atop them.
  
• You should choose a typeface that's not just the standard create-react-app font-family.
  
• I like the pink, but you need to be more careful with how you use it on the page with the copy; the pink subheads don't pass a color contrast checker.
  
• Check your line-spacing consistency; the first block and the second block appear to have different values.
  
• Back to the cursed three column layout, I don't think it makes sense for you even from a content perspective. You're presumably applying to be a software developer, so I don't know why hierarchically it makes sense to include "Designer" and "Mentoring" as equally weighted facets of your skillset. And if I'm being brutally honest, those two columns are doing a lot of work to highlight your inexperience. Listing skills like "Pen and paper" and "rubber ducking" just screams to me that you're padding out a page with nonsense because you don't have a large body of work to fall back on.
  
• I'd dedicate a lot more real estate towards your portfolio. I don't like having to click in to see your projects. Could you embed images directly on your page? Or better yet, add some write ups about the work you did on them? It doesn't help that it seems like a lot of your projects require me to register to see them. And I don't know why you're linking to a Google Drive file instead of, like, embedding or linking to a YouTube video.
  
• As prom candy alluded to, might be worth getting someone to help you with some quick copy editing. No major problems, but I agree with his comment re: "Thousands" and you have a lot of extra/misplaced commas and inconsistent cases.
  
• Back to the technical side of things, check your semantic markup more closely. You're leaning on <div>s too much and you have multiple <h1>s on the page.
  
• You're missing alt tags on your images.
  
• Putzing around in the inspector, it doesn't appear like you leaned too heavily on a CSS framework, which is great! That said—and this is just a personal thing—I hate the Bootstrappy utility classes for responsive columns. If you went full Tailwind or whatever I can see it a bit more, but I really put a premium on people being able to use flexbox and grid well and the .col-lg-4 approach to me just strikes me as sort of a shortcut.
  
• Nitpicky-ish, but I don't like to see !important's in your CSS.
  
• Again nitpicky-ish (and possibly a dated piece of advice?), but I'm not sure CRA is the right tool for the job here because it won't generate SSR pages. You probably don't need React at all for a site this simple, but if you want to use it, maybe worth giving a glance at a Gatsby or Next.js or something.

• In spite of me calling various parts of your design "trite" or "cursed", The Dave is right in that your portfolio's quiet competence automatically checks boxes maybe half of the applicants' don't. We hired someone recently (who's been great!) with a portfolio that was considerably less attractive than yours. We also had a ton of applicants with no portfolio site at all.
  
• One thing I always hone in on is side padding for container elements and I was pleased to see you had those!
  
• Outside the accessibility-related concerns I had, your Lighthouse audit score was superb.
  
• Too often I see newer developers list 20–30 different techs they claim to have expertise in and it was nice to see you keep your list to six. It's hard to get an accurate picture of an applicants' skillset when they're fresh out of bootcamp and purport to know React, Vue, Svelte, Postgres, MySQL, MongoDB, or when they fill a list up like that with silly things like "AJAX" or "JSON".
  
• Similarly, I liked that you listed the techs you used in an organic way in both your portfolio blurbs and in your site's footer.
  
• Logo design is totally impossible but I like the bones of yours. I think if you just removed your name and centered and stretched out the remaining elements it could work.
  
• Nice to have a contact form directly on your page.
  
• A big one: your personal projects look like they're actually personal. Too often people include things that were either obviously their bootcamp assignments or the final result of a coding tutorial. Being able to apply what you've learned to a bespoke project is huge.

fsif fucked around with this message at 15:40 on Dec 5, 2021

<style>
.container {
  position: relative;
  width: 100%;
  padding-bottom: 56.2%;
}
.container video {
  position: absolute;
  width: 100%;
  height: 100%;
  top: 0;
  left: 0;
}
</style>

<div class="container">
  <video>...</video>
</div>

kedo fucked around with this message at 19:47 on Dec 6, 2021