Jumagobe
@JumanjigobezAll comments
- @sanee2025alese@Jumanjigobez
Hey bro, it seems this approach isn't quite in line with the intended use of the platform. Please consider revising your contribution so that it also serves as a valuable resource for future aspiring developers.
Happy Coding :)
- @dhafindzahinWhat are you most proud of, and what would you do differently next time?
I'm proud I was able to built this in less than 12 hour despite not touching css for 4 month.
What challenges did you encounter, and how did you overcome them?Had hard time making it resposive because of my rustiness, but still able to make it work. Definitely not my best resposiveness design.
@JumanjigobezGREAT JOB @dhafindzahin
- @tonia83731What are you most proud of, and what would you do differently next time?
Finally learns how to use useContext in react
What challenges did you encounter, and how did you overcome them?I think for this project, styling is the biggest problem. I do try to use unset but found out didn't work, so I change all style and comes to the final result.
What specific areas of your project would you like help with?@JumanjigobezHey @tonia83731, Congrats on finishing this project it looks great. I also learnt use of Context have you tried out REDUX also, It is also the best and easy in managing of States on all components at once.
Have a great Coding Day Maan....
- @riqqer@Jumanjigobez
Hey, @riqqer Congrats for your successful completion of the challenge.
It's Dope! Wonderful job.
#Don't Give Up
- @Johannes-pries@Jumanjigobez
Hey, @Johannes-pries Congrats for your successful completion of the challenge.
It's Dope! Wonderful job.
#Don't Give Up
- @justinconnell@Jumanjigobez
Hey, @justinconnell Congrats for your successful completion of the challenge.
It's Dope! Wonderful job.
#Don't Give Up
- @Ayoubrabiae@Jumanjigobez
Hey @Ayoubrabiae,
Loved your solution especially on the animation part but Can I kindly give a suggestion on a little adjustment on your code.
I saw a lot of if conditions statements, which are not bad, but I would recommend you check on my Js code on this link -> here where I have used an object of inputs with their properties and looped through each by using the if condition statements. (This will enhance at least better readability and structure of your code)
Hope this was Helpful :)
Otherwise, you are doing a great job to grow in the field.
Marked as helpful - @MuxammedL@Jumanjigobez
Hey, @Muxammedzi Congrats for your successful completion of the challenge. Loved the Idea of LocalStorage. It is a dope idea maan :)
- @Ahmed-dev-98@Jumanjigobez
Sorry, I am unable to pass the phoneNumber field! Kindly check on that.
Hope this will be helpful to counter check your code.
- @ivannetreba@Jumanjigobez
Hey there Ivan,
Congrats on finishing your first challenge :)
I have seen you didn't try on adding the pattern images and also on the carousel section. You are free to check on my code for my solution I did a simple trick using css only to make the pattern images fit their position and also the carousel part.
Otherwise, the solution is responsive that's great.
Hope this will be helpful to you.
Marked as helpful - @tahobbit11@Jumanjigobez
Hey @tahobbit11; Congratulation on completing this challenge but maybe if I could add sth.
When the number is so large maybe if you could place the input box to have a css of
overflow: scroll
so that the number doesn't go outside the input box. input I tested with 6500000000000000000000000000000I inspected your code and the below is how you should have done on the input part:
#input{ width: 100%; /*to fit the wrapper container*/ overflow: scroll; /*enable scroll when the input is so large*/ }
If the scrollbar is huge and looking bad you can add the following in your css:
*::-webkit-scrollbar{/*the size of the scrollbar*/ width: 5px; height: 5px; } *::-webkit-scrollbar-track{ background: rgba(0,0,0,0.2); /*the background color of the scroll track*/ } *::-webkit-scrollbar-thumb{ background: #fafafa; }
Sorry for the long text :) otherwise I hope this will be helpfull to you.
Kudos and Happy Coding Bro.
Marked as helpful - @jonesmarquelle@Jumanjigobez
Perfectly done bro, Keep Coding. I like the idea of how you placed the countries container into a scrollable mode :)
- @J11tendra@Jumanjigobez
Nice One Bhai keep on coding I like the way you made the hero picture be perfectly set, not like mine :)
- @hackerYuvrajSingh@Jumanjigobez
Congrats @hackerYuvrajSingh, I love how you were not confused with the colors not like me, but I had a suggestion maybe making the numbers greater than a thousand to have commas and that is easy (i.e 1,000).
Just checked out your code in the
index1.js
inside thebutton.click(function())
. there is a conditionif (buttonElm === '='){ screen.value = eval(screenValue); // just add .toLocaleString() function and it will do for you :) }
screen.value = eval(screenValue).toLocaleString();
You can check out my solution also on this to check my code, but I was confused on the colors :)
Otherwise, Kudos, Keep Coding Keep Pushing.
Hope it was helpful :)
Marked as helpful - P@Lisviks@Jumanjigobez
Wonderful, Love the use of Signing Up that's big thing maan, Thumbs Up
Keep Coding Keep Smiling 😎
- @AlgirdasU@Jumanjigobez
Hey @AlgirdasU, Your solution looks great, I tried entering a domain, but it didn't go through.
I just also viewed your question problem, it was also disturbing me, but I did a small trick with the URL of the Api, I would suggest if you can view my code on this challenge on my js script code you can find the url trick I placed.
Otherwise, it's great and Hope this will be helpful.
Marked as helpful - @AdrianoEscarabote@Jumanjigobez
I love the solution you did, but I have a suggestion as in, the Number of people you could have made it to be only whole numbers not a decimal like I tried placing 1.5 and it calculated, so the logic gets lost.
I reviewed your index.html and I so on the input you place
type="text"
, Maybe you can try it to betype="number"
and check it out maybe :)Otherwise it is wonderful it calculates nicely only the logic of number of people doesn't make sense.
Hope this will be helpful.
Have a nice day coding, Don't Give Up :)
Marked as helpful - @Juanies@Jumanjigobez
Hello Juanies I checked out your solution it was great but the submit button should not submit unless I have choosed a rate value.
Through viewing of your script code in:
submit.addEventListener('click', () =>{ tarjeta.style.display = 'none'; thanks.style.display = 'flex'; nota.textContent =
${value}});
In the above function I think you should have placed an if condition that checks if a rate is selected then proceed to thank card page else maybe alert or disable the button.Kindly checkout my solution on this challenge and check out my script code.
Otherwise Hope this is Helpful and keep coding :)
Marked as helpful