Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
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

I only used VSCode and Figma

@jonhenrikaavitsland

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I am unsure if the responsiveness is good enough.

Community feedback

Enes Eken 540

@eneseken

Posted

Hello coder 👋

  • You should change <a></a> to <button></button>
  • Also you can add "transition" to <button></button>

Here is some tips for you 💡

button{
border: none;
background-color: var(--dark-cyan);
padding: 16px 0 16px 0;
border-radius: 6px;
display: flex;
justify-content: center;
transition: .3s all;
}

For more about transitions you should visit here

Good Job ! 🎉

Visit My Website 🌐

Marked as helpful

0

@jonhenrikaavitsland

Posted

@eneseken Hi and thank you for your reply. At school, they teach us to use <a></a> for this kind of button. And so I think I will stick to that for now, but I appreciate the feedback and will keep it in mind for future projects. I have learned a little bit about transitions at school already, and I might add that for future projects.

0
Enes Eken 540

@eneseken

Posted

@jonhenrikaavitsland You should nested a tag in button, sorry about confusing

Solved Problem

<button> <a> </a> </button>

0

@jonhenrikaavitsland

Posted

@eneseken Thanks, I will discuss this with my teacher and see what he thinks as it is a little different than what we have learned so far.

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