• Calum SmithCalum Smith, 4 years ago

    Until it is responsive, I'd definitely take out the viewport meta tag, as it's preventing zooming out on mobile. Also the "Say hi! More about me." CTA at the bottom of the homepage is kind of unclear on what it's going to do—I had to check the link to make sure I wasn't about to email you.

    All in all, though, really fun, well put-together design.

    7 points
    • Kyle Decker, 4 years ago (edited 4 years ago )

      Solid points, thank you.

      The "Say hi!" CTA actually did link to an email at one point, but some peoples' moms get pissed when Outlook opens unexpectedly and they can't figure out how to close it. ;) I'll change the verbiage.

      Edit: The viewport meta tag has been removed for the time being. Scale away. Edit 2: 'About' CTA text has also been clarified.

      1 point
      • Jacob TaylorJacob Taylor, 4 years ago

        but some peoples' moms get pissed when Outlook opens unexpectedly and they can't figure out how to close it.

        Be prepared for a lot more of this when you land an actual job. You'll be designing for the lowest common denominator, and this sort of stuff will be a constant consideration.

        3 points
        • David McGillivrayDavid McGillivray, 4 years ago

          I get pissed when an email client opens and it's not been clear it was going to do that, and I'm no-one's mom.

          Also, my mom is not the lowest common denominator.

          2 points