Sadique
@sadiquex
All comments
- Sadique• 450
@sadiquex
Posted
off to a good start!... but please check the project and the submission's images...thanks
0 - Goodness Okanlawon• 80
@goody-dev
Submitted
Thanks for checking out my solution! Pls, suggest ways in which the solution can be improved.
Sadique• 450@sadiquex
Posted
i must say i'm impressed by the way you used reducer functions to create the project!
1 - Juliendy Mendes• 90
@juliendymendes
Submitted
Sadique• 450@sadiquex
Posted
i created a new branch with the solution on it :p https://github.com/sadiquex/advice-generator--i-m-contributing-
0 - Juliendy Mendes• 90
@juliendymendes
Submitted
Sadique• 450@sadiquex
Posted
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 Muthoni• 230
@Brenda-M
Submitted
Sadique• 450@sadiquex
Posted
great work rookie:)
0 - Musinguzi Shafiq• 90
@Shafiquedev256
Submitted
Sadique• 450@sadiquex
Posted
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 - MadidaSupreme• 10
@okuhlemadida
Submitted
I could not display the text next to the perfume picture all I use 'Display inline'
Sadique• 450@sadiquex
Posted
hey @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-ribeiro• 200
@edilan-ribeiro
Submitted
Sadique• 450@sadiquex
Posted
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 - Joseph Adofo Osei• 40
@theboyce
Submitted
Sadique• 450@sadiquex
Posted
This is great progress Joseph! Awesome work!
0