Design comparison
Solution retrospective
Open for suggestions
Community feedback
- @Augs0Posted over 4 years ago
Really great job overall!! You managed to hit the design spec almost perfectly :) It's cool to see how everybody tackles the challenge.
If I were to offer any suggestions, it would be to maybe add another media query. Right now, everything up to 900px has the same rule-set, as such, if you drag the screen to a mobile-ish size, you'll notice your text gets hard to read as it sits on the bottom curved shape and they are more or less the same colour.
2@jayaspiyaPosted over 4 years agoThank you for your suggestion i will try to implement them in my future projects.
0 - @ovidiuantonioPosted over 4 years ago
Hello,
Very nice solution so far! The site is responsive and the phone illustration looks very nice! There are a few things that you can improve:
- add
cursor: pointer;
to all the elements that are clickable (back arrow, text field, send message button etc) - for the text field use an input element instead of a div, for the send button use a button element, for the option buttons use radio buttons. In short, make the app "usable" so the user can select one of the options, click the buttons, enter a text
Happy coding! Keep going!
1@jayaspiyaPosted over 4 years agoThank you for your suggestion i will try to implement them in my future projects.
0 - add
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