Kindly drop your feedback!
hariprasad9899
@hariprasad9899All comments
- @hariprasad9899Submitted over 2 years ago@hariprasad9899Posted over 2 years ago
Opps ! due to extended monitor issues ! i am missing some backgrounds and colors
0 - @catherineisonlineSubmitted almost 3 years ago
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
@hariprasad9899Posted almost 3 years agoReally likes the way you structured your script and CSS code. I'll try like this !!
0 - @julabinaSubmitted almost 3 years ago@hariprasad9899Posted almost 3 years ago
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 !! :)
0 - @sansanthiSubmitted almost 3 years ago
Any feedback and suggestions would be appreciated. It takes me a lot of time to do positioning shadow background image in desktop view.
@hariprasad9899Posted almost 3 years agoyou 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 helpful0