Stored the dynamic data of the slides in the javascript file.
Design comparison
Solution retrospective
When clicking the buttons to change the slides, there is a square box behind the image (.image-wrapper)...Tried to blend this with the rest of the page but it stands out with a different color. Couldn't find an easy fix.
Note: Added an additional testimony to test it out
Community feedback
- @mattstuddertPosted over 4 years ago
Hey Aju, to avoid that square box I'd recommend putting the
box-shadow
on theimg
element instead of the.image-wrapper
. You're moving theimg
when the slides change but keeping the.image-wrapper
where it is. That's why you're seeing the box.Nice work on this challenge. My main recommendation beyond changing the
box-shadow
would be to go through the CSS one more time matching your solution up to the design to try and get your project as close to the design as possible. Accuracy is a key part of being a front-end developer, so it's a skill that's well worth taking some time to build.I hope you're enjoying building the project!
0@ajujayapalPosted over 4 years agoThank you. That worked. I'll update the solution and bring it as close to the challenge as possible. Thank you again.
0 - @emestabilloPosted over 4 years ago
Hi @ajujayapal, it's the
box-shadow
property on the.image-wrapper
that is causing the div to stand out. Perhaps applying it directly to theimg
tag would solve your issue. Hope this helps :-)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