Skip Navigation
14 comments
    1. Site wasn't properly reflexive for mobile
    2. If this is a portfolio then i would remove a lot of stuff like "watch list" and "current obsession". The focus should be on your work and future projects
    3. Notes are ok for a start but can be improved. I think a "posts" or "blog" would be better section title, and the content should try to teach something you've learned rather than be the notes you took for a subject. The difference is that teaching reinforces your understanding of the topic. So pick something smaller from those topics and teach it. I wouldn't redo your current notes necessarily, but going forward i would pick a more focused topic and teach.
    4. i would then move the "blog" or "posts" to your front page to show the most recent content and then link to /posts where the rest of it can be found. Or highlight projects on front page instead depending on what you want focus to be.
    5. move your front page content to a more "resume" section that includes a section for the tools you know. And still think about the length/space of this page. Like a printed resume, too long is bad. So make sure it outlines things nicely

    Overall if it was just a personal site id say its ok. But as a portfolio site you have some work to make it align with your goals. Good luck!

  • My comments from someone who doesn't know much about programming yet:

    • Make the "get in touch" button functional
    • The "Language" category could be named "Experience working with:" Or something more professional
    • Search for better logos for the programming languages you know, and make them smaller
    • Many elements have contrast issues, black text on very dark blue backgrounds, etc. That's very very important
    • Cursor doesn't change on top of links

    Idea:

    • The landing page could contain some pictures of your best websites or more visually pleasing projects. And if this is a personal website "watchlist" and such are okay, but if it's meant to be a portfolio for companies... Keep it professional

    Overall, it was still okay, but needs some polishing

  • First thing I noticed is that your navbar links don't behave like links. This might be related why reloading the page takes you to the main page. Are you using react-router? You might no be using it correctly

    Is it normal to doxx yourself putting your address online these days? Isn't it better to have the city you are located?

    I wouldn't link to all my social networks on my professional page, unless you are 100% sure you aren't saying anything that might make the recruiter be biased.

    Having a PDF like someone else mentioned! Resumes usually go past RH people and they are never willing to look more than 10 seconds for the information they need.

    • Might not want to share your personal information publically - That information is PII and in western countries need to be handled with care.
    • When I hover my mous on the top menu my mouse shape is for text, not clicking
    • The cards are of different sizes. Makes it look weird.
    • Look into Polars. Read their User Guide. Better API (no weird use of brackets[[]]), faster, nicer to work with than Pandas (and can still convert to Pandas, just in case).
    • Not a big fan of all the

      <div>

      <div>

      <div>

      <div>

      - Use

      <nav>

      ,

      <article>

      (semantic HTML). You can find details on the HTML spec page: https://html.spec.whatwg.org/dev/ (you should look at chapter 4). Maybe also looks into CSS Box Model instead of using flex all over the place :)
14 comments