@hariprasad9899
Submitted
Kindly drop your feedback!
Looking to hire developers?
@hariprasad9899
@hariprasad9899
Submitted
Kindly drop your feedback!
@hariprasad9899
Posted
Opps ! due to extended monitor issues ! i am missing some backgrounds and colors
@catherineisonline
Submitted
Hello, Frontend Mentor community! This is my solution to the Testimonials grid section.
I have read all the feedback on this project and improved my code. Due to the fact that I published this project very long ago, I am no longer updating it and changing its status to Public Archive on my Github.
You are free to download or use the code for reference in your projects, but I no longer update it or accept any feedback.
Thank you
@hariprasad9899
Posted
Really likes the way you structured your script and CSS code. I'll try like this !!
@julabina
Submitted
@hariprasad9899
Posted
Hi,
Just a suggestion for precise js. Instead of adding an event listener for every tipstipsAmount[1....5], create a for loop and iterate over it for each tipsAmount in a single block. Like the below.
for ( let i = 0; i < tipsAmount.length;i++ ){
customVal = 0;
customAmount.value = "";
if (tipsAmount[i].classList.contains("left__selectTips__cont__amount--selected")) {
tipsAmount[i].classList.remove("left__selectTips__cont__amount--selected");
tipPercent = 0;
} else {
resetSelectedAmount();
tipsAmount[i].classList.add("left__selectTips__cont__amount--selected");
tipPercent = 5;
}
calcul();
}
this will add an event listener to all the buttons, as you wish.
Happy Coding !! :)
@sansanthi
Submitted
Any feedback and suggestions would be appreciated. It takes me a lot of time to do positioning shadow background image in desktop view.
@hariprasad9899
Posted
you just nailed it with the JS code. Just view my js code for this lol ! I complicated too much and you did it like ' hold my beer ' ! :)
Marked as helpful