Sadique
@sadiquexAll comments
- @Eleniyan-codeSubmitted 10 months ago@sadiquexPosted 10 months ago
off to a good start!... but please check the project and the submission's images...thanks
0 - @goody-devSubmitted 11 months ago
Thanks for checking out my solution! Pls, suggest ways in which the solution can be improved.
@sadiquexPosted 11 months agoi must say i'm impressed by the way you used reducer functions to create the project!
1 - @juliendymendesSubmitted 11 months ago@sadiquexPosted 11 months ago
i created a new branch with the solution on it :p https://github.com/sadiquex/advice-generator--i-m-contributing-
0 - @juliendymendesSubmitted 11 months ago@sadiquexPosted 11 months ago
hello @juliendymendes, i have some contributions to help in vertically aligning your card, i think putting a 100vh and display:flex; align-items:center on the body is going to fix the issue. thanks!
0 - @Brenda-MSubmitted 11 months ago
- @Shafiquedev256Submitted 11 months ago@sadiquexPosted 11 months ago
hello @Shafiquedev, this is great effort, however, the app when the last button is clicked on, this error pops up...>. You might want to check that. Happy coding :)
index-1w_SmaCl.js:40 TypeError: Cannot read properties of null (reading 'planEl') at Bd (index-1w_SmaCl.js:40:72242) at yi (index-1w_SmaCl.js:38:19478) at Ga (index-1w_SmaCl.js:40:43712) at Ka (index-1w_SmaCl.js:40:39513) at vd (index-1w_SmaCl.js:40:39444) at qr (index-1w_SmaCl.js:40:39302) at Do (index-1w_SmaCl.js:40:35719) at $u (index-1w_SmaCl.js:40:36520) at ht (index-1w_SmaCl.js:38:3258) at index-1w_SmaCl.js:40:34066
0 - @TimmyOGHSubmitted 11 months ago
Any encouraging feedback is appreciated.
@sadiquexPosted 11 months agoQuite impressive you made this with vanilla, but i was having a problem with the phone number input field, when i entered a ghanaian number, it still indicated that it's an 'Invalid Phone Number'
0 - @Yup03Submitted 12 months ago
Fun Challenge ✔✔✔ I'm down for any improvement
@sadiquexPosted 12 months agothis is awesomely done!, however it'll make things smoother is a transition was put on the hover for the cards
Marked as helpful1 - @okuhlemadidaSubmitted 12 months ago
I could not display the text next to the perfume picture all I use 'Display inline'
@sadiquexPosted 12 months agohey @okuhlemadida I think the reason why the image is showing on your local machine and not the submission is because you didn't push the /image folder to github
0 - @edilan-ribeiroSubmitted 12 months ago@sadiquexPosted 12 months ago
Hello, edilan-ribeiro, I think setting a min-height: 100vh on the background image of the technology page will solve the problem with the vertical overflow
0 - @theboyceSubmitted about 1 year ago