DEV Community

Discussion on: Can I get some portfolio feedback.

Collapse
 
moopet profile image
Ben Sinclair

Looks pretty good. It does everything I imagine you want it to and doesn't have any obvious issues except a couple I'll list.

Accessibility

  • The way the background image moves just a little bit under the main CTA, in combination with the animated typing text gives me motion sickness.
  • Same but less so with the way things ease into view.
  • When I use the tab key to navigate I can't see what has focus. I have no idea where I am on the page.
  • The overlay on portfolio items has poor contrast in some places (see the WIX logo on "Unintentional Purpose" for example)
  • The "view design" popup doesn't let you zoom and is... a popup (popups baaaad). This means we don't really get to see it very well unless we have a 2m tall monitor (see the popup for "Comfy Libra" for example)

Your CSS does have some support for prefers-reduced-motion, but it only covers some aspects (like smooth scrolling) and not others. Everything is on by default, and this means that users on systems where it's not well-supported (so 95% of visitors) will see the transitions and movements.

Usability

  • Make the email and phone links into... links.

Privacy

  • I'd probably not want my personal phone number on my website - your choice, but...
Collapse
 
dwa_1145 profile image
Dwayne

This is awesome! Thanks for the feedback.