Design comparison
Solution retrospective
I am unsure if the responsiveness is good enough.
Community feedback
- @enesekenPosted over 1 year ago
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 ! 🎉
Marked as helpful0@jonhenrikaavitslandPosted over 1 year ago@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@enesekenPosted over 1 year ago@jonhenrikaavitsland You should nested a tag in button, sorry about confusing
Solved Problem
<button> <a> </a> </button>
0@jonhenrikaavitslandPosted over 1 year ago@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 - You should change
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