Looking to hire developers?
@aishwarya-mali
@Gandah
Submitted
I took me a little while but finally completed this project, there is one issue though I couldn't get jquery to submit data to the thank you page using submit button so it submits on clicking one of the buttons. Please Let me know if there are other ways to approach this. Thanks!
@aishwarya-mali
Posted
Hello,
Congratulations on completing the challenge! You've done a great job. 👏🎉
In regards to your question, I would suggest storing the selected rating and adding a click listener to the submit button. Then, you can append the selected rating value to the "Thank You" component. This will provide users with an indication of the rating they selected and improve their experience on the website.
If you have any doubts or need any assistance in implementing these suggestions, please don't hesitate to reach out to me.
Happy Coding! 😊
@engelbrechtz
Submitted
Hey everyone I would like some feedback. cheers!
@aishwarya-mali
Posted
Hello DECLAN,
Congratulations on completing the challenge! You've done a great job. 👏🎉
I wanted to provide you with some suggestions to improve the user experience on your project:
Happy Coding! 😊
@wpittman06
Submitted
Followed along with a video by Coder Coder and used this video to gauge how much HTML and CSS I knew and understood. I liked her breakdown of the project and understood her rationale of the project. I wanted to submit this so I can look and see how other coders saw this project and understand their thought process.
@aishwarya-mali
Posted
Hi William,
I think you have uploaded starter files from Frontendmentor on both Github and Vercel. Please check once.
Have a Nice Day!
@Oikawasan
Submitted
Want some feedbacks!
@aishwarya-mali
Posted
Hi Toru, excellent work on this challenge!
I just wanted to suggest, you could center your .container class. You could either use Flexbox or Grid for the same.
Keep up the brilliant work!
@otmanezahhari
Submitted
If anybody would like to review my code and give me feedback that would be great!
Any other type of feedback is appreciated as well! Thank You!
@aishwarya-mali
Posted
Hi ZAHHARI,
Nice work on this challenge!
Here are some pointers after taking a look at your code:
I hope this helps. Let me know if you have any questions. Keep it up!
@GeorgeCaldarescu
Submitted
Hi everyone! this is my first challenge and also my first project. I know the code can be better so feel free to give me some suggestions where I can change something. Cheers George
@aishwarya-mali
Posted
Hello George!
Good work on this challenge! Your solution looks great and is responsive as well! I would only suggest to add width to .main class and to reduce font-size, padding and margin.
Happy Coding!
Marked as helpful
@Enol-Igareta
Submitted
Hello, I am new to web development, any feedback with which I can improve would be very grateful. I have used flexbox and tried to use BEM. Thanks for your time
@aishwarya-mali
Posted
Hi ENOL IGARETA,
You have done fantastic job with the challenge. It is responsive as well. I would just like to suggest that you could add width to your .container class and also change the background color. It would look similar to design.
Thank you!
@funnyducksup
Submitted
If you had time, feedback please, what need to change.
@aishwarya-mali
Posted
Hi Mark,
You've done excellent job replicating the design. I would suggest is that on .main_card you could apply Flexbox. So that it will solve your issues regarding responsiveness.
Thank you!
@aishwarya-mali
Submitted
@aishwarya-mali
Posted
Hey Abhik,
Thank you for your advice. I'll surely look into it.
@samzyconcepts
Submitted
I need feedback on what to improve on. Thanks
@aishwarya-mali
Posted
Hi ODERINWALE,
You've done a great job completing the challenge. It is just not responsive on mobile device. You could use max-width instead of width and avoid using heights if you could.
Thank you!
@iivanov-2
Submitted
This is my first challenge and first steps with Web Dev. Any feedback will be much appreciated.
@aishwarya-mali
Posted
Hi IIVANOV,
You've done an excellent job! Learn Flexbox. It will make creating Layouts a lot more easier.
Thank you!
@kennyctran
Submitted
Hi! First thing I've ever built. It's not really responsive :( if anyone has any idea on where I could start or maybe another path I should take so that the build is more @media friendly, please let me know!
I didn't have the figma or anything so I tried the best I could to recreate it by eyeing it!
@aishwarya-mali
Posted
Hi Kenny,
You've done a great job! Instead of using justify-content: space-evenly; you should try justify-content: center; and try to avoid using heights and position if you can.
Thank you!
@GriegoLoco
Submitted
Kindly provide feedback.
Thank you in advance.
@aishwarya-mali
Posted
Hi GriegoLoco,
You've done a great job! You can avoid using position and use Flexbox. Using Flexbox will reduce your worry about Responsiveness.
Hope this helps!
