Skip to content
OverviewSolutions
21
Comments
11
Sadique
@sadiquex

All comments

  • Sadique•450
    @sadiquexPosted 10 months ago

    off to a good start!... but please check the project and the submission's images...thanks

    0
  • Sadique•450
    @sadiquexPosted 11 months ago

    i must say i'm impressed by the way you used reducer functions to create the project!

    1
  • Sadique•450
    @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
  • Sadique•450
    @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
  • Sadique•450
    @sadiquexPosted 11 months ago

    great work rookie:)

    0
  • Sadique•450
    @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
  • Sadique•450
    @sadiquexPosted 11 months ago

    Quite 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
  • Sadique•450
    @sadiquexPosted 12 months ago

    this is awesomely done!, however it'll make things smoother is a transition was put on the hover for the cards

    Marked as helpful
    1
  • MadidaSupreme•10
    @okuhlemadidaSubmitted 12 months ago

    I could not display the text next to the perfume picture all I use 'Display inline'

    html and css

    1
    Sadique•450
    @sadiquexPosted 12 months ago

    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
  • Sadique•450
    @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
  • Sadique•450
    @sadiquexPosted about 1 year ago

    This is great progress Joseph! Awesome work!

    0