Design comparison
Solution retrospective
Any comments and/or suggestions are welcomed. Thanks in advance.
Community feedback
- @artimysPosted about 4 years ago
Nice job BlackTech718, responds nicely
Some observations:
- add a
top
value to your quotation background position.background-position: top right
. If the right position sticks too close to the container edge add a px value after the value right.
// example background-position: top right 30px; // play with value
- I think the
box-shadow
can be improved. Move the shadow a tad to the right on the x-axis and maybe play with the spread.
box-shadow Tip:
I've noticed some focus on the amount of blur and color and not so much on the positioning of the shadow. When writing the values focus on just the first two values for x and y axis without the blur and color. You'll be able to visually see the shadow without a blue. Once you find your position apply blur and spread if needed. Followed by your color
Keep it up!! 👍👍
1 - add a
- @ApplePieGiraffePosted about 4 years ago
Hey, Fernando B! 👋
Great work! Your solution responds well! 🙌
I just suggest setting a
max-width
on the main container or wrapper so that the content of the page doesn't stretch too much on extra-large screens!Keep coding (and happy coding, too)! 😁
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