Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Your session has expired please log in again.
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Create an object using Object Literal Syntax

Krista Calleja• 300

@KristaCalleja

Desktop design screenshot for the Coding bootcamp testimonials slider coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi, please give me feedback on the quality of my work. I need help to fix the opacity of the image and re-sizing the curve in the pseudo-element. Thank you!

Community feedback

P
Grace• 27,950

@grace-snow

Posted

I can't get this to work as a keyboard user. It's important for events to be attached to interactive elements (should be button not image) and for those elements to have a focus-visible state

0

P
Grace• 27,950

@grace-snow

Posted

Also, on mobile I think you need to check styles again. On portrait and landscape content is being cut off screen. I think this is caused by explicit widths and quite large margins being used instead of max-width and minimal/auto margins.

These bugs may also be caused by not having padding on outer wrappers, so content hits screen edges. And by having a height 100vh on the wrapper instead of min-height (think how small 100vh is when a mobile device is turned to be landscape)

I hope this helps

0
Krista Calleja• 300

@KristaCalleja

Posted

@grace-snow Thank you so much for this. I'll be working on it!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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