Hi it's me, bunee! I really wanted to learn CSS grids, I figured this challenge would be a great start.
I've learnt quite a bit solving this challenge. As embarrassing as it is to admit, I spent more time centering the testimonials container than learning about grids themselves... Shout out to @tediko, His solution really helped me solve this issue.
Not to sound cheesy but, I need some good resources on "how to center a div" :')
Also if you could take a look at my code and give me some feedback, I'd respect it. Also also, Let me know how you feel about the animations.
Phew! Took me a good minute to complete this one. I've learnt a lot whilst coding this. Even though I have used a lot of Hacky ways(if you will) to position most of the elements on the page, I still have a ton of room for improvement :') If you could provide me some Feedback, I'd respect it. Thank you.
I had a ton of fun solving this challenge... So much so that, I spent extra time adding a few personal elements ^.^
If you do decide to go through my code, I would whole heartedly accept your feedback. I personally feel like my markup needs some work.
Hey! This is my first ever solution to a challenge on this website, I thoroughly enjoyed coding this. Hope to keep learning more!βΊ
I would truly appreciate some feedback but, I must warn you. My code isn't the most prettiest code you'll come across. It's super inefficient and, You may lose brain cells trying to understand it. If you do decide to go through my code regardless and give me feedback... I RESPECT you!
Hey! @AminaArif0005, Great job on this challenge, Your site responds pretty well. Here are my suggestions:
box-shadow: 0px 32px 43px rgba(79 166 175 / 20%);.That is all. Hope this helps :)
P.S. Why did you use a heading tag as the logo instead of the SVG? I'm curious.
Hey! @Alice-Githui, Congrats on completing this challenge! you've done good. Here's a couple of things that I want to suggest...
background-image: url("link to your svg");
background-position: 5%;
background-repeat: no-repeat;
background-size: 10px;
Your buttons are missing focus styles which is needed for accessibility.
You also need to implement the active style for the buttons. This video shows you one way of achieving that.
If you're having trouble creating responsive layouts, I strongly recommend This free course by Kevin Powell. This course should help you gain a better understanding of responsive layouts.
Hope this helps :)
Hey @catbennett, Congrats! on completing this challenge, Your page looks great and responds well. The only thing you're missing is the box shadow on the card itself.
If you're having trouble using flexbox, I suggest you take This free course by Kevin Powell. This should help you gain a better understanding of them.
Hope this helps :)
Hey @Simonecyber, Great job on completing this challenge, your page responds very well! The only thing you're missing is the box-shadow. Happy Coding :)
Hey @Anna60991, Your page looks great and is completely responsive, Great job!
However, I did notice a bug. you can experience this by clicking right under any of the questions. doing this reveals the answers of every question and gets quite tricky to toggle them back off. I think the reason behind this is the event listener on the li tag, If this is the case... I think adding the event listener to the div with the class of question should solve this issue.
Hope this helps :)
