New website live

almost 5 years ago from Phil Stringfellow, Web & UI Designer

  • Daniel WinterDaniel Winter, almost 5 years ago

    Overall: pretty good!

    Small notices as you asked for feedback: Please add active/hover states for all links. Maybe just a different background color for buttons, underlining or whatever for nav links. The feedback is a bit too minimal for the eye imho.

    Will anyone hire you because you place HTML5 and CSS3 Icons? I don't think so. It is almost mandatory to have these skills these days, therefore... just drop them.

    Get a header for the beginning of your portfolio, it just starts with the Zelda Timeline, without any explanation why this is here. Open the links in target _blank and link the image as well.

    And maybe let the header hide behind the line:

    from this Imgur

    to this Imgur

    3 points
    • Charlie HieldCharlie Hield, almost 5 years ago (edited almost 5 years ago )

      Try adding the following CSS:

      h1.tlt::after { content: ''; position: absolute; left: 0; right: 0; bottom: -12px; height: 21px; background: #9f0ae5; }

      0 points