React-Powered PWA Product Feedback App
Design comparison
Solution retrospective
Please review on both mobile and desktop and let me know what can be improved on, or added. Better still, add the feedback within the app too, after all it's a feedback app 😀
To access the app, you have to sign up, but you don't have to use any real details, as long as the email is the correct format it'll work
Community feedback
- @grace-snowPosted about 3 years ago
Hello
I'll give feedback as requested on the html / css in browser, but don't do React so can't really comment there :)
- width: 100vw; is causing a horizontal scrollbar
- I recommend making focus-visible styles more consistent, and much more obvious on button
- no page heading or landmarks so it's not clear what this site/page is, let alone its content
- tab interface is not structured like tabs. I'd expect role tab and tabpanels and buttons to be the triggers not anchor tags. This all means at the moment, content changes are not communicated to assistive technology at all. The anchor tags imply navigation but there is no indication that anything has happened or content changed on click
- error messages are not linked to inputs with aria-describedby on input
- hint/helper text is not linked to it's input with aria-describedby on the input
- error messages are not announced to screenreaders with aria-live on error
- there's no required/aria-required on required inputs
- I am seeing a modal popup that says app ready to work offline, but this is hard to see due to lack of contrast with background/overlay, isn't announced to assistive tech, nor is focus moved/trapped and it auto-dismissed. All of this in combination makes it very confusing
I can't get any further as it's not letting me past signup/login. Lots of console errors (I'm in firefox)
I hope this helps for starters
Marked as helpful1@frontendtonyPosted about 3 years ago@grace-snow This is highly appreciated. Giving myself a face palm 🤦♂️ for not realising some of the things you mentioned, especially on input fields. I'll be fixing all these and be assured I'll make a point to not miss them in my next project/challenge
You can sign up with any random email address though. I just opened this in Firefox but can't reproduce the console errors. Curious to see what those are and get them resolved
0@grace-snowPosted about 3 years ago@frontendtony I definitely can't get past signup. I just get the app is ready to work offline (on mobile chrome now)
0 - @mbart13Posted about 3 years ago
I managed to log in, even left feedback there :)
but I'm also getting 406 errors in the console, some request keeps firing
I'm also not able to upvote comments, getting 409 error
Marked as helpful0@frontendtonyPosted about 3 years ago@mbart13 Thank you for the feedback, I'll take a look this weekend. PS: You can only upvote once so a 409 most likely means you've upvoted the same feedback before
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord