Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Request path contains unescaped characters
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

HTML CSS JS

Alex 80

@TheAlex01

Desktop design screenshot for the Article preview component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello community, any suggestions and feedback are welcome.

Community feedback

Eric Salvi 1,330

@ericsalvi

Posted

Hey looks good.

Out of curiosity, did you purposefully set the social sharing pop-up to be open when the page loads? If so, that just seems buggy. Typically you'd want them to be hidden unless the visitor clicks the sharing icon.

1
Alex 80

@TheAlex01

Posted

Hello thanks you for feedback. Yes, left it pop up for scrinshot )

0

Eric Salvi 1,330

@ericsalvi

Posted

@TheAlex01 Got it. I would honestly set it back to only show when clicked. I like adding the different states in the readme file in Git repository.

Looks like you didn't remove the current README.md file and rename the README-template.md to be just README.md. This is where you'd want to post screenshots of your work so when someone wants to look at your code, they can read about what this project was about. Check out one of mine to get an idea of what I am referring to.

0
Alex 80

@TheAlex01

Posted

@ericsalvi Done :) thanks you

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