@chinh1402
Posted
Well done on finishing the challenge, here are my takes on your solution
- Both description texts and headline texts are black, which gives a strong feeling when looking at it, therefore it just makes the headline feel.. not that interesting. Maybe you should follow the style-guide color pallets of the challenge
- The spacing between headline texts and description texts is too large, consider removing the gap in your text-div class; same goes with the spacing between your headline and the qr-image
- The image is unfortunately not centered, the part to the right of the image is bigger than the part to the left of the image.
I'm not an expert, but my general opinion on this is.. This looks like a solution that was made in a hurry. Take your time, follow the style guide, and as a beginner, you should try to match the design as much as possible.
Everyone has a starting point, hope to see more of your solutions in the future
Marked as helpful